Bug 13453

Summary: Koha.t daily quote tests assume sample 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, katrin.fischer, mtj, tomascohen, veron
Version: 3.16   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 13453: Koha.t daily quote tests assume sample data
Bug 13453: Koha.t daily quote tests assume sample data
[PASSED QA] Bug 13453: Koha.t daily quote tests assume sample data

Description Mark Tompsett 2014-12-12 17:20:10 UTC
This assumes that id 3 will be Abraham Lincoln. If you haven't loaded the sample data prove t/db_dependent/Koha.t failed on the daily quote tests.
Comment 1 Mark Tompsett 2014-12-12 17:31:12 UTC Comment hidden (obsolete)
Comment 2 Marc Véron 2014-12-20 18:43:10 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2014-12-27 10:21:36 UTC
Created attachment 34698 [details] [review]
[PASSED QA] Bug 13453: Koha.t daily quote tests assume sample data

By adding quotes 3 and 25 from the sample data, this test can
pass without having the sample quote data loaded.

TEST PLAN
---------
1) Ensure there is no quote id=3 or that it is NOT
   Abraham Lincoln.
2) prove t/db_dependent/Koha.t
   -- this should fail the daily quote test.
3) apply patch
4) prove t/db_dependent/Koha.t
   -- this should *NOT* fail the daily quote test.
5) run koha qa test tools

Followed test plan 1)-4). Without patch, daily quote test failed. With patch, test passed OK.
Signed-off-by: Marc Véron <veron@veron.ch>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Works as described, leaves actual data unchanged.
Comment 4 Tomás Cohen Arazi 2014-12-28 23:03:27 UTC
Patch pushed to master.

Thanks Mark!
Comment 5 Chris Cormack 2014-12-29 21:33:21 UTC
Pushed to 3.18.x will be in 3.18.3
Comment 6 Mason James 2015-01-16 19:28:20 UTC
Pushed to 3.16.x, patch will be in 3.16.7