Bug 13115

Summary: Holidays.t will fail some unit tests on Sundays
Product: Koha Reporter: Kyle M Hall <kyle>
Component: Architecture, internals, and plumbingAssignee: Galen Charlton <gmcharlt>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: major    
Priority: P5 - low CC: katrin.fischer, tomascohen
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 13115 - Holidays.t will fail some unit tests on Sundays
Bug 13115 - Holidays.t will fail some unit tests on Sundays
[PASSED QA] Bug 13115 - Holidays.t will fail some unit tests on Sundays

Description Kyle M Hall 2014-10-20 13:03:51 UTC
Due to the fact that Sunday is a repeating holiday in the example data which is used for part but not all of Holidays.t, the last test will fail on Sundays. The test is to see if today is *not* a special holiday for MPL, but since it's already a repeating holiday, is_holiday will return true rather than false.
Comment 1 Kyle M Hall 2014-10-20 13:05:20 UTC Comment hidden (obsolete)
Comment 2 Tomás Cohen Arazi 2014-10-20 13:08:49 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2014-10-21 21:26:15 UTC
Created attachment 32567 [details] [review]
[PASSED QA] Bug 13115 - Holidays.t will fail some unit tests on Sundays

Due to the fact that Sunday is a repeating holiday in the example data
which is used for part but not all of Holidays.t, the last test will
fail on Sundays. The test is to see if today is *not* a special holiday
for MPL, but since it's already a repeating holiday, is_holiday will
return true rather than false.

Test Plan:
1) Wait until Sunday
2) prove t/db_dependent/Holidays.t
3) Note the failure
4) Apply this patch
5) Repeat step 2
6) Note there is no longer a failure

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Used my time machine... :)
Comment 4 Tomás Cohen Arazi 2014-10-22 00:12:48 UTC
Patch pushed to master.

Thanks Kyle!