July 2025 - Koha-bugs - lists.koha-community.org

[Bug 39367] New: Add POD to all subroutines/methods
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39367 Bug ID: 39367 Summary: Add POD to all subroutines/methods Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Test Suite Assignee: jonathan.druart(a)gmail.com Reporter: jonathan.druart(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org Depends on: 39319 Blocks: 39324 In order to have a POD coverage test we need a starting point where all subroutines and methods will have a POD entry, even if empty. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39319 [Bug 39319] xt/author/podcorrectness.t only test POD for perl files within C4 and Koha https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39324 [Bug 39324] [OMNIBUS] All QA tests should pass -- You are receiving this mail because: You are watching all bug changes.

1 43

[Bug 39700] New: Fix test case t/db_dependent/Authority/Merge.t broken in 34739
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39700 Bug ID: 39700 Summary: Fix test case t/db_dependent/Authority/Merge.t broken in 34739 Change sponsored?: --- Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Test Suite Assignee: chris(a)bigballofwax.co.nz Reporter: domm(a)plix.at QA Contact: testopia(a)bugs.koha-community.org Bug 34739 adds some new tests to t/db_dependent/Authority/Merge.t, which work when run alone. But when running the whole test suite, some weird fails show up. This patch fixes that bug in the test suite The new test case calls AddBiblio. And AddBiblio returns ($biblionumber, $biblioitemnumber). But the test calls it like this: my $biblionumber = AddBiblio; AddBiblio returns a list, but the calling code only stores a single value, so it get's the last value, i.e. the biblioitemnumber. Now, when running the test alone, the DB sequences for biblio and biblioitem are in sync and both return the same value so the test works (by accident) But some tests add biblios with multiple items (or biblios without items), thus the sequences are no longer in sync and when the test tries to load a biblio via a biblioitemid, it finds nothing, which causes the fail bad: my $biblionumber = AddBiblio( $biblio_record, '', { skip_record_index => 1 } ); ok-ish: my ($biblionumber) = AddBiblio( $biblio_record, '', { skip_record_index => 1 } ); -- You are receiving this mail because: You are watching all bug changes.

1 11

[Bug 39874] New: Template error prevents authority merging
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39874 Bug ID: 39874 Summary: Template error prevents authority merging Change sponsored?: --- Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: major Priority: P5 - low Component: Authentication Assignee: oleonard(a)myacpl.org Reporter: oleonard(a)myacpl.org QA Contact: testopia(a)bugs.koha-community.org CC: dpavlin(a)rot13.org Depends on: 39325 Bug 39325 renamed the variable used inside of the FOREACH, but one of the instances was unchanged: [% FOREACH line IN result %] <tr data-authid="[% result.authid | html %]"> Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39325 [Bug 39325] Run codespell successfully on the whole codebase -- You are receiving this mail because: You are watching all bug changes.

1 16

[Bug 39319] New: xt/author/podcorrectness.t only test POD for perl files within C4 and Koha
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39319 Bug ID: 39319 Summary: xt/author/podcorrectness.t only test POD for perl files within C4 and Koha Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Test Suite Assignee: jonathan.druart(a)gmail.com Reporter: jonathan.druart(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org It should also test controllers and tests. -- You are receiving this mail because: You are watching all bug changes.

1 26

[Bug 38461] New: Table features needs to be covered by e2e tests - Cypress
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38461 Bug ID: 38461 Summary: Table features needs to be covered by e2e tests - Cypress Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Test Suite Assignee: jonathan.druart(a)gmail.com Reporter: jonathan.druart(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org Depends on: 36640 We need UI tests for the table features. We have different kinds of table and a lot of features that must be covered by tests. 3 types of table: * Tables using the REST API (kohaTable) * Tables not using the REST API (KohaTable) * Tables using dataTables constructor directly (should be removed by bug 38255) Features: * Save state * Copy shareable link * Save configuration state on page change * Save search state on page change: * Export * Search (and the 'Clear filters' button) * Columns filters * External filters * etc. Specific tables: * Patrons search * Items list (catalogue/detail.pl) * Items search * Pending requests (should not be different but it is) * Libraries ("classic" using kohaTable) Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36640 [Bug 36640] Upgrade DataTables from 1.13.6 to 2.x -- You are receiving this mail because: You are watching all bug changes.

1 88

[Bug 38503] New: Add a Cypress task to generate objects based on its swagger def spec
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38503 Bug ID: 38503 Summary: Add a Cypress task to generate objects based on its swagger def spec Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Test Suite Assignee: jonathan.druart(a)gmail.com Reporter: jonathan.druart(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org We have TestBuilder for the Perl tests. However for the Cypress tests we have to: 1. either mock all the routes (which is painful and time-consuming), 2. or rely on the existing data in DB (which will produce failures if data has been inserted, and tests will have to cleanup before/after, which is painful, time-consuming and not robust) We should have a way to generate objects easily, in order to mock the response of a GET route easily. We can actually rely on the swagger spec files (the definition file) to generate random data and mock the response of routes. The first intended goal is to provide tests for the tables (bug 38461) and allow future enhancements in this area (see dependent bugs of bug 36640). -- You are receiving this mail because: You are watching all bug changes.

1 72

[Bug 40379] New: t/db_dependent/www tests generate warnings
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40379 Bug ID: 40379 Summary: t/db_dependent/www tests generate warnings Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Test Suite Assignee: chris(a)bigballofwax.co.nz Reporter: jonathan.druart(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org Blocks: 25515 03:17:21 koha_1 | Warning: <retrieval> element has non-unique value in 'name' key attribute: F at t/db_dependent/www/auth_values_input_www.t line 36. 03:17:21 koha_1 | Warning: <retrieval> element has non-unique value in 'name' key attribute: B at t/db_dependent/www/auth_values_input_www.t line 36. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25515 [Bug 25515] [OMNIBUS] Remove warnings and errors from tests output -- You are receiving this mail because: You are watching all bug changes.

1 21

[Bug 37302] New: xt/api.t should fail if swagger-cli is missing
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37302 Bug ID: 37302 Summary: xt/api.t should fail if swagger-cli is missing Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Test Suite Assignee: chris(a)bigballofwax.co.nz Reporter: martin.renvoize(a)ptfs-europe.com QA Contact: testopia(a)bugs.koha-community.org Currently, we skip if the swagger-cli utility is missing. This was hiding genuine issues in Jenkins, we should make it as a failure with a suitable message to the end user. -- You are receiving this mail because: You are watching all bug changes.

1 39

[Bug 35904] New: C4::Auth::checkauth cannot be tested easily
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35904 Bug ID: 35904 Summary: C4::Auth::checkauth cannot be tested easily Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: ASSIGNED Severity: normal Priority: P5 - low Component: Test Suite Assignee: jonathan.druart+koha(a)gmail.com Reporter: jonathan.druart+koha(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org We are printing headers and the tests are hard to write. -- You are receiving this mail because: You are watching all bug changes.

1 40

[Bug 38474] New: t/db_dependent/Letters.t can fail randomly
by bugzilla-daemon＠bugs.koha-community.org 29 Aug '25

29 Aug '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38474 Bug ID: 38474 Summary: t/db_dependent/Letters.t can fail randomly Change sponsored?: --- Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Test Suite Assignee: chris(a)bigballofwax.co.nz Reporter: katrin.fischer(a)bsz-bw.de QA Contact: testopia(a)bugs.koha-community.org CC: jonathan.druart(a)gmail.com 17:00:43 koha_1 | # Failed test 'Time status changed equals time queued when created in message_queue table' 17:00:43 koha_1 | # at t/db_dependent/Letters.t line 156. 17:00:43 koha_1 | # got: '2024-11-15 16:00:00' 17:00:43 koha_1 | # expected: '2024-11-15 15:59:59' 17:00:43 koha_1 | # Looks like you failed 1 test of 103. 17:00:43 koha_1 | [16:00:09] t/db_dependent/Letters.t .................................................... It looks like it took a little longer than expected maybe and the test fails because of it. If this is the issue, we could make it a little more robust. -- You are receiving this mail because: You are watching all bug changes.

1 11