From e93241a7174930c52c6f03560027cd527492148d Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Mon, 7 May 2018 13:15:34 -0300 Subject: [PATCH] Bug 20721: (bug 19403 follow-up) Prevent Circulation.t to fail randomly If items.restricted == 1, CanBookBeIssued will not returned what we are testing. The easiest and global fix is to define a default value at TestBuilder package level Signed-off-by: Jonathan Druart Signed-off-by: Nick Clemens --- t/db_dependent/Circulation.t | 22 ---------------------- t/lib/TestBuilder.pm | 4 ++++ 2 files changed, 4 insertions(+), 22 deletions(-) diff --git a/t/db_dependent/Circulation.t b/t/db_dependent/Circulation.t index a6afc56cb7..b062383473 100755 --- a/t/db_dependent/Circulation.t +++ b/t/db_dependent/Circulation.t @@ -1207,10 +1207,6 @@ subtest 'CanBookBeIssued & AllowReturnToBranch' => sub { value => { homebranch => $homebranch->{branchcode}, holdingbranch => $holdingbranch->{branchcode}, - notforloan => 0, - itemlost => 0, - withdrawn => 0, - restricted => 0, biblionumber => $biblioitem->{biblionumber} } } @@ -1368,10 +1364,6 @@ subtest 'CanBookBeIssued + Koha::Patron->is_debarred|has_overdues' => sub { value => { homebranch => $library->{branchcode}, holdingbranch => $library->{branchcode}, - notforloan => 0, - itemlost => 0, - withdrawn => 0, - restricted => 0, biblionumber => $biblioitem_1->{biblionumber} } } @@ -1382,10 +1374,6 @@ subtest 'CanBookBeIssued + Koha::Patron->is_debarred|has_overdues' => sub { value => { homebranch => $library->{branchcode}, holdingbranch => $library->{branchcode}, - notforloan => 0, - itemlost => 0, - withdrawn => 0, - restricted => 0, biblionumber => $biblioitem_2->{biblionumber} } } @@ -1449,10 +1437,6 @@ subtest 'CanBookBeIssued + Statistic patrons "X"' => sub { value => { homebranch => $library->branchcode, holdingbranch => $library->branchcode, - notforloan => 0, - itemlost => 0, - withdrawn => 0, - restricted => 0, biblionumber => $biblioitem_1->{biblionumber} } } @@ -1579,9 +1563,6 @@ subtest 'CanBookBeIssued + AllowMultipleIssuesOnABiblio' => sub { value => { homebranch => $library->{branchcode}, holdingbranch => $library->{branchcode}, - notforloan => 0, - itemlost => 0, - withdrawn => 0, biblionumber => $biblionumber, } } @@ -1591,9 +1572,6 @@ subtest 'CanBookBeIssued + AllowMultipleIssuesOnABiblio' => sub { value => { homebranch => $library->{branchcode}, holdingbranch => $library->{branchcode}, - notforloan => 0, - itemlost => 0, - withdrawn => 0, biblionumber => $biblionumber, } } diff --git a/t/lib/TestBuilder.pm b/t/lib/TestBuilder.pm index d42cb03346..1b8d5818b5 100644 --- a/t/lib/TestBuilder.pm +++ b/t/lib/TestBuilder.pm @@ -471,6 +471,10 @@ sub _gen_default_values { borrowernotes => '', }, Item => { + notforloan => 0, + itemlost => 0, + withdrawn => 0, + restricted => 0, more_subfields_xml => undef, }, }; -- 2.39.5