action #159576
closedcoordination #151990: [epic] Optimize test coverage
Remove test data from console/scc_cleanup_reregister.pm
0%
Description
Motivation¶
As settings seems to be the preferred way to do this, because they are set in mediums and we might need even pair of settings from two different products in the same job, for example for migration, we can remove this functionality. Referring to this:
my $addons = get_test_suite_data()->{addons}; in console/scc_cleanup_reregister.pm
Next function will use SCC_ADDONS by default internally.
Check where this was used with test data and change it so we simplify this module.
Acceptance criteria¶
- AC1: Addons are provided via settings and not via test data mechanism.
- AC2: Other uses of this test module where test data is passed are changed to use settings instead.
Updated by tinawang123 8 months ago
- Status changed from Workable to In Progress
- Assignee set to tinawang123
Updated by tinawang123 8 months ago
PR: https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/19198
At register_addons_cmd, use get_var('SCC_ADDONS') by default.
Updated by JERiveraMoya 8 months ago
- Status changed from In Progress to Rejected
I'm working on this, sorry for the colision, I forgot I created it.
Anyway this test module is used with test data so cannot be updated so easily.