From 82e7c4020d95d195fc5df685ce5fd009ece4f99e Mon Sep 17 00:00:00 2001 From: Marcel de Rooy Date: Thu, 3 Mar 2022 10:27:20 +0000 Subject: [PATCH] Bug 30203: Circulation.t: prevent locking by setting envvar Running this test without prove is disastrous. The LOCK in the called SendCirculationAlert will ruin your data. But no longer when you apply this patch. Test plan: Run prove Circulation.t Inspect your data (e.g. borrowers table). Run perl Circulation.t Inspect your data (e.g. borrowers table) again. Signed-off-by: Marcel de Rooy Signed-off-by: Martin Renvoize Signed-off-by: Fridolin Somers Signed-off-by: Kyle M Hall (cherry picked from commit ac53c18af0896115db7cb8f7e9fd6c09ed3fd243) Signed-off-by: Andrew Fuerste-Henry --- t/db_dependent/Circulation.t | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/t/db_dependent/Circulation.t b/t/db_dependent/Circulation.t index da92680c7b..a5e36b2cbd 100755 --- a/t/db_dependent/Circulation.t +++ b/t/db_dependent/Circulation.t @@ -4905,6 +4905,11 @@ subtest "updateWrongTransfer tests" => sub { subtest "SendCirculationAlert" => sub { plan tests => 2; + # When you would unsuspectingly call this unit test (with perl, not prove), you will be bitten by LOCK. + # LOCK will commit changes and ruin your data + # In order to prevent that, we will add KOHA_TESTING to $ENV; see further Circulation.pm + $ENV{KOHA_TESTING} = 1; + # Setup branch, borrowr, and notice my $library = $builder->build_object({ class => 'Koha::Libraries' }); set_userenv( $library->unblessed); -- 2.39.5