With the release of 6.3.0 we forced headless by default in CI and broke the old behavior of using DISPLAY locally to enable/disable headless.
Let's add back that functionality. Make sure it also works in fresh-node.
Description
Description
Details
Details
Related Changes in Gerrit:
| Subject | Author | Repo | Branch | Lines +/- | |
|---|---|---|---|---|---|
| selenium: Bring back DISPLAY to enable/disable headless locally | Phedenskog | mediawiki/core | master | +7 -3 |
Event Timeline
Comment Actions
Change #1247533 had a related patch set uploaded (by Phedenskog; author: Phedenskog):
[mediawiki/core@master] selenium: Bring back DISPLAY to enable/disable headless locally
Comment Actions
@Peter I made just one suggested edit on the patch and maybe we should change the task description as well. Instead of running DISPLAY='' npm run selenium-test to get it to run without opening a browser window on local, I think only running DISPLAY=1 npm run selenium-test should make the browser windows open. Thank you for this!
Comment Actions
Change #1247533 merged by jenkins-bot:
[mediawiki/core@master] selenium: Bring back DISPLAY to enable/disable headless locally