Page MenuHomePhabricator

Cleanup wdio-defaults.conf.js in core
Closed, ResolvedPublic1 Estimated Story Points

Description

We should aim to have our default configuration as clean as possible and move out some code. At a first look the process.on functions and the long Chrome-args list could be moved to another file to make the configuration easier to read. Maybe there are more things todo.

The goal here is to make the configuration as easy as possible to read.

Acceptance Criteria:

  • Move the uncaught handlers so that code is imported
  • Move the Chrome args list to another file

Related Objects

Event Timeline

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

[mediawiki/core@master] selenium: Move process handlers to new file

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

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

[mediawiki/core@master] selenium: Move Chrome setting/options to file

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

Change #1228484 merged by jenkins-bot:

[mediawiki/core@master] selenium: Move Chrome setting/options to file

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

Change #1228477 merged by jenkins-bot:

[mediawiki/core@master] selenium: Move process handlers to new file

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