Bug 33235 - Cypress tests are failing
Summary: Cypress tests are failing
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Test Suite (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal (vote)
Assignee: Pedro Amorim
QA Contact: Testopia
URL:
Keywords:
Depends on: 32495 32997
Blocks:
  Show dependency treegraph
 
Reported: 2023-03-15 15:21 UTC by Pedro Amorim
Modified: 2023-12-28 20:42 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.05.00,22.11.04


Attachments
Bug 33235: Fix cypress tests (4.09 KB, patch)
2023-03-15 15:24 UTC, Pedro Amorim
Details | Diff | Splinter Review
Bug 33235: Fix cypress tests (4.17 KB, patch)
2023-03-17 10:01 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 Pedro Amorim 2023-03-15 15:21:58 UTC

    
Comment 1 Pedro Amorim 2023-03-15 15:24:20 UTC
Created attachment 148219 [details] [review]
Bug 33235: Fix cypress tests

Some inconsistencies leftovers from 32495 and 32997 were causing cypress tests to fail, this fixes them.

* Apply patch
* Run cypress tests and verify all pass
Comment 2 Jonathan Druart 2023-03-17 10:01:02 UTC
It's fixing the problem, but I am getting some random failures.

    <testcase name="License CRUD operations Add license" time="0.0000" classname="Add license">     
      <failure message="Timed out retrying after 4000ms: Expected to find content: &apos;New license&apos; but never did." type="AssertionError"><![CDATA[AssertionError: Timed out retrying after 4000ms: Expected t
o find content: 'New license' but never did.   



    <testcase name="Title CRUD operations Add title" time="0.0000" classname="Add title">                 
      <failure message="Timed out retrying after 4000ms: Expected to find element: `main div[class=&apos;dialog alert&apos;]`, but never found it." type="AssertionError"><![CDATA[AssertionError: Timed out retrying
 after 4000ms: Expected to find element: `main div[class='dialog alert']`, but never found it.            
    at Context.eval (http://kohadev-intra.mydnsname.org:8081/__cypress/tests?p=t/cypress/integration/Titles_spec.ts:304:12)]]></failure>  


Not related to this patch, but to keep in mind!
Comment 3 Jonathan Druart 2023-03-17 10:01:21 UTC
Created attachment 148319 [details] [review]
Bug 33235: Fix cypress tests

Some inconsistencies leftovers from 32495 and 32997 were causing cypress tests to fail, this fixes them.

* Apply patch
* Run cypress tests and verify all pass

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 4 Jonathan Druart 2023-03-17 10:02:19 UTC
Trivial and fix failing tests. Passing QA.
Comment 5 Tomás Cohen Arazi 2023-03-17 13:00:17 UTC
Pushed to master for 23.05.

Nice work everyone, thanks!
Comment 6 Matt Blenkinsop 2023-03-17 17:21:10 UTC
Nice work everyone!

Pushed to stable for 22.11.x
Comment 7 Lucas Gass 2023-04-21 16:51:32 UTC
Not needed in 22.05.x, no backport