[y] Refactor 'validate_addon_repos' and 'validate_mirror_repos' test modules
parse_repo_data function was introduced in
lib/repo_tools.pm, which allows to parse the information about repository and provides access to it. Though there is the
validate_repo_enablement function which performs the validation, but it's a little bit limited (e.g. all the values are hard-coded, no way to verify only specific values). So it was decided to remove the
validate_repo_enablement and replace it with
validate_repo_enablement is used only in 'validate_addon_repos' and 'validate_mirror_repos' test modules, currently (please, double-check this).
Not to mix openQA variables and test data we can do following:
- Allow expansion of the variables in the test data, like %VERSION% (see https://github.com/os-autoinst/os-autoinst-distri-opensuse/blob/master/lib/scheduler.pm#L85 ) We should also add unit tests for testing this functionality
- Define everything as openQA variables
validate_repo_enablementis replaced with
parse_repo_datafunction in all test modules where it is used.
- Status changed from In Progress to Feedback