Project

General

Profile

Refactor #33782

Tracker #29939: Improve setting definition DSL and move setting registry to memory

Initialize Setting for tests without fixtures

Added by Ondřej Ezr 8 months ago. Updated 4 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Settings
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Do not use fixtures for tests, use the defaults where possible.

This came from https://github.com/theforeman/foreman/pull/8802#issuecomment-951089137


Related issues

Related to Foreman - Bug #34592: Remove max_trend setting test fixtureDuplicate

Associated revisions

Revision 9b83dfe5 (diff)
Added by Ondřej Ezr 4 months ago

Fixes #33782 - do not use fixtures for Settings

As we no longer require settings in DB, we may drop the fixtures for settings.

History

#1 Updated by Ondřej Ezr 8 months ago

  • Parent task set to #29939

#2 Updated by Ewoud Kohl van Wijngaarden 8 months ago

  • Description updated (diff)

#3 Updated by The Foreman Bot 5 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/9042 added

#4 Updated by Ewoud Kohl van Wijngaarden 4 months ago

  • Related to Bug #34592: Remove max_trend setting test fixture added

#5 Updated by The Foreman Bot 4 months ago

  • Fixed in Releases 3.3.0 added

#6 Updated by Ondřej Ezr 4 months ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF