action #44858
closed[functional][u] loadtests method should check that the filename is not ending with a .pm
0%
Description
To avoid occurences of https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/6365 it would be nice to have a check (regexp?) in travis perhaps?
Updated by okurz over 5 years ago
- Subject changed from [functional][u][y] loadtests method should check that the filename is not ending with a .pm to [functional][u] loadtests method should check that the filename is not ending with a .pm
- Category set to Enhancement to existing tests
- Target version set to Milestone 23
I guess one team can handle it, let's pick … [u] :) Yes, I guess a simple static check could help. We already check for "unused_modules" with a regex covering the loadtest calls, something similar could be done here.
Updated by okurz about 5 years ago
- Target version changed from Milestone 23 to Milestone 25
Updated by okurz about 5 years ago
- Priority changed from Normal to Low
easy but "low" prio. Still keeping in "Workable" for the time being though to have at least also one "easy" "low-prio" task in there :)
Updated by zluo almost 5 years ago
- Status changed from Workable to In Progress
- Assignee set to zluo
take over
Updated by zluo almost 5 years ago
- Status changed from In Progress to Feedback
Updated by zluo almost 5 years ago
- Status changed from Feedback to Resolved
verification test on osd with my test pm:
openqa-clone-custom-git-refspec https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/7459 https://openqa.suse.de/tests/2874413 TEST=poo#7459 _GROUP_ID="0"
https://openqa.suse.de/tests/2889183/file/autoinst-log.txt
verified now.
Updated by SLindoMansilla almost 5 years ago
- Related to action #51473: [functional][u] Set up a travis check to avoid merging PR when loadtest *.pm is found added