Directory

⚓ T377666 Flaky ve-cite cypress tests
Page MenuHomePhabricator

Flaky ve-cite cypress tests
Closed, ResolvedPublic

Description

There are different flaky tests around ve-cite

https://integration.wikimedia.org/ci/job/wmf-quibble-selenium-php74/34859/console for ve-cite/wt2017Integration.cy.js

CypressError: Timed out retrying after 5050ms: cy.click() failed because this element:

https://integration.wikimedia.org/ci/job/quibble-vendor-mysql-php74-selenium/45579/console for ve-cite/veCiteIntegration.cy.js

AssertionError: Timed out retrying after 5000ms: Expected to find element: .oo-ui-tabSelectWidget .oo-ui-labelElement-label, but never found it

These tests failure seem related, e.g https://integration.wikimedia.org/ci/job/wmf-quibble-selenium-php74/34831/console

18:27:38   Running:  ve-cite/reuseRefs.cy.js                                                         (3 of 6)
18:27:41 
18:27:41 
18:27:41   Re-using refs in Visual Editor
18:27:45     1) "before all" hook for "should display re-used reference in article with correct footnote number and notification in context dialog"
18:27:45 
18:27:45 
18:27:45   0 passing (4s)
18:27:45   1 failing
18:27:45 
18:27:45   1) Re-using refs in Visual Editor
18:27:45        "before all" hook for "should display re-used reference in article with correct footnote number and notification in context dialog":
18:27:45      Error: http
18:27:45 
18:27:45 Because this error occurred during a `before all` hook we are skipping the remaining tests in the current suite: `Re-using refs in Visual Editor`
18:27:45 
18:27:45 Although you have test retries enabled, we do not retry tests when `before all` or `after all` hooks fail

Event Timeline

Jdlrobson subscribed.

This is blocking a change that is blocking the MW release.

Adding subscribers from T360026. This is still blocking merge

Change #1084227 had a related patch set uploaded (by WMDE-Fisch; author: WMDE-Fisch):

[mediawiki/extensions/Cite@master] [cypress] Focus the surface at least once before testing

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

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

[mediawiki/extensions/Cite@master] wt2017Integration.cy.js: Disable flaky test

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

kostajh triaged this task as High priority.Oct 30 2024, 7:06 PM

Change #1084867 merged by jenkins-bot:

[mediawiki/extensions/Cite@master] wt2017Integration.cy.js: Disable flaky test

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

Change #1084227 merged by jenkins-bot:

[mediawiki/extensions/Cite@master] [cypress] Focus the surface at least once before testing WT2017

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

Change #1085349 had a related patch set uploaded (by WMDE-Fisch; author: WMDE-Fisch):

[mediawiki/extensions/Cite@master] [cypress] Re-enable flaky test

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

Change #1085565 had a related patch set uploaded (by WMDE-Fisch; author: WMDE-Fisch):

[mediawiki/extensions/Cite@master] [cypress] Always wait for the Citoid inspector

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

Change #1085349 merged by jenkins-bot:

[mediawiki/extensions/Cite@master] [cypress] Re-enable flaky test

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

I saw this just now at https://integration.wikimedia.org/ci/job/wmf-quibble-selenium-php74/41077/console

  1) Re-using refs in Visual Editor
       "before all" hook for "should display re-used reference in article with correct footnote number and notification in context dialog":
     CypressError: `cy.then()` timed out after waiting `5000ms`.

Your callback function returned a promise that never resolved.