summaryrefslogtreecommitdiff
path: root/spec/mailers
diff options
context:
space:
mode:
authorPierre de La Morinerie <pierre@capitainetrain.com>2014-02-21 19:39:09 +0100
committerPierre de La Morinerie <pierre@capitainetrain.com>2014-06-10 17:09:15 +0200
commit466b768bb34730ee6a24d950333c232009c34bbd (patch)
tree7ee2e0263b60e495c4c5013376d3564a92811f22 /spec/mailers
parent772f2f1ac8aab40a2cd62f285af131eb61ad12bb (diff)
downloadgitlab-ce-466b768bb34730ee6a24d950333c232009c34bbd.tar.gz
Send notification emails to the "project", and put people in Cc
This fixes email threading in Mail.app, that doesn't like when a thread doesn't have stable recipients. For instance, here is a possible sender-recipient combinations before: From: A To: Me New issue From: B To: Me Reply on new issue From: A To: Me Another reply Mail.app doesn't see B as a participant to the original email thread, and decides to break the thread: it will group all messages from A together, and separately all messages from B. This commit makes the thread look like this: From: A To: gitlab/project Cc: Me New issue From: B To: gitlab/project Cc: Me Reply on new issue From: A To: gitlab/project Cc: Me Another reply Mail.app sees a common recipient, and group the thread correctly.
Diffstat (limited to 'spec/mailers')
-rw-r--r--spec/mailers/notify_spec.rb8
1 files changed, 4 insertions, 4 deletions
diff --git a/spec/mailers/notify_spec.rb b/spec/mailers/notify_spec.rb
index 547268d44f0..6725e350898 100644
--- a/spec/mailers/notify_spec.rb
+++ b/spec/mailers/notify_spec.rb
@@ -10,7 +10,7 @@ describe Notify do
shared_examples 'a multiple recipients email' do
it 'is sent to the given recipient' do
- should deliver_to recipient.email
+ should cc_to recipient.email
end
end
@@ -141,7 +141,7 @@ describe Notify do
end
it 'is sent to the assignee' do
- should deliver_to assignee.email
+ should cc_to assignee.email
end
end
@@ -394,7 +394,7 @@ describe Notify do
end
it 'is sent to the given recipient' do
- should deliver_to recipient.email
+ should cc_to recipient.email
end
it 'contains the message from the note' do
@@ -538,7 +538,7 @@ describe Notify do
end
it 'is sent to recipient' do
- should deliver_to 'devs@company.name'
+ should cc_to 'devs@company.name'
end
it 'has the correct subject' do