Bug 36486

Summary: Add tests for Koha::DateTime::Format::SQL
Product: Koha Reporter: Julian Maurice <julian.maurice>
Component: Test SuiteAssignee: Julian Maurice <julian.maurice>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: fridolin.somers, lucas, m.de.rooy
Version: Main   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: Small patch Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
24.05.00,23.11.06
Circulation function:
Bug Depends on: 36432    
Bug Blocks:    
Attachments: Bug 36486: Add tests for Koha::DateTime::Format::SQL
Bug 36486: Add tests for Koha::DateTime::Format::SQL
Bug 36486: Fix file perms and tidiness
Bug 36486: Add tests for Koha::DateTime::Format::SQL
Bug 36486: Fix file perms and tidiness

Description Julian Maurice 2024-04-02 07:33:57 UTC
Bug 36432 is missing some tests for Koha::DateTime::Format::SQL.
Since it is already Passed QA, I will attach them in this new bug report.
Comment 1 Julian Maurice 2024-04-02 07:35:14 UTC
Created attachment 164235 [details] [review]
Bug 36486: Add tests for Koha::DateTime::Format::SQL
Comment 2 Matt Blenkinsop 2024-05-09 10:40:20 UTC
Created attachment 166452 [details] [review]
Bug 36486: Add tests for Koha::DateTime::Format::SQL

Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Comment 3 Marcel de Rooy 2024-05-10 08:19:40 UTC
 FAIL   t/Koha/DateTime/Format/SQL.t
   FAIL   file permissions
                File must have the exec flag
   WARN   tidiness
                The file is less tidy than before (bad/messy lines before: 0, now: 27)
Comment 4 Julian Maurice 2024-05-10 08:31:18 UTC
Created attachment 166508 [details] [review]
Bug 36486: Fix file perms and tidiness
Comment 5 Marcel de Rooy 2024-05-10 08:45:01 UTC
Created attachment 166511 [details] [review]
Bug 36486: Add tests for Koha::DateTime::Format::SQL

Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 6 Marcel de Rooy 2024-05-10 08:45:03 UTC
Created attachment 166512 [details] [review]
Bug 36486: Fix file perms and tidiness

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 7 Katrin Fischer 2024-05-10 13:32:34 UTC
I am never saying no to more tests :)
Comment 8 Katrin Fischer 2024-05-10 14:47:08 UTC
Pushed for 24.05!

Well done everyone, thank you!
Comment 9 Fridolin Somers 2024-05-29 09:00:48 UTC
Pushed to 23.11.x for 23.11.06
Comment 10 Lucas Gass (lukeg) 2024-05-30 19:06:45 UTC
Enhancement will not be included in 23.05.x