Nothing Special   »   [go: up one dir, main page]

Page MenuHomePhabricator

Fix flaky Selenium test: Rollback without confirmation should perform rollback via GET request without asking the user to confirm
Closed, InvalidPublic

Description

See https://integration.wikimedia.org/ci/job/mediawiki-quibble-vendor-postgres-php72-docker/953/consoleFull for patch https://gerrit.wikimedia.org/r/c/mediawiki/core/+/517701#message-02cba8cf6012362e4ded279c79da2a4c98744f94

17:19:31 1) Rollback without confirmation should perform rollback via GET request without asking the user to confirm:
17:19:31 Expected rollback page to appear.
17:19:31 running chrome
17:19:31 Error: Expected rollback page to appear.
17:19:31     at elementIdText("0.9789767140534527-1") - getText.js:35:50

Screenshot from the failure:

image.png (1×1 px, 497 KB)

Video:

Event Timeline

Change 517958 had a related patch set uploaded (by Kosta Harlan; owner: Kosta Harlan):
[mediawiki/core@master] Disable flaky Selenium test

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

Change 517961 had a related patch set uploaded (by Krinkle; owner: Kosta Harlan):
[mediawiki/core@wmf/1.34.0-wmf.10] Disable flaky Selenium test

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

Change 517958 merged by jenkins-bot:
[mediawiki/core@master] Disable flaky Selenium test

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

Change 517961 merged by jenkins-bot:
[mediawiki/core@wmf/1.34.0-wmf.10] Disable flaky Selenium test

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

The POST equivalent seems flaky as well:
https://integration.wikimedia.org/ci/job/mediawiki-quibble-vendor-postgres-php72-docker/1011/console

21:10:35 1) Rollback without confirmation should perform rollback via POST request without asking the user to confirm:
21:10:35 Expected rollback page to appear.
21:10:35 running chrome
21:10:35 Error: Expected rollback page to appear.
21:10:35     at elementIdText("0.1626589557410456-1") - getText.js:35:50

(patch is a no-op)

Change 541618 had a related patch set uploaded (by Reedy; owner: Reedy):
[mediawiki/core@master] Disable flaky Selenium test

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

Change 541633 had a related patch set uploaded (by Reedy; owner: Reedy):
[mediawiki/core@REL1_34] Disable flaky Selenium test

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

Change 541618 merged by jenkins-bot:
[mediawiki/core@master] Disable flaky Selenium test

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

Change 541633 merged by jenkins-bot:
[mediawiki/core@REL1_34] Disable flaky Selenium test

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

WMDE-Fisch closed this task as Invalid.EditedJan 26 2022, 2:08 PM
WMDE-Fisch subscribed.

Currently browser tests for rollback confirmation got disabled and removed due to flackyness. It might make sense to re-add at least one browser tests for that functionality in the future. This currently de-prioritized by WMDE.

See T300141: Add basic browser test for rollback confirmation