Attachment 61056 Details for Bug 18242 – Bug 18242: [SOLUTION 2]Add tests

[patch] Bug 18242: [SOLUTION 2]Add tests

Bug-18242-SOLUTION-2Add-tests.patch (text/plain), 1.83 KB, created by Chris Cormack on 2017-03-14 00:36:16 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Chris Cormack

Created: 2017-03-14 00:36:16 UTC

Size: 1.83 KB

patch

obsolete

>From 9ac5db9cd9964966979d6dcfba6207bffbc532d8 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Thu, 9 Mar 2017 16:41:31 -0300
>Subject: [PATCH] Bug 18242: [SOLUTION 2]Add tests
>
>Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
>---
> t/db_dependent/Circulation/Returns.t | 28 +++++++++++++++++++++++++++-
> 1 file changed, 27 insertions(+), 1 deletion(-)
>
>diff --git a/t/db_dependent/Circulation/Returns.t b/t/db_dependent/Circulation/Returns.t
>index f13dfd2305..de7bf64d1e 100644
>--- a/t/db_dependent/Circulation/Returns.t
>+++ b/t/db_dependent/Circulation/Returns.t
>@@ -17,7 +17,7 @@
> 
> use Modern::Perl;
> 
>-use Test::More tests => 3;
>+use Test::More tests => 4;
> use Test::MockModule;
> use Test::Warn;
> 
>@@ -262,4 +262,30 @@ subtest "AddReturn logging on statistics table (item-level_itypes=0)" => sub {
>         "biblio-level itype recorded on statistics for return");
> };
> 
>+subtest 'Handle ids duplication' => sub {
>+    plan tests => 1;
>+
>+    my $biblio = $builder->build( { source => 'Biblio' } );
>+    my $item = $builder->build(
>+        {
>+            source => 'Item',
>+            value  => {
>+                biblionumber => $biblio->{biblionumber},
>+                notforloan => 0,
>+                itemlost   => 0,
>+                withdrawn  => 0,
>+
>+            }
>+        }
>+    );
>+    my $patron = $builder->build({source => 'Borrower'});
>+
>+    my $checkout = AddIssue( $patron, $item->{barcode} );
>+    $builder->build({ source => 'OldIssue', value => { issue_id => $checkout->issue_id } });
>+
>+    my @a = AddReturn( $item->{barcode} );
>+    my $old_checkout = Koha::Old::Checkouts->find( $checkout->issue_id );
>+    isnt( $old_checkout->itemnumber, $item->{itemnumber}, 'If an item is checked-in, it should be moved to old_issues even if the issue_id already existed in the table' );
>+};
>+
> 1;
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 18242: 60969 | 60970 | 60971 | 60972 | 60973 | 61056 | 61057 | 61254 | 61255 | 61794 | 63168 | 63169 | 63170