Bug 13451

Summary: Holidays.t assumes default data
Product: Koha Reporter: Mark Tompsett <mtompset>
Component: Test SuiteAssignee: Mark Tompsett <mtompset>
Status: CLOSED FIXED QA Contact: Galen Charlton <gmcharlt>
Severity: minor    
Priority: P5 - low CC: chris, tomascohen
Version: Main   
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 13451: Holidays.t assumes default data
[SIGNED OFF] Bug 13451: Holidays.t assumes default data
[PASSED QA] Bug 13451: Holidays.t assumes default data

Description Mark Tompsett 2014-12-12 16:30:15 UTC
With rollback and non-commit capabilities, we should be able to have prove t/db_dependent/Holidays.t pass without having loaded default data! This will fix that.
Comment 1 Mark Tompsett 2014-12-12 16:56:22 UTC Comment hidden (obsolete)
Comment 2 Nick Clemens 2014-12-13 17:08:43 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2014-12-14 20:15:48 UTC
Created attachment 34379 [details] [review]
[PASSED QA] Bug 13451: Holidays.t assumes default data

This patch removes that assumption by expressly setting data
to be default.

TEST PLAN
---------
1) Ensure that branch code is NOT 'MPL' in the
   repeatable_holidays table in your database.
2) Ensure that branch codes 'MPL' and 'CPL' do not exist
   in the branches table in your database.
3) prove t/db_dependent/Holidays.t
   -- this should bomb horribly.
4) Apply patch
5) prove t/db_dependent/Holidays.t
   -- all tests should succeed.
6) run koha qa tests

Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Tests pass without holidays in the calendar.
Comment 4 Tomás Cohen Arazi 2014-12-17 22:33:36 UTC
Patch pushed to master.

Thanks Mark!
Comment 5 Chris Cormack 2014-12-18 00:10:58 UTC
No string changes, pushed to 3.18.x will be in 3.18.2