Page MenuHomePhabricator

selenium: Default to larger window size
Open, Needs TriagePublic

Description

Some Selenium tests can fail when the viewport is smaller than anticipated (e.g. T314114: Selenium: Close help panel before editing), because an element may be obscured by another one and so is no longer clickable or visible to Selenium. Let's start with a larger window size as the default; individual tests can override this with browser.setWindowSize().

Event Timeline

Change 818403 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):

[mediawiki/core@master] selenium: Default to larger window size

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

Change 818448 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):

[mediawiki/core@master] wdio-mediawiki: Release 2.2.0

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

zeljkofilipin changed the task status from Open to In Progress.Aug 25 2022, 10:55 AM
zeljkofilipin claimed this task.
zeljkofilipin moved this task from Watching 📺 to Deep work 🌊 on the User-zeljkofilipin board.

Change 818403 merged by jenkins-bot:

[mediawiki/core@master] selenium: Default to larger window size

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

Change 818448 merged by jenkins-bot:

[mediawiki/core@master] wdio-mediawiki: Release 2.2.0

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

Change 849073 had a related patch set uploaded (by Zfilipin; author: Zfilipin):

[mediawiki/core@master] Revert "selenium: Default to larger window size"

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

818403 broke video recording in CI. I've reverted it with 849073. See T317879: ffmpeg stderr Capture area 1920x1080 at position 0.0 outside the screen size 1280x1024. I'm not opposed to changing the screen size, but looks like CI has to be updated to support larger screen size too.

Change 849073 merged by jenkins-bot:

[mediawiki/core@master] Revert "selenium: Default to larger window size"

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

818403 broke video recording in CI. I've reverted it with 849073. See T317879: ffmpeg stderr Capture area 1920x1080 at position 0.0 outside the screen size 1280x1024. I'm not opposed to changing the screen size, but looks like CI has to be updated to support larger screen size too.

@zeljkofilipin is this something that your team could work on? I don't think the Growth team can pursue it, as we already have a workaround in T314114: Selenium: Close help panel before editing.

@kostajh sure, but we're currently busy with T324685: Upgrade WebdriverIO to v8 in all repositories. Is this urgent?

No, it is not urgent.

@kostajh feel free to unassign the task from yourself, I can work on it, but I'm not sure when I'll have the time.

The latest version of wdio-mediawiki is 2.2.0, but as noted above, it breaks video recording in CI. I think it's fine if this task is not fixed immediately, but shouldn't we publish a new version including the revert, so that people don't try to upgrade just to later realize that the new shiny version doesn't work? (This is definitely not what happened to yours truly)

@Daimona I agree, we should release new version of wdio-mediawiki. It's on my todo list, but I'm not sure when I'll get to it.