From 4b277f3feeaa2f2223982be8e1501831ce4fe3d2 Mon Sep 17 00:00:00 2001 From: Tomas Cohen Arazi Date: Wed, 28 Aug 2024 14:56:44 -0300 Subject: [PATCH] Bug 37757: Unit tests Signed-off-by: David Nind Signed-off-by: Martin Renvoize --- t/db_dependent/Koha/Patron.t | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/t/db_dependent/Koha/Patron.t b/t/db_dependent/Koha/Patron.t index bf4aeef0970..fec2bb5ef2c 100755 --- a/t/db_dependent/Koha/Patron.t +++ b/t/db_dependent/Koha/Patron.t @@ -1789,17 +1789,19 @@ subtest 'notify_library_of_registration()' => sub { $schema->storage->txn_rollback; }; -subtest 'notice_email_address' => sub { +subtest 'notice_email_address() tests' => sub { + plan tests => 3; + $schema->storage->txn_begin; my $patron = $builder->build_object( { class => 'Koha::Patrons' } ); t::lib::Mocks::mock_preference( 'EmailFieldPrecedence', 'email|emailpro' ); - t::lib::Mocks::mock_preference( 'EmailFieldPrimary', 'OFF' ); + t::lib::Mocks::mock_preference( 'EmailFieldPrimary', undef ); is( $patron->notice_email_address, $patron->email, - "Koha::Patron->notice_email_address returns correct value when EmailFieldPrimary is off" + "Koha::Patron->notice_email_address returns correct value when EmailFieldPrimary is not defined" ); t::lib::Mocks::mock_preference( 'EmailFieldPrimary', 'emailpro' ); @@ -1815,7 +1817,6 @@ subtest 'notice_email_address' => sub { "Koha::Patron->notice_email_address returns correct value when EmailFieldPrimary is 'MULTI' and EmailFieldSelection is 'email,emailpro'" ); - $patron->delete; $schema->storage->txn_rollback; }; -- 2.46.0