Bug 32269 - Circulation.t is failing randomly
Summary: Circulation.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: Martin Renvoize
URL:
Keywords:
: 32670 (view as bug list)
Depends on:
Blocks: 25551
  Show dependency treegraph
 
Reported: 2022-11-18 14:31 UTC by Jonathan Druart
Modified: 2023-06-08 22:32 UTC (History)
3 users (show)

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


Attachments
Bug 32269: Prevent Circulation.t to fail randomly (1.65 KB, patch)
2022-11-18 14:32 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 32269: Prevent Circulation.t to fail randomly (1.71 KB, patch)
2022-11-19 20:02 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 32269: Prevent Circulation.t to fail randomly (1.77 KB, patch)
2022-11-23 10:04 UTC, Martin Renvoize
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2022-11-18 14:31:06 UTC
21:28:55 koha_1       |     #   Failed test 'Checkouts with auto-renewal can be renewed earliest on due date if no renewalbefore'
21:28:55 koha_1       |     #   at t/db_dependent/Circulation.t line 5683.
21:28:55 koha_1       |     #          got: '2022-11-17T20:25:30'
21:28:55 koha_1       |     #     expected: '2022-12-01T00:00:00'
21:28:55 koha_1       | 
21:28:55 koha_1       |     #   Failed test 'Checkouts with auto-renewal can be renewed earliest on due date if no renewalbefore'
21:28:55 koha_1       |     #   at t/db_dependent/Circulation.t line 5689.
21:28:55 koha_1       |     #          got: '2022-11-17T20:25:30'
21:28:55 koha_1       |     #     expected: '2022-12-01T23:59:00'
21:28:55 koha_1       |     # Looks like you failed 2 tests of 5.
Comment 1 Jonathan Druart 2022-11-18 14:32:56 UTC
Created attachment 144067 [details] [review]
Bug 32269: Prevent Circulation.t to fail randomly

Highlighted by Bug 32030: fix test builder for boolean

    #   Failed test 'Checkouts with auto-renewal can be renewed earliest on due date if no renewalbefore'
    #   at t/db_dependent/Circulation.t line 343.
    #          got: '2022-11-18T14:16:12'
    #     expected: '2022-12-02T00:00:00'

    #   Failed test 'Checkouts with auto-renewal can be renewed earliest on due date if no renewalbefore'
    #   at t/db_dependent/Circulation.t line 349.
    #          got: '2022-11-18T14:16:12'
    #     expected: '2022-12-02T23:59:00'
    # Looks like you failed 2 tests of 5

We were reaching return $now
Comment 2 Katrin Fischer 2022-11-19 20:02:02 UTC
Created attachment 144090 [details] [review]
Bug 32269: Prevent Circulation.t to fail randomly

Highlighted by Bug 32030: fix test builder for boolean

    #   Failed test 'Checkouts with auto-renewal can be renewed earliest on due date if no renewalbefore'
    #   at t/db_dependent/Circulation.t line 343.
    #          got: '2022-11-18T14:16:12'
    #     expected: '2022-12-02T00:00:00'

    #   Failed test 'Checkouts with auto-renewal can be renewed earliest on due date if no renewalbefore'
    #   at t/db_dependent/Circulation.t line 349.
    #          got: '2022-11-18T14:16:12'
    #     expected: '2022-12-02T23:59:00'
    # Looks like you failed 2 tests of 5

We were reaching return $now

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 3 Martin Renvoize 2022-11-23 10:04:23 UTC
Created attachment 144192 [details] [review]
Bug 32269: Prevent Circulation.t to fail randomly

Highlighted by Bug 32030: fix test builder for boolean

    #   Failed test 'Checkouts with auto-renewal can be renewed earliest on due date if no renewalbefore'
    #   at t/db_dependent/Circulation.t line 343.
    #          got: '2022-11-18T14:16:12'
    #     expected: '2022-12-02T00:00:00'

    #   Failed test 'Checkouts with auto-renewal can be renewed earliest on due date if no renewalbefore'
    #   at t/db_dependent/Circulation.t line 349.
    #          got: '2022-11-18T14:16:12'
    #     expected: '2022-12-02T23:59:00'
    # Looks like you failed 2 tests of 5

We were reaching return $now

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 4 Tomás Cohen Arazi 2022-11-23 12:48:21 UTC
Pushed to master for 22.11.

Nice work everyone, thanks!
Comment 5 Jonathan Druart 2023-01-18 16:22:04 UTC
Can you please backport random failure fixes?
Comment 6 Jonathan Druart 2023-01-18 16:22:31 UTC
*** Bug 32674 has been marked as a duplicate of this bug. ***
Comment 7 Jonathan Druart 2023-01-18 16:26:39 UTC
*** Bug 32670 has been marked as a duplicate of this bug. ***