Bug 40301

Summary: Missing Cypress tests for 'Type' column visibility
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Test SuiteAssignee: Jonathan Druart <jonathan.druart>
Status: Needs documenting --- QA Contact: Martin Renvoize (ashimema) <martin.renvoize>
Severity: enhancement    
Priority: P5 - low CC: fridolin.somers, lucas, martin.renvoize
Version: unspecified   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: Small patch Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
25.11.00,25.05.03
Circulation function:
Bug Depends on: 40181, 40281    
Bug Blocks: 40345    
Attachments: Bug 40301: Add insertSampleCheckout
Bug 40301: (bug 40299 follow-up) Add Cypress tests
Bug 40301: Add insertSampleCheckout
Bug 40301: (bug 40299 follow-up) Add Cypress tests
Bug 40301: Add insertSampleCheckout
Bug 40301: (bug 40299 follow-up) Add Cypress tests
Bug 40301: Add insertSampleCheckout
Bug 40301: (bug 40299 follow-up) Add Cypress tests

Description Jonathan Druart 2025-07-03 11:59:06 UTC

    
Comment 1 Jonathan Druart 2025-07-03 12:13:13 UTC Comment hidden (obsolete)
Comment 2 Jonathan Druart 2025-07-03 12:13:15 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2025-07-03 12:14:09 UTC
Test plan:
The new Cypress tests should pass.

yarn cypress run --spec t/cypress/integration/KohaTable/CirculationHistory_spec.ts

If you revert bug 40299 it should fail.
Comment 4 Jonathan Druart 2025-07-10 12:47:41 UTC
Created attachment 183938 [details] [review]
Bug 40301: Add insertSampleCheckout
Comment 5 Jonathan Druart 2025-07-10 12:47:43 UTC
Created attachment 183939 [details] [review]
Bug 40301: (bug 40299 follow-up) Add Cypress tests
Comment 6 Owen Leonard 2025-07-17 15:48:41 UTC
Created attachment 184214 [details] [review]
Bug 40301: Add insertSampleCheckout

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 7 Owen Leonard 2025-07-17 15:48:44 UTC
Created attachment 184215 [details] [review]
Bug 40301: (bug 40299 follow-up) Add Cypress tests

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 8 Martin Renvoize (ashimema) 2025-07-18 11:25:24 UTC
Created attachment 184338 [details] [review]
Bug 40301: Add insertSampleCheckout

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Comment 9 Martin Renvoize (ashimema) 2025-07-18 11:25:26 UTC
Created attachment 184339 [details] [review]
Bug 40301: (bug 40299 follow-up) Add Cypress tests

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Comment 10 Lucas Gass (lukeg) 2025-07-18 16:40:00 UTC
  Running:  CirculationHistory_spec.ts                                                      (1 of 1)


  1) "before all" hook for "'Type' column should be hidden"

  0 passing (361ms)
  1 failing

  1) members/readingrec
       "before all" hook for "'Type' column should be hidden":
     Error: Cypress tests tests require 'RESTBasicAuth'. Skipping suite.

Because this error occurred during a `before all` hook we are skipping all of the remaining tests.
      at Context.eval (webpack://koha/./t/cypress/support/e2e.js:1944:18)
Comment 11 Jonathan Druart 2025-07-21 07:45:11 UTC
(In reply to Lucas Gass (lukeg) from comment #10)
>   Running:  CirculationHistory_spec.ts                                      
> (1 of 1)
> 
> 
>   1) "before all" hook for "'Type' column should be hidden"
> 
>   0 passing (361ms)
>   1 failing
> 
>   1) members/readingrec
>        "before all" hook for "'Type' column should be hidden":
>      Error: Cypress tests tests require 'RESTBasicAuth'. Skipping suite.
> 
> Because this error occurred during a `before all` hook we are skipping all
> of the remaining tests.
>       at Context.eval (webpack://koha/./t/cypress/support/e2e.js:1944:18)

See "Bug 40174: Require RESTBasicAuth turned ON" and https://gitlab.com/koha-community/koha-misc4dev/-/issues/91
Comment 12 Lucas Gass (lukeg) 2025-07-21 20:03:31 UTC
Nice work everyone!

Pushed to main for 25.11
Comment 13 Paul Derscheid 2025-08-22 08:29:16 UTC
Nice work everyone!

Pushed to 25.05.x
Comment 14 Fridolin Somers 2025-09-17 13:37:56 UTC
Depends on Bug 40181 not in 24.11.x