Summary: | Make Hold.t not depend on two existing branches | ||
---|---|---|---|
Product: | Koha | Reporter: | Marcel de Rooy <m.de.rooy> |
Component: | Test Suite | Assignee: | 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
Created attachment 51071 [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. Created attachment 51094 [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> 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> Pushed to Master - Should be in the May 2016 release. Thanks! |