From a989a151c2b333419b47b3463faa7fc6cd842200 Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Wed, 3 Jan 2024 19:46:15 +0000 Subject: [PATCH] Bug 35293: Unit test Signed-off-by: David Nind Signed-off-by: Nick Clemens --- t/db_dependent/Circulation/issue.t | 28 ++++++++++++++++++++++++---- 1 file changed, 24 insertions(+), 4 deletions(-) diff --git a/t/db_dependent/Circulation/issue.t b/t/db_dependent/Circulation/issue.t index c9cd7f26ffa..86720d8edfb 100755 --- a/t/db_dependent/Circulation/issue.t +++ b/t/db_dependent/Circulation/issue.t @@ -17,7 +17,7 @@ use Modern::Perl; -use Test::More tests => 66; +use Test::More tests => 67; use DateTime::Duration; use t::lib::Mocks; @@ -462,11 +462,31 @@ AddReturn( 'barcode_3', $branchcode_1 ); $item = Koha::Items->find( $itemnumber ); ok( $item->notforloan eq 0, q{UpdateNotForLoanStatusOnCheckin _ALL_ rules are applied if there are no specific item type rule matching the returned item} ); -t::lib::Mocks::mock_preference( 'UpdateNotForLoanStatusOnCheckin', '1: ONLYMESSAGE' ); +t::lib::Mocks::mock_preference( + 'UpdateNotForLoanStatusOnCheckin', q{_ALL_: + 1: ONLYMESSAGE +} +); $item->notforloan(1)->store; AddReturn( 'barcode_3', $branchcode_1 ); -$item = Koha::Items->find( $itemnumber ); -ok( $item->notforloan eq 1, q{UpdateNotForLoanStatusOnCheckin does not update notforloan value from 1 with setting "1: ONLYMESSAGE"} ); +$item->discard_changes; +is( + $item->notforloan, 1, + q{UpdateNotForLoanStatusOnCheckin does not update notforloan value from 1 for _ALL_ with setting "1: ONLYMESSAGE"} +); + + +t::lib::Mocks::mock_preference( + 'UpdateNotForLoanStatusOnCheckin', "$itemtype:\n + 1: ONLYMESSAGE +" +); +AddReturn( 'barcode_3', $branchcode_1 ); +$item->discard_changes; +is( + $item->notforloan, 1, + q{UpdateNotForLoanStatusOnCheckin does not update notforloan value for specific itemtype from 1 with setting "1: ONLYMESSAGE"} +); my $itemnumber2 = Koha::Item->new( { -- 2.30.2