Bug 27342: (QA follow-up) Fix test
[koha-ffzg.git] / t / Biblio.t
index 9a1b8f4..c298349 100755 (executable)
 #!/usr/bin/perl
+
+# This file is part of Koha.
+#
+# Koha is free software; you can redistribute it and/or modify it
+# under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
+#
+# Koha is distributed in the hope that it will be useful, but
+# WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
 #
-use strict;
-use warnings;
+# You should have received a copy of the GNU General Public License
+# along with Koha; if not, see <http://www.gnu.org/licenses>.
+
+use Modern::Perl;
+
+use Test::More;
+use Test::MockModule;
+use Test::Warn;
 
-use Test::More tests => 22;
+plan tests => 34;
 
-BEGIN {
-        use_ok('C4::Biblio');
-}
 
-# test returns if undef record passed
-# carp messages appear on stdout
+use_ok('C4::Biblio', qw( AddBiblio ModBiblio BiblioAutoLink LinkBibHeadingsToAuthorities GetMarcPrice GetMarcQuantity GetMarcControlnumber GetMarcISBN GetMarcISSN GetMarcSubjects GetMarcUrls GetMarcSeries TransformMarcToKoha ModBiblioMarc RemoveAllNsb UpdateTotalIssues ));
+
+my $db = Test::MockModule->new('Koha::Database');
+$db->mock( _new_schema => sub { return Schema(); } );
+
+my @arr;
+my $ret;
+
+warning_is { @arr = AddBiblio(undef, q{}) }
+           { carped => 'AddBiblio called with undefined record'},
+           "AddBiblio returns carped warning on undef record";
 
-my @arr = AddBiblio(undef, q{});
 my $elements = @arr;
 
 is($elements, 0, 'Add Biblio returns empty array for undef record');
 
-my $ret = ModBiblio(undef, 0, '');
+warning_is { $ret = ModBiblio(undef, 0, '') }
+           { carped => 'No record passed to ModBiblio'},
+           "ModBiblio returns carped warning on undef record";
 
 is( $ret, 0, 'ModBiblio returns zero if not passed rec');
 
-$ret = BiblioAutoLink(undef, q{});
+warning_is { $ret = BiblioAutoLink(undef, q{}) }
+           { carped => 'Undefined record passed to BiblioAutoLink'},
+           "BiblioAutoLink returns carped warning on undef record";
 
 is( $ret, 0, 'BiblioAutoLink returns zero if not passed rec');
 
-$ret = GetRecordValue('100', undef, q{});
-ok( !defined $ret, 'GetRecordValue returns undef if not passed rec');
+warning_is { @arr = LinkBibHeadingsToAuthorities(q{}, q{}) }
+           { carped => 'LinkBibHeadingsToAuthorities called on undefined bib record'},
+           "LinkBibHeadingsToAuthorities returns carped warning on undef record";
 
-@arr = LinkBibHeadingsToAuthorities(q{}, q{});
 is($arr[0], 0, 'LinkBibHeadingsToAuthorities correct error return');
 
-$ret = GetCOinSBiblio();
-ok( !defined $ret, 'GetCOinSBiblio returns undef if not passed rec');
+warning_is { $ret = GetMarcPrice(undef, 'MARC21') }
+           { carped => 'GetMarcPrice called on undefined record'},
+           "GetMarcPrice returns carped warning on undef record";
 
-$ret = GetMarcPrice(undef, 'MARC21');
 ok( !defined $ret, 'GetMarcPrice returns undef if not passed rec');
 
-$ret = GetMarcQuantity(undef, 'MARC21');
+warning_is { $ret = GetMarcQuantity(undef, 'MARC21') }
+           { carped => 'GetMarcQuantity called on undefined record'},
+           "GetMarcQuantity returns carped warning on undef record";
+
 ok( !defined $ret, 'GetMarcQuantity returns undef if not passed rec');
 
-$ret = GetMarcControlnumber();
+warning_is { $ret = GetMarcControlnumber() }
+           { carped => 'GetMarcControlnumber called on undefined record'},
+           "GetMarcControlnumber returns carped warning on undef record";
+
 ok( !defined $ret, 'GetMarcControlnumber returns undef if not passed rec');
 
-$ret = GetMarcISBN();
+warning_is { $ret = GetMarcISBN() }
+           { carped => 'GetMarcISBN called on undefined record'},
+           "GetMarcISBN returns carped warning on undef record";
+
 ok( !defined $ret, 'GetMarcISBN returns undef if not passed rec');
 
-$ret = GetMarcISSN();
+warning_is { $ret = GetMarcISSN() }
+           { carped => 'GetMarcISSN called on undefined record'},
+           "GetMarcISSN returns carped warning on undef record";
+
 ok( !defined $ret, 'GetMarcISSN returns undef if not passed rec');
 
-$ret = GetMarcNotes();
-ok( !defined $ret, 'GetMarcNotes returns undef if not passed rec');
+warning_is { $ret = GetMarcSubjects() }
+           { carped => 'GetMarcSubjects called on undefined record'},
+           "GetMarcSubjects returns carped warning on undef record";
 
-$ret = GetMarcSubjects();
 ok( !defined $ret, 'GetMarcSubjects returns undef if not passed rec');
 
-$ret = GetMarcAuthors();
-ok( !defined $ret, 'GetMarcAuthors returns undef if not passed rec');
+warning_is { $ret = GetMarcUrls() }
+           { carped => 'GetMarcUrls called on undefined record'},
+           "GetMarcUrls returns carped warning on undef record";
 
-$ret = GetMarcUrls();
 ok( !defined $ret, 'GetMarcUrls returns undef if not passed rec');
 
-$ret = GetMarcSeries();
+warning_is { $ret = GetMarcSeries() }
+           { carped => 'GetMarcSeries called on undefined record'},
+           "GetMarcSeries returns carped warning on undef record";
+
 ok( !defined $ret, 'GetMarcSeries returns undef if not passed rec');
 
-$ret = GetMarcHosts();
-ok( !defined $ret, 'GetMarcHosts returns undef if not passed rec');
+my $hash_ref;
 
-my $hash_ref = TransformMarcToKoha(undef, undef);
+warning_is { $hash_ref = TransformMarcToKoha( undef) }
+           { carped => 'TransformMarcToKoha called with undefined record'},
+           "TransformMarcToKoha returns carped warning on undef record";
 
 isa_ok( $hash_ref, 'HASH');
 
@@ -75,8 +118,22 @@ $elements = keys %{$hash_ref};
 
 is($elements, 0, 'Empty hashref returned for undefined record in TransformMarcToKoha');
 
-$ret = ModBiblioMarc();
+warning_is { $ret = ModBiblioMarc() }
+           { carped => 'ModBiblioMarc passed an undefined record'},
+           "ModBiblioMarc returns carped warning on undef record";
+
 ok( !defined $ret, 'ModBiblioMarc returns undef if not passed rec');
 
-$ret = RemoveAllNsb();
+warning_is { $ret = RemoveAllNsb() }
+           { carped => 'RemoveAllNsb called with undefined record'},
+           "RemoveAllNsb returns carped warning on undef record";
+
 ok( !defined $ret, 'RemoveAllNsb returns undef if not passed rec');
+
+
+warnings_like { $ret = UpdateTotalIssues() }
+              [
+                { carped => qr/UpdateTotalIssues could not get biblio/ } ],
+    "UpdateTotalIssues returns carped warnings if biblio record does not exist";
+
+ok( !defined $ret, 'UpdateTotalIssues returns carped warning if biblio record does not exist');