From ae55978515a48d104a533551583ec5e9e9f244a2 Mon Sep 17 00:00:00 2001
From: Jean-Philippe Lang <jp_lang@yahoo.fr>
Date: Sun, 28 Feb 2010 17:18:21 +0000
Subject: [PATCH] Adds a test for not sending email without recipient (#4920).

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@3518 e93f8b46-1217-0410-a6f0-8f06a7374b81
---
 test/unit/mailer_test.rb | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/test/unit/mailer_test.rb b/test/unit/mailer_test.rb
index 24de2067..8850f823 100644
--- a/test/unit/mailer_test.rb
+++ b/test/unit/mailer_test.rb
@@ -134,6 +134,27 @@ class MailerTest < ActiveSupport::TestCase
     assert_not_nil mail
     assert_equal 'Redmine app', mail.from_addrs.first.name
   end
+  
+  def test_should_not_send_email_without_recipient
+    news = News.find(:first)
+    user = news.author
+    # Remove members except news author
+    news.project.memberships.each {|m| m.destroy unless m.user == user}
+    
+    user.pref[:no_self_notified] = false
+    user.pref.save
+    User.current = user
+    Mailer.deliver_news_added(news.reload)
+    assert_equal 1, last_email.bcc.size
+
+    # nobody to notify
+    user.pref[:no_self_notified] = true
+    user.pref.save
+    User.current = user
+    ActionMailer::Base.deliveries.clear
+    Mailer.deliver_news_added(news.reload)
+    assert ActionMailer::Base.deliveries.empty?
+  end
 
   def test_issue_add_message_id
     issue = Issue.find(1)