Page MenuHomePhabricator

Tidy up browser tests
Closed, ResolvedPublic

Description

Our browser test suite is looking rather messy. Cleaning it up will make it easier to maintain and to add to.

We should

  • remove unnecessary duplication in TermboxPage of selectors, elements, and helpers methods (e.g. TermboxPage.clickMyFavoriteButton() is not better than TermboxPage.myFavoriteButton.click())
  • split specs into multiple files where it makes sense
  • create new test items to operate on where it makes sense
  • figure out what (if anything) is wrong with login/logout logic, try removing the handcrafted logic and use standard login/logout functionality
  • split the item creation logic out of TermboxPage

Event Timeline

Change 528501 had a related patch set uploaded (by Jakob; owner: Jakob):
[wikibase/termbox@master] browser tests: tidy up Termbox.page.js

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

Change 528728 had a related patch set uploaded (by Jakob; owner: Jakob):
[wikibase/termbox@master] browser tests: use existing LoginPage login mechanisms

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

Change 528792 had a related patch set uploaded (by Jakob; owner: Jakob):
[wikibase/termbox@master] browser tests: split item creation out of TermboxPage

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

Change 528879 had a related patch set uploaded (by Jakob; owner: Jakob):
[wikibase/termbox@master] browser tests: simplify term value tests

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

Change 528501 merged by jenkins-bot:
[wikibase/termbox@master] browser tests: tidy up Termbox.page.js

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

Change 528728 merged by jenkins-bot:
[wikibase/termbox@master] browser tests: use existing LoginPage login mechanisms

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

Change 529110 had a related patch set uploaded (by Jakob; owner: Jakob):
[wikibase/termbox@master] browser tests: split Termbox.spec.js

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

Change 528792 merged by jenkins-bot:
[wikibase/termbox@master] browser tests: split item creation out of TermboxPage

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

Change 528879 merged by jenkins-bot:
[wikibase/termbox@master] browser tests: simplify term value tests

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

Change 529110 merged by jenkins-bot:
[wikibase/termbox@master] browser tests: split Termbox.spec.js

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

Change 529932 had a related patch set uploaded (by Jakob; owner: Jakob):
[mediawiki/extensions/Wikibase@master] Update termbox

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

Change 529932 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Update termbox

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