Page MenuHomePhabricator

Refactor TemplateWizard Selenium tests
Closed, ResolvedPublic

Description

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.

Event Timeline

Restricted Application added a subscriber: Aklapper. ยท View Herald Transcript

Change 557278 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: refactor TemplateWizard.js test

https://gerrit.wikimedia.org/r/557278

Change 557634 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: refactor TemplateWizard.js test

https://gerrit.wikimedia.org/r/557634

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.

https://gerrit.wikimedia.org/r/557634

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.

https://gerrit.wikimedia.org/r/557278

Change 565046 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: fix test

https://gerrit.wikimedia.org/r/565046

Change 565053 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: break up test

https://gerrit.wikimedia.org/r/565053

Change 565054 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: remove extraneous code

https://gerrit.wikimedia.org/r/565054

Change 565068 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: add page object

https://gerrit.wikimedia.org/r/565068

Change 565046 merged by jenkins-bot:
[mediawiki/extensions/TemplateWizard@master] Selenium: fix test

https://gerrit.wikimedia.org/r/565046

Change 565068 abandoned by Ephemeralwaves:
Selenium: add page object

Reason:
Patch is too big. Will work on smaller patch.

https://gerrit.wikimedia.org/r/565068

Change 566864 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: add page object

https://gerrit.wikimedia.org/r/566864

Change 566865 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: add page object

https://gerrit.wikimedia.org/r/566865

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/

https://gerrit.wikimedia.org/r/566864

Change 566865 abandoned by Ephemeralwaves:
Selenium: add page object

https://gerrit.wikimedia.org/r/566865

Change 566868 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: add page object

https://gerrit.wikimedia.org/r/566868

Change 565053 merged by jenkins-bot:
[mediawiki/extensions/TemplateWizard@master] Selenium: break up test

https://gerrit.wikimedia.org/r/565053

Change 565054 merged by jenkins-bot:
[mediawiki/extensions/TemplateWizard@master] Selenium: remove extraneous code

https://gerrit.wikimedia.org/r/565054

Change 566868 abandoned by Ephemeralwaves:
Selenium: add page object

https://gerrit.wikimedia.org/r/566868

Change 574515 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: add page object

https://gerrit.wikimedia.org/r/574515

Change 574515 merged by jenkins-bot:
[mediawiki/extensions/TemplateWizard@master] Selenium: add page object

https://gerrit.wikimedia.org/r/574515

Change 592508 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: add page object

https://gerrit.wikimedia.org/r/592508

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

https://gerrit.wikimedia.org/r/596821

Change 596821 abandoned by Ephemeralwaves:
Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/TemplateWizard into review/ephemeralwaves/T240766

https://gerrit.wikimedia.org/r/596821

Change 598220 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: move helper function

https://gerrit.wikimedia.org/r/598220

Change 592508 merged by jenkins-bot:
[mediawiki/extensions/TemplateWizard@master] Selenium: add page object

https://gerrit.wikimedia.org/r/592508

Change 598220 merged by jenkins-bot:
[mediawiki/extensions/TemplateWizard@master] Selenium: move helper function

https://gerrit.wikimedia.org/r/598220

Change 600478 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: add iterator

https://gerrit.wikimedia.org/r/600478

Change 600478 merged by jenkins-bot:
[mediawiki/extensions/TemplateWizard@master] Selenium: add iterator

https://gerrit.wikimedia.org/r/600478

Change 605238 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: split last test

https://gerrit.wikimedia.org/r/605238

Change 606802 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: split last test

https://gerrit.wikimedia.org/r/606802

Change 626458 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: Template insertion dialog error test

https://gerrit.wikimedia.org/r/626458

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

https://gerrit.wikimedia.org/r/606802

Change 626479 had a related patch set uploaded (by Ephemeralwaves; owner: Ephemeralwaves):
[mediawiki/extensions/TemplateWizard@master] Selenium: Template insertion death date field test

https://gerrit.wikimedia.org/r/626479

Change 605238 abandoned by Ephemeralwaves:
[mediawiki/extensions/TemplateWizard@master] Selenium: death date in focus test

Reason:
Redid this patch

https://gerrit.wikimedia.org/r/605238

Change 626479 abandoned by Ephemeralwaves:
[mediawiki/extensions/TemplateWizard@master] Selenium: Template insertion death date field test

Reason:
fragile test

https://gerrit.wikimedia.org/r/626479

Reedy renamed this task from Refactor TemplateWizard Selenium tests to Refactor TemplateWizard Selenium tests.Jun 28 2021, 8:44 PM

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.

https://gerrit.wikimedia.org/r/626458