First time we get this one (Koha_Master/1700): 12:01:12 koha_1 | # Failed test 'Holds retrieved' 12:01:12 koha_1 | # at t/db_dependent/api/v1/patrons_holds.t line 56. 12:01:12 koha_1 | # Structures begin differing at: 12:01:12 koha_1 | # $got->[0]{hold_id} = '256' 12:01:12 koha_1 | # $expected->[0]{hold_id} = '255' 12:01:12 koha_1 | # Looks like you failed 1 test of 9. 12:01:12 koha_1 | 12:01:12 koha_1 | # Failed test 'list() tests' 12:01:12 koha_1 | # at t/db_dependent/api/v1/patrons_holds.t line 70.
Still valid : https://jenkins.koha-community.org/view/master/job/Koha_Master_D11_CPAN/510/testReport/
Koha_Master_D10/1003
(In reply to Jonathan Druart from comment #2) > Koha_Master_D10/1003 13:42:30 koha_1 | # Failed test 'Holds retrieved' 13:42:30 koha_1 | # at t/db_dependent/api/v1/patrons_holds.t line 56. 13:42:30 koha_1 | # Structures begin differing at: 13:42:30 koha_1 | # $got->[0]{priority} = '3042' 13:47:59 koha_1 | # $expected->[0]{priority} = '18716' 13:47:59 koha_1 | # Looks like you failed 1 test of 9. 13:47:59 koha_1 | 13:47:59 koha_1 | # Failed test 'list() tests' 13:47:59 koha_1 | # at t/db_dependent/api/v1/patrons_holds.t line 70.
Created attachment 145396 [details] [review] Bug 28670: Prevent api/v1/patrons_holds.t to fail randomly 12:01:12 koha_1 | # Failed test 'Holds retrieved' 12:01:12 koha_1 | # at t/db_dependent/api/v1/patrons_holds.t line 56. 12:01:12 koha_1 | # Structures begin differing at: 12:01:12 koha_1 | # $got->[0]{hold_id} = '256' 12:01:12 koha_1 | # $expected->[0]{hold_id} = '255' 12:01:12 koha_1 | # Looks like you failed 1 test of 9. 12:01:12 koha_1 | 12:01:12 koha_1 | # Failed test 'list() tests' 12:01:12 koha_1 | # at t/db_dependent/api/v1/patrons_holds.t line 70. This is a weird one, and the only solution I see it to force the order we want to receive the holds. I guess we should not rely on an order if no order by clause is passed
Created attachment 145398 [details] [review] Bug 28670: Prevent api/v1/patrons_holds.t to fail randomly 12:01:12 koha_1 | # Failed test 'Holds retrieved' 12:01:12 koha_1 | # at t/db_dependent/api/v1/patrons_holds.t line 56. 12:01:12 koha_1 | # Structures begin differing at: 12:01:12 koha_1 | # $got->[0]{hold_id} = '256' 12:01:12 koha_1 | # $expected->[0]{hold_id} = '255' 12:01:12 koha_1 | # Looks like you failed 1 test of 9. 12:01:12 koha_1 | 12:01:12 koha_1 | # Failed test 'list() tests' 12:01:12 koha_1 | # at t/db_dependent/api/v1/patrons_holds.t line 70. This is a weird one, and the only solution I see it to force the order we want to receive the holds. I guess we should not rely on an order if no order by clause is passed Signed-off-by: Owen Leonard <oleonard@myacpl.org>
I'm not able to reproduce the failure but I can confirm that the test passes successfully with the patch.
Created attachment 145401 [details] [review] Bug 28670: Prevent api/v1/patrons_holds.t to fail randomly 12:01:12 koha_1 | # Failed test 'Holds retrieved' 12:01:12 koha_1 | # at t/db_dependent/api/v1/patrons_holds.t line 56. 12:01:12 koha_1 | # Structures begin differing at: 12:01:12 koha_1 | # $got->[0]{hold_id} = '256' 12:01:12 koha_1 | # $expected->[0]{hold_id} = '255' 12:01:12 koha_1 | # Looks like you failed 1 test of 9. 12:01:12 koha_1 | 12:01:12 koha_1 | # Failed test 'list() tests' 12:01:12 koha_1 | # at t/db_dependent/api/v1/patrons_holds.t line 70. This is a weird one, and the only solution I see it to force the order we want to receive the holds. I guess we should not rely on an order if no order by clause is passed Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
I've seen this before and had no idea why it happened then either.. but I ended up fixing it in the same way. I think we should push this patch to appease the Jenkins gods but bear it in mind for the future to try and work out why it happens. Passing QA
Nice work everyone! Pushed to 23.05.x for the next release
Nice work, thanks everyone! Pushed to 22.11.x for the next release.