From ad141bbad6636cefdeef00c26b1c8c505b8e92d6 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Wed, 2 Sep 2015 14:43:15 +0100 Subject: [PATCH] [PASSED QA] Bug 7634: Make the tests pass This patch just fixed a master bug, if your database already has some items.homebranch set to CPL Signed-off-by: Chris Cormack Signed-off-by: Katrin Fischer --- 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 47afb41..c6d4ccc 100755 --- a/t/db_dependent/Items.t +++ b/t/db_dependent/Items.t @@ -250,6 +250,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 @@ -398,8 +400,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; }; -- 1.9.1