From b5a4badd6d84d330d25fe3d9320b75446fcbec38 Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Date: Fri, 25 Nov 2022 13:06:40 +0100
Subject: [PATCH] Bug 32353: Don't generate item group for holds in tests

See bug 31447, we don't want the tests to deal with potential failures
because of some item groups.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit 813bd47b4c6052da7f320096e226ab013a054c5f)
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
---
 t/db_dependent/Circulation.t | 1 -
 t/lib/TestBuilder.pm         | 3 ++-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/t/db_dependent/Circulation.t b/t/db_dependent/Circulation.t
index 2ecb44bec4..35e4d91a0f 100755
--- a/t/db_dependent/Circulation.t
+++ b/t/db_dependent/Circulation.t
@@ -5767,7 +5767,6 @@ subtest "CanBookBeIssued + needsconfirmation message" => sub {
         priority => 1,
         found => undef,
         suspend => 0,
-        item_group_id => $item->item_group
     }});
 
     my ( $error, $needsconfirmation, $alerts, $messages );
diff --git a/t/lib/TestBuilder.pm b/t/lib/TestBuilder.pm
index 015d7bbd9a..8f8d79699a 100644
--- a/t/lib/TestBuilder.pm
+++ b/t/lib/TestBuilder.pm
@@ -602,7 +602,8 @@ sub _gen_default_values {
             pickup_location => 0,
         },
         Reserve => {
-            non_priority => 0,
+            non_priority  => 0,
+            item_group_id => undef,
         },
         Itemtype => {
             rentalcharge => 0,
-- 
2.39.5