Page MenuHomePhabricator

Speed up PageTriage webdriver.io tests
Closed, ResolvedPublic

Description

I've been going through the jobs that runs in the gated extensions job for core to check if there's any small performance wins that can be done. In PageTriage I think there's two things:

  • We use browser.reloadSession but I don't think we actually needed it. It's recommended by the wdio to only use it if it's really needed since it slow.
  • We use the RunJobs but I'm not sure we actually need it. Lets create a patch, see if it works without it and ask the team if they see if it needs to be used

Acceptance Criteria:

  • Remove browser.reloadSession and run jobs from PageTriage and see if its speed up the jobs (and still work)

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change #1248813 had a related patch set uploaded (by Phedenskog; author: Phedenskog):

[mediawiki/extensions/PageTriage@master] selenium: Remove RunJobs.run()

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

Change #1248811 had a related patch set uploaded (by Phedenskog; author: Phedenskog):

[mediawiki/extensions/PageTriage@master] selenium: Remove reloadSession to speed up the tests

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

Change #1248813 merged by jenkins-bot:

[mediawiki/extensions/PageTriage@master] selenium: Remove RunJobs.run() to speed up the tests

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

Change #1248811 merged by jenkins-bot:

[mediawiki/extensions/PageTriage@master] selenium: Remove reloadSession to speed up the tests

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