Bug 40051 - cy.wait(delay) should not be used in Cypress tests
Summary: cy.wait(delay) should not be used in Cypress tests
Status: BLOCKED
Alias: None
Product: Koha
Classification: Unclassified
Component: Test Suite (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low normal
Assignee: Jonathan Druart
QA Contact: Testopia
URL:
Keywords:
Depends on: 40046
Blocks:
  Show dependency treegraph
 
Reported: 2025-06-02 11:49 UTC by Jonathan Druart
Modified: 2025-06-03 12:09 UTC (History)
1 user (show)

See Also:
GIT URL:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
Bug 40051: Remove cy.wait(delay) from cypress tests (23.37 KB, patch)
2025-06-02 11:54 UTC, Jonathan Druart
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2025-06-02 11:49:22 UTC
It can cause random failures. We should rely on the existence of something, Cypress will retry if the test fails.
Comment 1 Jonathan Druart 2025-06-02 11:54:02 UTC
Created attachment 182905 [details] [review]
Bug 40051: Remove cy.wait(delay) from cypress tests

We should not use cy.wait(delay), it's a poor pattern and can cause
random failures.

Test plan:
All cypress tests should still pass after this patchset.

Note that t/cypress/integration/ERM/Titles_spec.ts is currently failing.
Comment 2 Jonathan Druart 2025-06-02 11:55:18 UTC
I've asked Pedro to have a look at Titles_spec.ts.