Refactor selenium tests. These tests have numerous anti-patterns. Refactoring them will allow them to be more easily understood and able to run in parallel.
Description
Details
Related Objects
- Mentioned In
- T272142: Mentor coworkers, volunteers and interns
Event Timeline
Change 557278 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: refactor TemplateWizard.js test
Change 557634 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: refactor TemplateWizard.js test
Change 557634 abandoned by Zfilipin:
Selenium: refactor TemplateWizard.js test
Reason:
We've agreed in a meeting to rewrite this as a series of smaller patches.
Change 557278 abandoned by Zfilipin:
Selenium: refactor TemplateWizard.js test
Reason:
We've agreed in a meeting to rewrite this as a series of smaller patches.
Change 565046 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: fix test
Change 565053 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: break up test
Change 565054 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: remove extraneous code
Change 565068 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: add page object
Change 565046 merged by jenkins-bot:
[mediawiki/extensions/TemplateWizard@master] Selenium: fix test
Change 565068 abandoned by Ephemeralwaves:
Selenium: add page object
Reason:
Patch is too big. Will work on smaller patch.
Change 566864 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: add page object
Change 566865 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: add page object
Change 566864 abandoned by Ephemeralwaves:
Selenium: add page object
Reason:
Accidentally added this, fixed it here: https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/TemplateWizard/ /566865/
Change 566868 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: add page object
Change 565053 merged by jenkins-bot:
[mediawiki/extensions/TemplateWizard@master] Selenium: break up test
Change 565054 merged by jenkins-bot:
[mediawiki/extensions/TemplateWizard@master] Selenium: remove extraneous code
Change 574515 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: add page object
Change 574515 merged by jenkins-bot:
[mediawiki/extensions/TemplateWizard@master] Selenium: add page object
Change 592508 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: add page object
Change 596821 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/TemplateWizard into review/ephemeralwaves/T240766
Change 596821 abandoned by Ephemeralwaves:
Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/TemplateWizard into review/ephemeralwaves/T240766
Change 598220 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: move helper function
Change 592508 merged by jenkins-bot:
[mediawiki/extensions/TemplateWizard@master] Selenium: add page object
Change 598220 merged by jenkins-bot:
[mediawiki/extensions/TemplateWizard@master] Selenium: move helper function
Change 600478 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: add iterator
Change 600478 merged by jenkins-bot:
[mediawiki/extensions/TemplateWizard@master] Selenium: add iterator
Change 605238 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: split last test
Change 606802 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: split last test
Change 626458 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: Template insertion dialog error test
Change 606802 abandoned by Ephemeralwaves:
[mediawiki/extensions/TemplateWizard@master] Selenium: split last test
Reason:
Rewrote test https://gerrit.wikimedia.org/r/c/mediawiki/extensions/TemplateWizard/ /626458
Change 626479 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: Template insertion death date field test
Change 605238 abandoned by Ephemeralwaves:
[mediawiki/extensions/TemplateWizard@master] Selenium: death date in focus test
Reason:
Redid this patch
Change 626479 abandoned by Ephemeralwaves:
[mediawiki/extensions/TemplateWizard@master] Selenium: Template insertion death date field test
Reason:
fragile test
Change 626458 abandoned by Zfilipin:
[mediawiki/extensions/TemplateWizard@master] Selenium: Template insertion dialog error test
Reason:
Looks like the team in charge of the repository isn't interested in the refactoring.