Bug 13456

Summary: Serials/Claims.t expects no subscriptions
Product: Koha Reporter: Mark Tompsett <mtompset>
Component: Test SuiteAssignee: Mark Tompsett <mtompset>
Status: CLOSED FIXED QA Contact: Jonathan Druart <jonathan.druart>
Severity: minor    
Priority: P5 - low CC: chris, jonathan.druart, paola.rossi, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 13456: Serials/Claims.t expects no subscriptions
Bug 13456: Serials/Claims.t expects no subscriptions
Bug 13456: Serials/Claims.t expects no subscriptions

Description Mark Tompsett 2014-12-12 21:31:19 UTC
Running prove t/db_dependent/Serials/Claims.t on any system with late subscriptions is going to trigger a failure. Temporary delete the subscriptions for testing purposes, and then rollback.
Comment 1 Mark Tompsett 2014-12-12 21:35:54 UTC Comment hidden (obsolete)
Comment 2 Paola Rossi 2014-12-17 11:18:00 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2014-12-19 10:12:01 UTC
Created attachment 34525 [details] [review]
Bug 13456: Serials/Claims.t expects no subscriptions

However, this is an unreasonable assumption for a system which
is in use (either lots of testing or production).

TEST PLAN
---------
1) Have a supplier with a late subscription.
2) prove t/db_dependent/Serials/Claims.t
   -- will fail
3) apply patch
4) prove t/db_dependent/Serials/Claims.t
   -- success
5) run koha qa test tools

Signed-off-by: Paola Rossi <paola.rossi@cineca.it>

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 4 Tomás Cohen Arazi 2014-12-19 21:01:15 UTC
Patch pushed to master.

Thanks Mark!
Comment 5 Mark Tompsett 2014-12-19 21:33:55 UTC
(In reply to Paola Rossi from comment #2)
[SNIP]
> Processing files before patches
> |========================>| 1 / 1 (100.00%)
> Processing files after patches
> |========================>| 1 / 1 (100.00%)
[SNIP]

Thanks to Paola Possi for making me check if there were updates to the QA test tool!
Comment 6 Chris Cormack 2014-12-19 23:55:44 UTC
Pushed to 3.18.x, will be in 3.18.2