Bug 32710 - UI/Form/Builder/Item.t is failing randomly
Summary: UI/Form/Builder/Item.t is failing randomly
Status: RESOLVED FIXED
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: 28445
Blocks: 25551
  Show dependency treegraph
 
Reported: 2023-01-24 15:11 UTC by Jonathan Druart
Modified: 2023-12-28 20:44 UTC (History)
1 user (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.05.00,22.11.05


Attachments
Bug 32710: Try to prevent UI/Form/Builder/Item.t to fail randomly (2.19 KB, patch)
2023-03-31 08:40 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 32710: Try to prevent UI/Form/Builder/Item.t to fail randomly (2.24 KB, patch)
2023-03-31 09:33 UTC, Tomás Cohen Arazi
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2023-01-24 15:11:20 UTC
koha_1       |         #   Failed test 'Item types should be sorted by description and an empty entry should be shown'
koha_1       |         #   at t/db_dependent/Koha/UI/Form/Builder/Item.t line 135.
koha_1       |         #     Structures begin differing at:
koha_1       |         #          $got->[8] = 'TjMK6RmH'
koha_1       |         #     $expected->[8] = 'REF'
koha_1       |         # Looks like you failed 1 test of 2.
koha_1       | 
koha_1       |     #   Failed test 'itemtypes'
koha_1       |     #   at t/db_dependent/Koha/UI/Form/Builder/Item.t line 143.
koha_1       |     # Looks like you failed 1 test of 5.
koha_1       | 
koha_1       | #   Failed test 'authorised values'
koha_1       | #   at t/db_dependent/Koha/UI/Form/Builder/Item.t line 144.
koha_1       | # Looks like you failed 1 test of 9.
koha_1       | [15:16:06] t/db_dependent/Koha/UI/Form/Builder/Item.t

Koha_Master_D10/1006
Comment 1 Jonathan Druart 2023-03-31 07:33:52 UTC
koha_1       |         #   Failed test 'Item types should be sorted by description and an empty entry should be shown'
koha_1       |         #   at t/db_dependent/Koha/UI/Form/Builder/Item.t line 135.
koha_1       |         #     Structures begin differing at:
koha_1       |         #          $got->[11] = 'qbZe6s__5Y'
koha_1       |         #     $expected->[11] = 'REF'
koha_1       |         # Looks like you failed 1 test of 2.
koha_1       | 
koha_1       |     #   Failed test 'itemtypes'
koha_1       |     #   at t/db_dependent/Koha/UI/Form/Builder/Item.t line 143.
koha_1       |     # Looks like you failed 1 test of 5.
koha_1       | 
koha_1       | #   Failed test 'authorised values'
koha_1       | #   at t/db_dependent/Koha/UI/Form/Builder/Item.t line 144.
koha_1       | # Looks like you failed 1 test of 9.
koha_1       | [16:48:09] t/db_dependent/Koha/UI/Form/Builder/Item.t

Koha_Master/2449
Comment 2 Jonathan Druart 2023-03-31 08:40:26 UTC
Created attachment 149018 [details] [review]
Bug 32710: Try to prevent UI/Form/Builder/Item.t to fail randomly

See the FIXME, and commit 77a34e099a9f3bcef2076a4170e8c8d843c403eb
Actually we are doing the trick (of removing _) before we are creating
another itemtype, we need to do it right before we are comparing.

Test plan:
Run in a loop, be patient, it failed for me (without the patch) at run
245
Comment 3 Tomás Cohen Arazi 2023-03-31 09:33:58 UTC
Created attachment 149020 [details] [review]
Bug 32710: Try to prevent UI/Form/Builder/Item.t to fail randomly

See the FIXME, and commit 77a34e099a9f3bcef2076a4170e8c8d843c403eb
Actually we are doing the trick (of removing _) before we are creating
another itemtype, we need to do it right before we are comparing.

Test plan:
Run in a loop, be patient, it failed for me (without the patch) at run
245

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 4 Tomás Cohen Arazi 2023-03-31 10:04:18 UTC
Pushed to master for 23.05.

Nice work everyone, thanks!
Comment 5 Matt Blenkinsop 2023-04-11 11:20:03 UTC
Nice work everyone!

Pushed to stable for 22.11.x