Bug 15956: Move SIPILS.t to SIP/ILS.t
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Wed, 2 Mar 2016 13:33:27 +0000 (14:33 +0100)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 29 Apr 2016 13:38:33 +0000 (13:38 +0000)
Move the file. Some trivial (cosmetic) edits: whitespace, comments.

Test plan:
Run SIP/ILS.t

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
t/db_dependent/SIP/ILS.t [new file with mode: 0755]
t/db_dependent/SIPILS.t [deleted file]

diff --git a/t/db_dependent/SIP/ILS.t b/t/db_dependent/SIP/ILS.t
new file mode 100755 (executable)
index 0000000..77eafa2
--- /dev/null
@@ -0,0 +1,58 @@
+#!/usr/bin/perl
+
+# Tests for C4::SIP::ILS
+# Please help to extend them!
+
+# 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.
+#
+# 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 tests => 9;
+
+BEGIN {
+    use_ok('C4::SIP::ILS');
+}
+
+my $class = 'C4::SIP::ILS';
+my $institution = { id => 'CPL', };
+
+my $ils = $class->new( $institution );
+
+isa_ok( $ils, $class );
+
+# Check all methods required for interface are there
+my @methods = qw(
+    find_patron find_item checkout_ok checkin_ok offline_ok status_update_ok
+    offline_ok checkout checkin end_patron_session pay_fee add_hold cancel_hold
+    alter_hold renew renew_all
+);
+
+can_ok( $ils, @methods );
+
+is( $ils->institution(), 'CPL', 'institution method returns id' );
+
+is( $ils->institution_id(), 'CPL', 'institution_id method returns id' );
+
+is( $ils->supports('checkout'), 1, 'checkout supported' );
+
+is( $ils->supports('security_inhibit'),
+    q{}, 'unsupported feature returns false' );
+
+is( $ils->test_cardnumber_compare( 'A1234', 'a1234' ),
+    1, 'borrower bc test is case insensitive' );
+
+is( $ils->test_cardnumber_compare( 'A1234', 'b1234' ),
+    q{}, 'borrower bc test identifies difference' );
diff --git a/t/db_dependent/SIPILS.t b/t/db_dependent/SIPILS.t
deleted file mode 100755 (executable)
index 196c9b2..0000000
+++ /dev/null
@@ -1,56 +0,0 @@
-#!/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.
-#
-# 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 tests => 9;
-
-BEGIN {
-    use_ok('C4::SIP::ILS');
-}
-
-my $class = 'C4::SIP::ILS';
-my $institution = { id => 'CPL', };
-
-my $ils = $class->new($institution);
-
-isa_ok( $ils, $class );
-
-# Check all methods required for interface are there
-# NB the mon_block routines are not here but Sip.pm thinks it can call them
-my @methods = (
-    qw( find_patron find_item checkout_ok checkin_ok offline_ok status_update_ok
-      offline_ok checkout checkin end_patron_session pay_fee add_hold cancel_hold
-      alter_hold renew renew_all)
-);
-
-can_ok( $ils, @methods );
-
-is( $ils->institution(), 'CPL', 'institution method returns id' );
-
-is( $ils->institution_id(), 'CPL', 'institution_id method returns id' );
-
-is( $ils->supports('checkout'), 1, 'checkout supported' );
-
-is( $ils->supports('security_inhibit'),
-    q{}, 'unsupported feature returns false' );
-
-is( $ils->test_cardnumber_compare( 'A1234', 'a1234' ),
-    1, 'borrower bc test is case insensitive' );
-
-is( $ils->test_cardnumber_compare( 'A1234', 'b1234' ),
-    q{}, 'borrower bc test identifies difference' );