Bug 24507

Summary: Checkouts/ReturnClaim.t is failing on MySQL 8
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Test SuiteAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: 1joynelson, lucas, martin.renvoize, tomascohen
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23825
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24509
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
20.05.00, 19.11.03
Bug Depends on: 14697    
Bug Blocks: 23996    
Attachments: Bug 24507: Make Checkouts/ReturnClaim.t pass on MySQL 8
Bug 24507: Make Checkouts/ReturnClaim.t pass on MySQL 8

Description Jonathan Druart 2020-01-24 10:14:27 UTC
$ prove t/db_dependent/Koha/Checkouts/ReturnClaim.t
t/db_dependent/Koha/Checkouts/ReturnClaim.t .. 
    #   Failed test 'Exception field is correct'
    #   at t/db_dependent/Koha/Checkouts/ReturnClaim.t line 97.
    #          got: 'return_claims.issue_id'
    #     expected: 'issue_id'
    # Looks like you failed 1 test of 8.
t/db_dependent/Koha/Checkouts/ReturnClaim.t .. 1/1 
#   Failed test 'store() tests'
#   at t/db_dependent/Koha/Checkouts/ReturnClaim.t line 133.

This is similar to bug 23825.
Comment 1 Jonathan Druart 2020-01-24 10:15:49 UTC
Created attachment 97864 [details] [review]
Bug 24507: Make Checkouts/ReturnClaim.t pass on MySQL 8

$ prove t/db_dependent/Koha/Checkouts/ReturnClaim.t
 t/db_dependent/Koha/Checkouts/ReturnClaim.t ..
     #   Failed test 'Exception field is correct'
     #   at t/db_dependent/Koha/Checkouts/ReturnClaim.t line 97.
     #          got: 'return_claims.issue_id'
     #     expected: 'issue_id'
     # Looks like you failed 1 test of 8.
 t/db_dependent/Koha/Checkouts/ReturnClaim.t .. 1/1
 #   Failed test 'store() tests'
 #   at t/db_dependent/Koha/Checkouts/ReturnClaim.t line 133.

This is similar to bug 23825.

Test plan:
Confirm that the tests pass on MySQL8 and MariaDB
Comment 2 Tomás Cohen Arazi 2020-01-24 12:29:13 UTC
Created attachment 97876 [details] [review]
Bug 24507: Make Checkouts/ReturnClaim.t pass on MySQL 8

 $ prove t/db_dependent/Koha/Checkouts/ReturnClaim.t
 t/db_dependent/Koha/Checkouts/ReturnClaim.t ..
     #   Failed test 'Exception field is correct'
     #   at t/db_dependent/Koha/Checkouts/ReturnClaim.t line 97.
     #          got: 'return_claims.issue_id'
     #     expected: 'issue_id'
     # Looks like you failed 1 test of 8.
 t/db_dependent/Koha/Checkouts/ReturnClaim.t .. 1/1
 #   Failed test 'store() tests'
 #   at t/db_dependent/Koha/Checkouts/ReturnClaim.t line 133.

This is similar to bug 23825.

Test plan:
Confirm that the tests pass on MySQL8 and MariaDB

Signed-off-by: Tomás Cohen Arazi <tomascohen@theke.io>
Comment 3 Tomás Cohen Arazi 2020-01-24 12:30:01 UTC
Trivial fix, tested on ku-my8. Passing QA.
Comment 4 Martin Renvoize 2020-01-24 15:21:27 UTC
Nice work everyone!

Pushed to master for 20.05
Comment 5 Joy Nelson 2020-02-10 22:31:26 UTC
Pushed to 19.11.x for 19.11.03
Comment 6 Lucas Gass 2020-02-14 17:50:54 UTC
missing dependencies for 19.05.x, no backport