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.
Created attachment 32530 [details] [review] 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
Created attachment 32532 [details] [review] 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>
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... :)
Patch pushed to master. Thanks Kyle!