Bug 36535

Summary: 33568 introduced too many changes in modules without tests
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Test SuiteAssignee: Jonathan Druart <jonathan.druart>
Status: Pushed to main --- QA Contact: Testopia <testopia>
Severity: major    
Priority: P5 - low CC: david, martin.renvoize
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
24.05.00
Bug Depends on: 33568    
Bug Blocks:    
Attachments: Bug 36535: (bug 33568 follow-up) Add missing tests
Bug 36535: (bug 33568 follow-up) Add missing tests
Bug 36535: (bug 33568 follow-up) Add missing tests
Bug 36535: (QA follow-up) Use dt_from_string

Description Jonathan Druart 2024-04-05 13:07:08 UTC
Bug 33568 introduced a lot of new simple accessors without providing unit tests for them.
Comment 1 Jonathan Druart 2024-04-18 11:49:05 UTC
Created attachment 165080 [details] [review]
Bug 36535: (bug 33568 follow-up) Add missing tests

Bug 33568 introduced a lot of new simple accessors without providing unit tests for them.

Test plan:
All tests modified by this patch should pass

Note that one bug was found! Logic issue in lost/not lost
Comment 2 David Nind 2024-04-20 13:01:10 UTC
Created attachment 165235 [details] [review]
Bug 36535: (bug 33568 follow-up) Add missing tests

Bug 33568 introduced a lot of new simple accessors without providing unit tests for them.

Test plan:
All tests modified by this patch should pass

Note that one bug was found! Logic issue in lost/not lost

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 David Nind 2024-04-20 13:02:13 UTC
Testing notes (using KTD):

1. Tests pass both before and after:

prove t/db_dependent/Items.t
prove t/db_dependent/Koha/Biblio/ItemGroups.t
prove t/db_dependent/Koha/Courses.t (new test)
prove t/db_dependent/Koha/Item.t
prove t/db_dependent/Koha/Items.t
prove t/db_dependent/api/v1/biblios.t
Comment 4 Martin Renvoize 2024-04-23 09:56:49 UTC
Created attachment 165363 [details] [review]
Bug 36535: (bug 33568 follow-up) Add missing tests

Bug 33568 introduced a lot of new simple accessors without providing unit tests for them.

Test plan:
All tests modified by this patch should pass

Note that one bug was found! Logic issue in lost/not lost

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 5 Martin Renvoize 2024-04-23 09:56:51 UTC
Created attachment 165364 [details] [review]
Bug 36535: (QA follow-up) Use dt_from_string

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 6 Martin Renvoize 2024-04-23 09:57:38 UTC
Great to see these tests coming through, thanks for the follow-up Jonathan!

Passing QA
Comment 7 Katrin Fischer 2024-04-26 12:17:07 UTC
Pushed for 24.05!

Well done everyone, thank you!