| 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 | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | 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! |