From dceca78e07661157b3b72994d04ea0744cf902b7 Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Tue, 31 Mar 2020 09:43:34 +0100 Subject: [PATCH] Bug 22750: (24474: (QA follow-up) Fix failing test) The test was assuming default data and would fail if you ran through the test plan (and disabled all but `onpayment` for the 'MarkLostItemsAsReturned' preference) on the data before running the included unit tests. Signed-off-by: David Nind Signed-off-by: Kyle M Hall --- t/db_dependent/Circulation.t | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/t/db_dependent/Circulation.t b/t/db_dependent/Circulation.t index 90d852d3bb..c7a6068b40 100755 --- a/t/db_dependent/Circulation.t +++ b/t/db_dependent/Circulation.t @@ -18,7 +18,7 @@ use Modern::Perl; use utf8; -use Test::More tests => 47; +use Test::More tests => 48; use Test::MockModule; use Data::Dumper; @@ -2036,6 +2036,7 @@ subtest 'CanBookBeIssued + AutoReturnCheckedOutItems' => sub { subtest 'AddReturn | is_overdue' => sub { plan tests => 6; + t::lib::Mocks::mock_preference('MarkLostItemsAsReturned', 'batchmod|moredetail|cronjob|additem|pendingreserves|onpayment'); t::lib::Mocks::mock_preference('CalculateFinesOnReturn', 1); t::lib::Mocks::mock_preference('finesMode', 'production'); t::lib::Mocks::mock_preference('MaxFine', '100'); -- 2.20.1