Attachment 115081 Details for Bug 17229 – Bug 17229: Add unit test for BlockExpiredPatronOpacActions check

[patch] Bug 17229: Add unit test for BlockExpiredPatronOpacActions check

Bug-17229-Add-unit-test-for-BlockExpiredPatronOpac.patch (text/plain), 1.79 KB, created by Martin Renvoize (ashimema) on 2021-01-12 12:22:32 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Renvoize (ashimema)

Created: 2021-01-12 12:22:32 UTC

Size: 1.79 KB

patch

obsolete

>From 2798ed154c9544acab0c298a70991e9a36259d8b Mon Sep 17 00:00:00 2001
>From: Julian Maurice <julian.maurice@biblibre.com>
>Date: Wed, 31 Aug 2016 15:52:08 +0200
>Subject: [PATCH] Bug 17229: Add unit test for BlockExpiredPatronOpacActions
> check
>
>Signed-off-by: Brendan Gallagher <brendan@bywatersolutions.com>
>Signed-off-by: David Nind <david@davidnind.com>
>Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>---
> t/db_dependent/Holds.t | 20 +++++++++++++++++++-
> 1 file changed, 19 insertions(+), 1 deletion(-)
>
>diff --git a/t/db_dependent/Holds.t b/t/db_dependent/Holds.t
>index 3e736f6774..cee1ce72ac 100755
>--- a/t/db_dependent/Holds.t
>+++ b/t/db_dependent/Holds.t
>@@ -43,7 +43,12 @@ my $dbh     = C4::Context->dbh;
> my $branch_1 = $builder->build({ source => 'Branch' })->{ branchcode };
> my $branch_2 = $builder->build({ source => 'Branch' })->{ branchcode };
> 
>-my $category = $builder->build({ source => 'Category' });
>+my $category = $builder->build({
>+    source => 'Category',
>+    value => {
>+        BlockExpiredPatronOpacActions => -1,
>+    },
>+});
> 
> my $borrowers_count = 5;
> 
>@@ -127,6 +132,19 @@ my $patron = Koha::Patrons->find( $borrowernumbers[0] );
> $holds = $patron->holds;
> is( $holds->next->borrowernumber, $borrowernumbers[0], "Test Koha::Patron->holds");
> 
>+my $expired_borrowernumber = Koha::Patron->new({
>+        firstname =>  'Expired',
>+        surname => 'Patron',
>+        categorycode => $category->{categorycode},
>+        branchcode => $branch_1,
>+	dateexpiry => '2000-01-01',
>+    })->store->borrowernumber;
>+
>+t::lib::Mocks::mock_preference('BlockExpiredPatronOpacActions', 1);
>+ok(
>+    CanItemBeReserved($expired_borrowernumber, $itemnumber)->{status} eq 'patronExpired',
>+    'Expired patron cannot reserve'
>+);
> 
> $holds = $item->current_holds;
> $first_hold = $holds->next;
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 17229: 55059 | 55060 | 62036 | 62037 | 67324 | 67325 | 70682 | 70683 | 72970 | 72971 | 85064 | 85065 | 85066 | 85070 | 85071 | 85072 | 85118 | 85883 | 89760 | 99701 | 99702 | 99703 | 99817 | 99818 | 99819 | 113741 | 113742 | 113743 | 115081 | 115082 | 115083 | 115411 | 115435