Bug 12583: (followup) rename new test file
authorTomas Cohen Arazi <tomascohen@gmail.com>
Sat, 26 Jul 2014 13:13:28 +0000 (10:13 -0300)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Sat, 26 Jul 2014 13:17:09 +0000 (10:17 -0300)
We should reserve the use of dirs for namespaces. Method specific tests
should be included in a single file with subtests, or stick to Package_method.t.

This should be discussed, so we rearrange the remaining tests.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
t/db_dependent/Items/DelItem.t [deleted file]
t/db_dependent/Items_DelItem.t [new file with mode: 0644]

diff --git a/t/db_dependent/Items/DelItem.t b/t/db_dependent/Items/DelItem.t
deleted file mode 100644 (file)
index 35ff6e4..0000000
+++ /dev/null
@@ -1,49 +0,0 @@
-use Modern::Perl;
-
-use MARC::Record;
-use C4::Biblio;
-
-use Test::More tests => 7;
-
-BEGIN {
-    use_ok('C4::Items');
-}
-
-my $dbh = C4::Context->dbh;
-$dbh->{AutoCommit} = 0;
-$dbh->{RaiseError} = 1;
-
-my ( $biblionumber, $bibitemnum ) = get_biblio();
-
-my ( $item_bibnum, $item_bibitemnum, $itemnumber );
-( $item_bibnum, $item_bibitemnum, $itemnumber ) =
-  AddItem( { homebranch => 'CPL', holdingbranch => 'CPL' }, $biblionumber );
-
-my $deleted = DelItem( { biblionumber => $biblionumber, itemnumber => $itemnumber } );
-is( $deleted, 1, "DelItem should return 1 if the item has been deleted" );
-my $deleted_item = GetItem($itemnumber);
-is( $deleted_item->{itemnumber}, undef, "DelItem with biblionumber parameter - the item should be deleted." );
-
-( $item_bibnum, $item_bibitemnum, $itemnumber ) =
-  AddItem( { homebranch => 'CPL', holdingbranch => 'CPL' }, $biblionumber );
-$deleted = DelItem( { biblionumber => $biblionumber, itemnumber => $itemnumber } );
-is( $deleted, 1, "DelItem should return 1 if the item has been deleted" );
-$deleted_item = GetItem($itemnumber);
-is( $deleted_item->{itemnumber}, undef, "DelItem without biblionumber parameter - the item should be deleted." );
-
-$deleted = DelItem( { itemnumber => $itemnumber + 1} );
-is ( $deleted, 0, "DelItem should return 0 if no item has been deleted" );
-
-$deleted = DelItem( { itemnumber => $itemnumber + 1, biblionumber => $biblionumber } );
-is ( $deleted, 0, "DelItem should return 0 if no item has been deleted" );
-
-# Helper method to set up a Biblio.
-sub get_biblio {
-    my $bib = MARC::Record->new();
-    $bib->append_fields(
-        MARC::Field->new( '100', ' ', ' ', a => 'Moffat, Steven' ),
-        MARC::Field->new( '245', ' ', ' ', a => 'Silence in the library' ),
-    );
-    my ( $bibnum, $bibitemnum ) = AddBiblio( $bib, '' );
-    return ( $bibnum, $bibitemnum );
-}
diff --git a/t/db_dependent/Items_DelItem.t b/t/db_dependent/Items_DelItem.t
new file mode 100644 (file)
index 0000000..35ff6e4
--- /dev/null
@@ -0,0 +1,49 @@
+use Modern::Perl;
+
+use MARC::Record;
+use C4::Biblio;
+
+use Test::More tests => 7;
+
+BEGIN {
+    use_ok('C4::Items');
+}
+
+my $dbh = C4::Context->dbh;
+$dbh->{AutoCommit} = 0;
+$dbh->{RaiseError} = 1;
+
+my ( $biblionumber, $bibitemnum ) = get_biblio();
+
+my ( $item_bibnum, $item_bibitemnum, $itemnumber );
+( $item_bibnum, $item_bibitemnum, $itemnumber ) =
+  AddItem( { homebranch => 'CPL', holdingbranch => 'CPL' }, $biblionumber );
+
+my $deleted = DelItem( { biblionumber => $biblionumber, itemnumber => $itemnumber } );
+is( $deleted, 1, "DelItem should return 1 if the item has been deleted" );
+my $deleted_item = GetItem($itemnumber);
+is( $deleted_item->{itemnumber}, undef, "DelItem with biblionumber parameter - the item should be deleted." );
+
+( $item_bibnum, $item_bibitemnum, $itemnumber ) =
+  AddItem( { homebranch => 'CPL', holdingbranch => 'CPL' }, $biblionumber );
+$deleted = DelItem( { biblionumber => $biblionumber, itemnumber => $itemnumber } );
+is( $deleted, 1, "DelItem should return 1 if the item has been deleted" );
+$deleted_item = GetItem($itemnumber);
+is( $deleted_item->{itemnumber}, undef, "DelItem without biblionumber parameter - the item should be deleted." );
+
+$deleted = DelItem( { itemnumber => $itemnumber + 1} );
+is ( $deleted, 0, "DelItem should return 0 if no item has been deleted" );
+
+$deleted = DelItem( { itemnumber => $itemnumber + 1, biblionumber => $biblionumber } );
+is ( $deleted, 0, "DelItem should return 0 if no item has been deleted" );
+
+# Helper method to set up a Biblio.
+sub get_biblio {
+    my $bib = MARC::Record->new();
+    $bib->append_fields(
+        MARC::Field->new( '100', ' ', ' ', a => 'Moffat, Steven' ),
+        MARC::Field->new( '245', ' ', ' ', a => 'Silence in the library' ),
+    );
+    my ( $bibnum, $bibitemnum ) = AddBiblio( $bib, '' );
+    return ( $bibnum, $bibitemnum );
+}