Bug 27062 - pickup_location tests don't deal correctly with existing libraries
Summary: pickup_location tests don't deal correctly with existing libraries
Status: Pushed to master
Alias: None
Product: Koha
Classification: Unclassified
Component: Test Suite (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low normal (vote)
Assignee: Jonathan Druart
QA Contact: Testopia
URL:
Keywords:
Depends on: 26984 27002 26963
Blocks:
  Show dependency treegraph
 
Reported: 2020-11-20 09:31 UTC by Jonathan Druart
Modified: 2020-11-20 09:37 UTC (History)
0 users

See Also:
Change sponsored?: ---
Patch complexity: ---
Who signed the patch off:
Text to go in the release notes:
Version(s) released in:
20.11.00


Attachments
Bug 27062: Fix pickup_location tests when existing libraries exist (9.80 KB, patch)
2020-11-20 09:33 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 2020-11-20 09:31:25 UTC
If there are existing libraries in the DB, the tests will fail.

There were several changes related to those tests recently, this bug report is a follow-up for them.
Comment 1 Jonathan Druart 2020-11-20 09:33:36 UTC
Was failing a lot today
https://jenkins.koha-community.org/job/Koha_Master_U18/954/consoleFull
https://jenkins.koha-community.org/job/Koha_Master_U16/134/consoleFull
https://jenkins.koha-community.org/job/Koha_Master_D9_MDB_Latest/449/consoleFull


08:38:45 koha_1       |     #   Failed test 'ReservesControlBranch: ItemHomeLibrary, biblio1, patron1 should return 27 but returns 18'
08:38:45 koha_1       |     #   at t/db_dependent/Koha/Biblio.t line 405.
08:38:45 koha_1       | 
08:38:45 koha_1       |     #   Failed test 'ReservesControlBranch: PatronLibrary, biblio1, patron1 should return 27 but returns 18'
08:38:45 koha_1       |     #   at t/db_dependent/Koha/Biblio.t line 405.
08:38:45 koha_1       |     # Looks like you failed 2 tests of 8.
08:38:45 koha_1       | 
08:38:45 koha_1       | #   Failed test 'pickup_locations'
08:38:45 koha_1       | #   at t/db_dependent/Koha/Biblio.t line 422.
Comment 2 Jonathan Druart 2020-11-20 09:33:59 UTC
Created attachment 113868 [details] [review]
Bug 27062: Fix pickup_location tests when existing libraries exist

If there are existing libraries in the DB, the tests will fail.

There were several changes related to those tests recently, this bug report is a follow-up for them.

bug 26984, bug 27002 and bug 26963.
Comment 3 Jonathan Druart 2020-11-20 09:34:38 UTC
This is not perfect, but I need the jenkins happy. Feel free to follow-up if you have a better fix.
Comment 4 Jonathan Druart 2020-11-20 09:37:46 UTC
Pushed to master for 20.11, thanks to everybody involved!