Attachment 65283 Details for Bug 18977 – Bug 18977: Rollback branch in t/db_dependent/SIP/Message.t

[patch] Bug 18977: Rollback branch in t/db_dependent/SIP/Message.t

Bug-18977-Rollback-branch-in-tdbdependentSIPMessag.patch (text/plain), 1.79 KB, created by Jonathan Druart on 2017-07-26 17:37:28 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2017-07-26 17:37:28 UTC

Size: 1.79 KB

patch

obsolete

>From 139b66f65611cff0a2551662e0fd5c2e50b4c10f Mon Sep 17 00:00:00 2001
>From: Lari Taskula <lari.taskula@jns.fi>
>Date: Mon, 24 Jul 2017 18:30:10 +0300
>Subject: [PATCH] Bug 18977: Rollback branch in t/db_dependent/SIP/Message.t
>
>To test:
>1. Before applying patch, check the amount of branches in database
>2. prove t/db_dependent/SIP/Message.t
>3. See that a new branch is stored
>4. Apply patch
>5. prove t/db_dependent/SIP/Message.t
>6. See that a new branch is no longer stored
>
>Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>---
> t/db_dependent/SIP/Message.t | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
>diff --git a/t/db_dependent/SIP/Message.t b/t/db_dependent/SIP/Message.t
>index 0c1ebccd61..e738405141 100755
>--- a/t/db_dependent/SIP/Message.t
>+++ b/t/db_dependent/SIP/Message.t
>@@ -54,15 +54,12 @@ my $mockILS = Test::MockObject->new;
> $mockILS->mock( 'check_inst_id', sub {} );
> $mockILS->mock( 'institution_id', sub { $branchcode; } );
> $mockILS->mock( 'find_patron', sub { $findpatron; } );
>-$branch = $builder->build({
>-    source => 'Branch',
>-});
>-$branchcode = $branch->{branchcode};
> 
> # START testing
> subtest 'Testing Patron Status Request V2' => sub {
>     $schema->storage->txn_begin;
>     plan tests => 13;
>+    $branchcode = $builder->build({ source => 'Branch' })->{branchcode};
>     $C4::SIP::Sip::protocol_version = 2;
>     test_request_patron_status_v2();
>     $schema->storage->txn_rollback;
>@@ -71,6 +68,7 @@ subtest 'Testing Patron Status Request V2' => sub {
> subtest 'Testing Patron Info Request V2' => sub {
>     $schema->storage->txn_begin;
>     plan tests => 18;
>+    $branchcode = $builder->build({ source => 'Branch' })->{branchcode};
>     $C4::SIP::Sip::protocol_version = 2;
>     test_request_patron_info_v2();
>     $schema->storage->txn_rollback;
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 18977: 65198 | 65283