From 8715457c701bf5ee5563ad02797146d85931a047 Mon Sep 17 00:00:00 2001 From: Mark Tompsett Date: Thu, 29 Mar 2018 18:44:32 +0000 Subject: [PATCH] Bug 20499: Unit Test to prove the problem TEST PLAN --------- 1) apply this patch 2) prove t/db_dependent/Circulation.t -- horrible death. 3) apply next path 4) prove t/db_dependent/Circulation.t -- passes. 5) run koha qa test tools --- t/db_dependent/Circulation.t | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/t/db_dependent/Circulation.t b/t/db_dependent/Circulation.t index f4ff177..9d1c6bc 100755 --- a/t/db_dependent/Circulation.t +++ b/t/db_dependent/Circulation.t @@ -1193,7 +1193,7 @@ C4::Context->dbh->do("DELETE FROM accountlines"); } subtest 'CanBookBeIssued & AllowReturnToBranch' => sub { - plan tests => 23; + plan tests => 24; my $homebranch = $builder->build( { source => 'Branch' } ); my $holdingbranch = $builder->build( { source => 'Branch' } ); @@ -1225,6 +1225,10 @@ subtest 'CanBookBeIssued & AllowReturnToBranch' => sub { # AllowReturnToBranch == anywhere t::lib::Mocks::mock_preference( 'AllowReturnToBranch', 'anywhere' ); + ## Test that unknown barcodes don't generate internal server errors + set_userenv($homebranch); + ( $error, $question, $alerts ) = CanBookBeIssued( $patron_2, 'KohaIsAwesome' ); + ok( $error->{UNKNOWN_BARCODE}, '"KohaIsAwesome" is not a valid barcode as expected.' ); ## Can be issued from homebranch set_userenv($homebranch); ( $error, $question, $alerts ) = CanBookBeIssued( $patron_2, $item->{barcode} ); -- 2.1.4