Bugzilla – Attachment 32876 Details for
Bug 11634
Allow renewal of item with unfilled holds if other available items can fill those holds
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[PASSED QA] Bug 11634 [QA Followup] - Unit Tests
PASSED-QA-Bug-11634-QA-Followup---Unit-Tests.patch (text/plain), 1.76 KB, created by
Katrin Fischer
on 2014-10-28 22:11:28 UTC
(
hide
)
Description:
[PASSED QA] Bug 11634 [QA Followup] - Unit Tests
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2014-10-28 22:11:28 UTC
Size:
1.76 KB
patch
obsolete
>From a70c2834dda85b961593b1d9b37596edd663ec59 Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Tue, 4 Mar 2014 12:58:28 -0500 >Subject: [PATCH] [PASSED QA] Bug 11634 [QA Followup] - Unit Tests > >These new unit tests will fail due to the fact that Koha::Database >uses a separate dbh handle than C4::Context->dbh > >Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> >--- > t/db_dependent/Circulation.t | 9 ++++++++- > 1 file changed, 8 insertions(+), 1 deletion(-) > >diff --git a/t/db_dependent/Circulation.t b/t/db_dependent/Circulation.t >index 1740795..a284c4f 100755 >--- a/t/db_dependent/Circulation.t >+++ b/t/db_dependent/Circulation.t >@@ -26,7 +26,7 @@ use C4::Reserves; > use Koha::DateUtils; > use Koha::Database; > >-use Test::More tests => 55; >+use Test::More tests => 57; > > BEGIN { > use_ok('C4::Circulation'); >@@ -282,6 +282,13 @@ C4::Context->dbh->do("DELETE FROM accountlines"); > $title, $checkitem, $found > ); > >+ C4::Context->set_preference('AllowRenewalIfOtherItemsAvailable', 1 ); >+ ( $renewokay, $error ) = CanBookBeRenewed($renewing_borrowernumber, $itemnumber); >+ is( $renewokay, 1, 'Bug 11634 - Allow renewal of item with unfilled holds if other available items can fill those holds'); >+ ( $renewokay, $error ) = CanBookBeRenewed($renewing_borrowernumber, $itemnumber2); >+ is( $renewokay, 1, 'Bug 11634 - Allow renewal of item with unfilled holds if other available items can fill those holds'); >+ C4::Context->set_preference('AllowRenewalIfOtherItemsAvailable', 0 ); >+ > ( $renewokay, $error ) = CanBookBeRenewed($renewing_borrowernumber, $itemnumber); > is( $renewokay, 0, '(Bug 10663) Cannot renew, reserved'); > is( $error, 'on_reserve', '(Bug 10663) Cannot renew, reserved (returned error is on_reserve)'); >-- >1.9.1
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 11634
:
24866
|
25032
|
25148
|
25822
|
25825
|
25826
|
25827
|
25837
|
28212
|
28213
|
28214
|
31706
|
31707
|
31708
|
31709
|
32034
|
32035
|
32036
|
32037
|
32875
| 32876 |
32877
|
32878
|
33492