Bugzilla – Attachment 45481 Details for
Bug 12802
Send notices using several email addresses
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 12802: Fix tests in Letters.t
Bug-12802-Fix-tests-in-Letterst.patch (text/plain), 2.68 KB, created by
Alex Arnaud
on 2015-12-08 09:42:17 UTC
(
hide
)
Description:
Bug 12802: Fix tests in Letters.t
Filename:
MIME Type:
Creator:
Alex Arnaud
Created:
2015-12-08 09:42:17 UTC
Size:
2.68 KB
patch
obsolete
>From 5d97a156497d8b4f4b2e4919fa3269fd286590a0 Mon Sep 17 00:00:00 2001 >From: Matthias Meusburger <matthias.meusburger@biblibre.com> >Date: Tue, 2 Jun 2015 10:19:32 +0200 >Subject: [PATCH] Bug 12802: Fix tests in Letters.t > >--- > t/db_dependent/Letters.t | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > >diff --git a/t/db_dependent/Letters.t b/t/db_dependent/Letters.t >index e7e00b9..efee863 100644 >--- a/t/db_dependent/Letters.t >+++ b/t/db_dependent/Letters.t >@@ -440,19 +440,19 @@ is( scalar( @$messages ), 1, 'The message has been queued for the expected patro > is( $messages->[0]{to_address}, $email, 'AutoEmailPrimaryAddress=OFF, EnqueueLetter used the patron email address if no to_address is given in parameter' ); > > $dbh->do(q|DELETE FROM message_queue|); >-t::lib::Mocks::mock_preference('AutoEmailPrimaryAddress', 'email|OFF|emailpro'); # This is no consistent. OFF should be alone. >+t::lib::Mocks::mock_preference('AutoEmailPrimaryAddress', 'email,OFF,emailpro'); # This is no consistent. OFF should be alone. > is( C4::Letters::EnqueueLetter($my_message), 1, 'message successfully queued' ); > $messages = C4::Letters::GetQueuedMessages({ borrowernumber => $borrowernumber }); > is( scalar( @$messages ), 1, 'The message has been queued for the expected patron' ); >-is( $messages->[0]{to_address}, $email, 'AutoEmailPrimaryAddress=email|OFF|emailpro, EnqueueLetter used the patron email address if no to_address is given in parameter' ); >+is( $messages->[0]{to_address}, $email, 'AutoEmailPrimaryAddress=email,OFF,emailpro, EnqueueLetter used the patron email address if no to_address is given in parameter' ); > > $dbh->do(q|DELETE FROM message_queue|); >-t::lib::Mocks::mock_preference('AutoEmailPrimaryAddress', 'email|emailpro'); >+t::lib::Mocks::mock_preference('AutoEmailPrimaryAddress', 'email,emailpro'); > is( C4::Letters::EnqueueLetter($my_message), 2, 'messages successfully queued' ); > $messages = C4::Letters::GetQueuedMessages({ borrowernumber => $borrowernumber }); > is( scalar( @$messages ), 2, 'The messages have been queued for the expected patron' ); >-is( $messages->[0]{to_address}, $email, 'AutoEmailPrimaryAddress=email|emailpro, EnqueueLetter used the patron email address for the first letter' ); >-is( $messages->[1]{to_address}, $emailpro, 'AutoEmailPrimaryAddress=email|emailpro, EnqueueLetter used the patron emailpro address for the second letter' ); >+is( $messages->[0]{to_address}, $email, 'AutoEmailPrimaryAddress=email,emailpro, EnqueueLetter used the patron email address for the first letter' ); >+is( $messages->[1]{to_address}, $emailpro, 'AutoEmailPrimaryAddress=email,emailpro, EnqueueLetter used the patron emailpro address for the second letter' ); > > { > use C4::Serials; >-- >1.7.10.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 12802
:
31058
|
31059
|
31132
|
31382
|
31383
|
31384
|
31833
|
33864
|
33865
|
33866
|
33867
|
35890
|
35891
|
35892
|
35893
|
35926
|
35927
|
35928
|
35929
|
35930
|
36055
|
36056
|
36057
|
36058
|
36059
|
39762
|
39770
|
40167
|
45476
|
45477
|
45478
|
45479
|
45480
|
45481
|
45482
|
57296
|
61878
|
61879
|
61880
|
61881
|
61882
|
61883
|
61884
|
66934
|
66935
|
66936
|
66937
|
66938
|
66939
|
66940
|
68002
|
68003
|
68004
|
68005
|
68006
|
68007
|
68008
|
68009
|
68010
|
68011
|
68012
|
68013
|
68375
|
68376
|
68377
|
68378
|
68379
|
68380
|
119016
|
149232
|
153540
|
154076
|
159780
|
159857
|
160051
|
160052
|
160090
|
160091
|
160092
|
160103
|
160104
|
160105
|
160106
|
165494
|
165495
|
165496
|
165497