Attachment 42475 Details for Bug 14801 – Bug 14801: Fix Reserves.t

[patch] Bug 14801: Fix Reserves.t

Bug-14801-Fix-Reservest.patch (text/plain), 2.51 KB, created by Marcel de Rooy on 2015-09-10 14:41:16 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2015-09-10 14:41:16 UTC

Size: 2.51 KB

patch

obsolete

>From 4df48876ee43f45b21f64759277df31ae2e5e631 Mon Sep 17 00:00:00 2001
>From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Date: Thu, 10 Sep 2015 16:39:51 +0200
>Subject: [PATCH] Bug 14801: Fix Reserves.t
>Content-Type: text/plain; charset=utf-8
>
>Fix the following errors:
>not ok 59 - Bug 14464 - No fines at beginning
>ok 60 - Bug 14464 - 1st reserve correctly created
>not ok 61 - Bug 14464 - No fines after cancelling reserve with no charge configured
>ok 62 - Bug 14464 - 2nd reserve correctly created
>not ok 63 - Bug 14464 - No fines after cancelling reserve with no charge desired
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> t/db_dependent/Reserves.t |    6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
>diff --git a/t/db_dependent/Reserves.t b/t/db_dependent/Reserves.t
>index 107cee9..c0007ba 100755
>--- a/t/db_dependent/Reserves.t
>+++ b/t/db_dependent/Reserves.t
>@@ -536,7 +536,7 @@ ok( !C4::Reserves::OnShelfHoldsAllowed($item, $borrower), "OnShelfHoldsAllowed()
> 
> $dbh->do("DELETE FROM reserves WHERE biblionumber=?",undef,($bibnum));
> my ( undef, undef, $bz14464_fines ) = GetMemberIssuesAndFines( $borrowernumber );
>-ok( !$bz14464_fines, 'Bug 14464 - No fines at beginning' );
>+is( $bz14464_fines==0, 1, 'Bug 14464 - No fines at beginning' );
> 
> # First, test cancelling a reserve when there's no charge configured.
> t::lib::Mocks::mock_preference('ExpireReservesMaxPickUpDelayCharge', 0);
>@@ -560,7 +560,7 @@ ok( $bz14464_reserve, 'Bug 14464 - 1st reserve correctly created' );
> CancelReserve({ reserve_id => $bz14464_reserve, charge_cancel_fee => 1 });
> 
> ( undef, undef, $bz14464_fines ) = GetMemberIssuesAndFines( $borrowernumber );
>-ok( !$bz14464_fines, 'Bug 14464 - No fines after cancelling reserve with no charge configured' );
>+is( $bz14464_fines==0, 1, 'Bug 14464 - No fines after cancelling reserve with no charge configured' );
> 
> # Then, test cancelling a reserve when there's no charge desired.
> t::lib::Mocks::mock_preference('ExpireReservesMaxPickUpDelayCharge', 42);
>@@ -584,7 +584,7 @@ ok( $bz14464_reserve, 'Bug 14464 - 2nd reserve correctly created' );
> CancelReserve({ reserve_id => $bz14464_reserve });
> 
> ( undef, undef, $bz14464_fines ) = GetMemberIssuesAndFines( $borrowernumber );
>-ok( !$bz14464_fines, 'Bug 14464 - No fines after cancelling reserve with no charge desired' );
>+is( $bz14464_fines==0, 1, 'Bug 14464 - No fines after cancelling reserve with no charge desired' );
> 
> # Finally, test cancelling a reserve when there's a charge desired and configured.
> $bz14464_reserve = AddReserve(
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 14801: 42475 | 42476 | 42500 | 42506 | 42507 | 42508