Bug 16411

Summary: Make Hold.t not depend on two existing branches
Product: Koha Reporter: Marcel de Rooy <m.de.rooy>
Component: Test SuiteAssignee: Marcel de Rooy <m.de.rooy>
Status: CLOSED FIXED QA Contact: Galen Charlton <gmcharlt>
Severity: normal    
Priority: P5 - low CC: brendan, veron
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:
Circulation function:
Attachments: Bug 16411: Make Hold.t not depend on two existing branches
Bug 16411: Make Hold.t not depend on two existing branches
Bug 16411: Make Hold.t not depend on two existing branches

Description Marcel de Rooy 2016-05-02 08:11:34 UTC
If you do not have two branches, this test will fail.
Can't call method "branchcode" on an undefined value at t/db_dependent/Hold.t line 60.
Comment 1 Marcel de Rooy 2016-05-02 08:13:29 UTC Comment hidden (obsolete)
Comment 2 Marc Véron 2016-05-02 12:08:35 UTC Comment hidden (obsolete)
Comment 3 Tomás Cohen Arazi (tcohen) 2016-05-04 14:53:23 UTC
Created attachment 51192 [details] [review]
Bug 16411: Make Hold.t not depend on two existing branches

If you do not have two branches, this test will fail.
Can't call method "branchcode" on an undefined value.

This patch adds a borrower and two branches with TestBuilder.

Test plan:
Run the test.

Signed-off-by: Marc Véron <veron@veron.ch>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 4 Brendan Gallagher 2016-05-05 18:27:45 UTC
Pushed to Master - Should be in the May 2016 release.  Thanks!