- Issue created by @acbramley
- Merge request !11826Issue #3519177: Add with-selenium-chrome to Repeat Class Test job → (Closed) created by acbramley
- 🇦🇺Australia mstrelan
Yes please. While we're at it, can we add a var for number of repeats? It's currently hardcoded at
--repeat "100". Would be good if we could set this var via the UI likeREPEAT_TEST_CLASS.We can do a separate issue for that, or we can re-title / re-purpose this issue to include this.
- 🇬🇧United Kingdom catch
Yes! Kept meaning to open an issue for this and hadn't done it yet, and agreed a variable for number of tests would be good - both for really long running tests that can time out with 100 runs, and also when you need 500, I think it's fine either here or split out.
- 🇦🇺Australia acbramley
Would be good if we could set this var via the UI
You can do it via the UI? 😲
Happy to add the variable here given this is such a small change.
- 🇬🇧United Kingdom catch
You can do it via the UI?
Once you have a pipeline running, you can manually run the job, and gitlab allows you to set environment variables there - this currently supports only the class name, but if we do the same for the number of runs, then it'll just be setting two variables - would be nice to keep it defaulting to 100 though I think so it's not required.
- 🇦🇺Australia mstrelan
For example, go to https://git.drupalcode.org/issue/drupal-3519177/-/pipelines/473447, click on Repeat Test Class then enter the variables.
- 🇦🇺Australia acbramley
Just found the help text in the job about how to set the variables in the UI. That's awesome.
Added REPEAT_COUNT
- 🇦🇺Australia acbramley
Tested with both variables here https://git.drupalcode.org/issue/drupal-3519177/-/jobs/4965595
- 🇬🇧United Kingdom catch
This is great, let's do it. Normally I'd leave an issue open a bit longer, but it's only changing a manually triggered test job, so quicker to commit than leave this then come back to it a day later.
