When going through logs I found out that test loading
shows some warnings. This corrects them.
Details
Try to schedule all tests again
Diff Detail
- Repository
- rOPENQATESTS os-autoinst-distri-fedora
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
lib/fedoradistribution.pm | ||
---|---|---|
17–19 | maybe a small comment to describe what's going on? I guess that's something standard perl, just that I'd like to be informed :) |
this seems fine, I've seen the same warnings but always been too lazy to change it all. Didn't test, I'm on vacation. :P
lib/fedoradistribution.pm | ||
---|---|---|
17–19 | It's maybe a bit too basic to comment, it'd be like commenting from foo import bar in Python...qw is quotewords, it creates an array (list) from some strings, you could just as well type ("send_key", "type_string", "wait_idle", "assert_screen") but perl folks seem to like qw although the convenience benefit seems marginal. In perl you almost always see it done this way (or as qw/foo bar moo/, because in perl there's always two ways to skin a cat...), not with the array syntax. |
lib/fedoradistribution.pm | ||
---|---|---|
17–19 | Still, I will document why is this needed (because importing whole testapi creates circural dependency for some reason). |
maybe a small comment to describe what's going on? I guess that's something standard perl, just that I'd like to be informed :)