Feature #9040: Encapsulate config.rb into TestSuiteConfig class

Feature #9040

Encapsulate config.rb into TestSuiteConfig class

Added by anonym 2015-03-09 13:52:11 . Updated 2016-10-07 13:34:01 .

Status:

Confirmed

Priority:

Normal

Assignee:

anonym

Category:

Test suite

Target version:

Start date:

2015-03-09

Due date:

% Done:

Feature Branch:

Type of work:

Code

Blueprint:

Starter:

Affected tool:

Deliverable for:

Description

Similar to Feature #9030, it’s potentially problematic to define stuff in the global context except classes and modules. We could fix this pretty easily by implementing a TestSuiteConfig class that inherits from Hash, and does the YAML stuff in initialize and then do $config = TestSuiteConfig.new in the global context (which is fine). That wouldn’t require any change to code using $config anywhere else.

We probably should move the constants into this class too, which would require e.g. TAILS_ISO to be changed to TestSuiteConfig.TAILS_ISO which actually looks like a good thing since it’s easier to understand that the constant is part of the test suite’s configuration.

Subtasks