Attachment 158177 Details for Bug 35215 – Bug 35215: Add few assumptions in Suggestions.t around emailing

[patch] Bug 35215: Add few assumptions in Suggestions.t around emailing

Bug-35215-Add-few-assumptions-in-Suggestionst-arou.patch (text/plain), 1.61 KB, created by Marcel de Rooy on 2023-11-01 15:23:11 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2023-11-01 15:23:11 UTC

Size: 1.61 KB

patch

obsolete

>From 9bc7a793625a7e9e6391d178653fc2d58e8fce56 Mon Sep 17 00:00:00 2001
>From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Date: Wed, 1 Nov 2023 14:50:47 +0000
>Subject: [PATCH] Bug 35215: Add few assumptions in Suggestions.t around
> emailing
>Content-Type: text/plain; charset=utf-8
>
>Test plan:
>Without this patch, set pref ReplytoDefault or EmailAddressForSuggestions.
>Run t/db_dependent/Suggestions.t (FAIL)
>Fill branch email and reply for CPL branch.
>Run t/db_dependent/Suggestions.t (FAIL: more errors)
>Apply patch.
>Run t/db_dependent/Suggestions.t (PASS)
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> t/db_dependent/Suggestions.t | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
>diff --git a/t/db_dependent/Suggestions.t b/t/db_dependent/Suggestions.t
>index 5a6f7a2bbe..abcbd626c5 100755
>--- a/t/db_dependent/Suggestions.t
>+++ b/t/db_dependent/Suggestions.t
>@@ -473,8 +473,10 @@ subtest 'EmailPurchaseSuggestions' => sub {
>         'New suggestion does not send an email when EmailPurchaseSuggestions disabled' );
> 
>     # EmailPurchaseSuggestions set to BranchEmailAddress
>-    t::lib::Mocks::mock_preference( "EmailPurchaseSuggestions",
>-        "BranchEmailAddress" );
>+    t::lib::Mocks::mock_preference( "EmailPurchaseSuggestions", "BranchEmailAddress" );
>+    t::lib::Mocks::mock_preference( "ReplytoDefault", "" );
>+    t::lib::Mocks::mock_preference( "EmailAddressForSuggestions", "" );
>+    Koha::Libraries->find('CPL')->update( { branchemail => '', branchreplyto => '' } );
>     Koha::Suggestion->new($my_suggestion)->store;
> 
>     t::lib::Mocks::mock_preference( "ReplytoDefault", 'library@b.c' );
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 35215: 158177 | 158393