From 38268581ed8c8a402c45826d2cfc41915e980cdd Mon Sep 17 00:00:00 2001 From: Aleisha Amohia Date: Tue, 13 Aug 2024 03:07:17 +0000 Subject: [PATCH] Bug 37623: Make t/db_dependent/Letters.t idempotent The tests should be unaffected by system preference settings, such as EmailFieldPrimary. To test: 1. Log into the staff interface 2. Go to Koha Administration -> Global system preferences, and search for EmailFieldPrimary 3. Change the EmailFieldPrimary preference to something other than 'primary email' and 'first valid email address', i.e. any of the following: * alternate email * card number * secondary email * selected addresses 4. Run tests t/db_dependent/Letters.t and notice they fail 5. Apply the patch 6. Run the tests again t/db_dependent/Letters.t with different settings of EmailFieldPrimary. The tests should always pass. Sponsored-by: Pymble Ladies' College Signed-off-by: Paul Derscheid --- t/db_dependent/Letters.t | 1 + 1 file changed, 1 insertion(+) diff --git a/t/db_dependent/Letters.t b/t/db_dependent/Letters.t index 662df68de9..d607fa7d04 100755 --- a/t/db_dependent/Letters.t +++ b/t/db_dependent/Letters.t @@ -74,6 +74,7 @@ my $dbh = C4::Context->dbh; $dbh->do(q|DELETE FROM letter|); $dbh->do(q|DELETE FROM message_queue|); $dbh->do(q|DELETE FROM message_transport_types|); +t::lib::Mocks::mock_preference( 'EmailFieldPrimary', 'OFF' ); my $library = $builder->build({ source => 'Branch', -- 2.46.0