From 9e3edaf4881f28abaa57014204ec36b7202c4a72 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Wed, 2 Sep 2015 14:43:15 +0100 Subject: [PATCH] Bug 7634: Make the tests pass This patch just fixed a master bug, if your database already has some items.homebranch set to CPL --- t/db_dependent/Items.t | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/t/db_dependent/Items.t b/t/db_dependent/Items.t index b1796bc..fb5d672 100755 --- a/t/db_dependent/Items.t +++ b/t/db_dependent/Items.t @@ -236,6 +236,8 @@ subtest 'SearchItems test' => sub { $dbh->{RaiseError} = 1; C4::Context->set_preference('marcflavour', 'MARC21'); + my $cpl_items_before = SearchItemsByField( 'homebranch', 'CPL'); + my ($biblionumber) = get_biblio(); # Add branches if they don't exist @@ -384,8 +386,8 @@ subtest 'SearchItems test' => sub { ($items, $total_results) = SearchItems($filter); ok(scalar @$items == 1, 'found 1 item with itemnotes = "foobar"'); - my $cpl_items = SearchItemsByField( 'homebranch', 'CPL'); - is( ( $cpl_items and scalar( @$cpl_items ) ), 1, 'SearchItemsByField should return something' ); + my $cpl_items_after = SearchItemsByField( 'homebranch', 'CPL'); + is( ( scalar( @$cpl_items_after ) - scalar ( @$cpl_items_before ) ), 1, 'SearchItemsByField should return something' ); $dbh->rollback; }; -- 2.1.0