Bug 31140

Summary: TestBuilder.t is failing on item groups modules
Product: Koha Reporter: Kyle M Hall (khall) <kyle>
Component: Architecture, internals, and plumbingAssignee: Kyle M Hall (khall) <kyle>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: major    
Priority: P5 - low CC: lucas, tomascohen
Version: unspecified   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: Trivial patch Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
22.11.00
Circulation function:
Bug Depends on: 24857    
Bug Blocks:    
Attachments: Bug 31140: TestBuilder.t is failing on item groups modules
Bug 31140: TestBuilder.t is failing on item groups modules
Bug 31140: TestBuilder.t is failing on item groups modules
Bug 31140: (QA follow-up) Remove leftover POD

Description Kyle M Hall (khall) 2022-07-12 10:11:57 UTC

    
Comment 1 Kyle M Hall (khall) 2022-07-12 10:15:02 UTC
Created attachment 137616 [details] [review]
Bug 31140: TestBuilder.t is failing on item groups modules

Test Plan:
1) prove t/db_dependent/TestBuilder.t
2) Note failures
3) Apply this patch
4) prove t/db_dependent/TestBuilder.t
5) Note all tests pass!
Comment 2 David Nind 2022-07-12 11:36:11 UTC
Created attachment 137620 [details] [review]
Bug 31140: TestBuilder.t is failing on item groups modules

Test Plan:
1) prove t/db_dependent/TestBuilder.t
2) Note failures
3) Apply this patch
4) prove t/db_dependent/TestBuilder.t
5) Note all tests pass!

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 Tomás Cohen Arazi (tcohen) 2022-07-12 13:32:15 UTC
Created attachment 137627 [details] [review]
Bug 31140: TestBuilder.t is failing on item groups modules

Test Plan:
1) prove t/db_dependent/TestBuilder.t
2) Note failures
3) Apply this patch
4) prove t/db_dependent/TestBuilder.t
5) Note all tests pass!

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 4 Tomás Cohen Arazi (tcohen) 2022-07-12 13:34:07 UTC
Pushed to master for 22.11.

Nice work everyone, thanks!
Comment 5 Tomás Cohen Arazi (tcohen) 2022-07-12 13:45:22 UTC
Created attachment 137629 [details] [review]
Bug 31140: (QA follow-up) Remove leftover POD

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 6 Lucas Gass (lukeg) 2022-08-23 15:54:22 UTC
Missing dependencies for 22.05.x, no backport