Attachment 42495 Details for Bug 7634 – [PASSED QA] Bug 7634: Make the tests pass

[patch] [PASSED QA] Bug 7634: Make the tests pass

PASSED-QA-Bug-7634-Make-the-tests-pass.patch (text/plain), 1.52 KB, created by Katrin Fischer on 2015-09-13 15:34:08 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2015-09-13 15:34:08 UTC

Size: 1.52 KB

patch

obsolete

>From ad141bbad6636cefdeef00c26b1c8c505b8e92d6 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>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 <chrisc@catalyst.net.nz>
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> 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

Actions: View | Diff | Splinter Review

Attachments on bug 7634: 42222 | 42223 | 42224 | 42260 | 42261 | 42262 | 42495 | 42496 | 42497