From b97565ac3b9385968ca3421ec1ce879d851ac6cd Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Date: Tue, 3 Mar 2020 14:26:06 +0100
Subject: [PATCH] Bug 24754: Test intranet at the same time

---
 t/db_dependent/Circulation/issue.t | 48 +++++++++++++++++++++---------
 1 file changed, 34 insertions(+), 14 deletions(-)

diff --git a/t/db_dependent/Circulation/issue.t b/t/db_dependent/Circulation/issue.t
index c45e9a1e4b..25c1dff432 100644
--- a/t/db_dependent/Circulation/issue.t
+++ b/t/db_dependent/Circulation/issue.t
@@ -245,9 +245,7 @@ my $stat = $dbh->selectrow_hashref("SELECT * FROM statistics WHERE type = 'renew
 ok( $stat, "Bug 17781 - 'Improper branchcode set during renewal' still fixed" );
 
 subtest 'Show that AddRenewal respects OpacRenewalBranch and interface' => sub {
-    plan tests => 5;
-
-    $se->mock( 'interface', sub { return 'opac' } );
+    plan tests => 10;
 
     my $item_library = $builder->build_object( { class => 'Koha::Libraries' } );
     my $patron       = $builder->build_object( { class => 'Koha::Patrons' } );
@@ -266,19 +264,41 @@ subtest 'Show that AddRenewal respects OpacRenewalBranch and interface' => sub {
 
         t::lib::Mocks::mock_preference( 'OpacRenewalBranch', $syspref );
 
-        my $item = $builder->build_sample_item(
-            { library => $item_library->branchcode, itype => $itemtype } );
-        my $opac_renew_issue =
-          C4::Circulation::AddIssue( $patron->unblessed, $item->barcode );
+        {
+            $se->mock( 'interface', sub { return 'opac' } );
+
+            my $item = $builder->build_sample_item(
+                { library => $item_library->branchcode, itype => $itemtype } );
+            my $opac_renew_issue =
+              C4::Circulation::AddIssue( $patron->unblessed, $item->barcode );
+
+            AddRenewal( $patron->borrowernumber, $item->itemnumber,
+                "Stavromula", $datedue1, $daysago10 );
+
+            my $stat = Koha::Statistics->search(
+                { itemnumber => $item->itemnumber, type => 'renew' } )->next;
+            is( $stat->branch, $expected_branchcode,
+                "->renewal_branchcode is respected for OpacRenewalBranch = $syspref"
+            );
+        }
+
+        {
+            $se->mock( 'interface', sub { return 'intranet' } );
 
-        AddRenewal( $patron->borrowernumber, $item->itemnumber,
-            "Stavromula", $datedue1, $daysago10 );
+            my $item = $builder->build_sample_item(
+                { library => $item_library->branchcode, itype => $itemtype } );
+            my $opac_renew_issue =
+              C4::Circulation::AddIssue( $patron->unblessed, $item->barcode );
 
-        my $stat = Koha::Statistics->search(
-            { itemnumber => $item->itemnumber, type => 'renew' } )->next;
-        is( $stat->branch, $expected_branchcode,
-            "->renewal_branchcode is respected for OpacRenewalBranch = $syspref"
-        );
+            AddRenewal( $patron->borrowernumber, $item->itemnumber,
+                "Stavromula", $datedue1, $daysago10 );
+
+            my $stat = Koha::Statistics->search(
+                { itemnumber => $item->itemnumber, type => 'renew' } )->next;
+            is( $stat->branch, $logged_in_user->branchcode,
+                "->renewal_branchcode is always logged in branch for intranet"
+            );
+        }
     }
 };
 
-- 
2.20.1