Receive e-mail replies to news and news comments (#38274).

Patch by Felix Schäfer.


git-svn-id: https://svn.redmine.org/redmine/trunk@22160 e93f8b46-1217-0410-a6f0-8f06a7374b81

3 files changed, 65 insertions, 1 deletions

diff --git a/test/fixtures/mail_handler/news_comment_reply.eml b/test/fixtures/mail_handler/news_comment_reply.eml
new file mode 100644
index 000000000..2ce1660f9
--- /dev/null
+++ b/test/fixtures/mail_handler/news_comment_reply.eml
@@ -0,0 +1,15 @@
+Message-ID: <4974C93E.3071105@somenet.foo>
+Date: Mon, 19 Jan 2023 19:41:02 +0100
+From: "John Smith" <jsmith@somenet.foo>
+User-Agent: Thunderbird 2.0.0.19 (Windows/20081209)
+MIME-Version: 1.0
+To: redmine@somenet.foo
+Subject: News comment reply via email
+References: <redmine.comment-1.20230214171800@somenet.foo>
+In-Reply-To: <redmine.comment-1.20230214171800@somenet.foo>
+Content-Type: text/plain; charset=UTF-8; format=flowed
+Content-Transfer-Encoding: 7bit
+
+This is a reply to a comment.
+
+
diff --git a/test/fixtures/mail_handler/news_reply.eml b/test/fixtures/mail_handler/news_reply.eml
new file mode 100644
index 000000000..880ebf30f
--- /dev/null
+++ b/test/fixtures/mail_handler/news_reply.eml
@@ -0,0 +1,15 @@
+Message-ID: <4974C93E.3071005@somenet.foo>
+Date: Mon, 19 Jan 2023 19:41:02 +0100
+From: "John Smith" <jsmith@somenet.foo>
+User-Agent: Thunderbird 2.0.0.19 (Windows/20081209)
+MIME-Version: 1.0
+To: redmine@somenet.foo
+Subject: News comment via email
+References: <redmine.news-1.20230214171800@somenet.foo>
+In-Reply-To: <redmine.news-1.20230214171800@somenet.foo>
+Content-Type: text/plain; charset=UTF-8; format=flowed
+Content-Transfer-Encoding: 7bit
+
+This is a reply to a news.
+
+
diff --git a/test/unit/mail_handler_test.rb b/test/unit/mail_handler_test.rb
index a038d59b1..c4771f556 100644
--- a/test/unit/mail_handler_test.rb
+++ b/test/unit/mail_handler_test.rb
@@ -28,7 +28,7 @@ class MailHandlerTest < ActiveSupport::TestCase
            :workflows, :trackers, :projects_trackers,
            :versions, :enumerations, :issue_categories,
            :custom_fields, :custom_fields_trackers, :custom_fields_projects, :custom_values,
-           :boards, :messages, :watchers
+           :boards, :messages, :watchers, :news, :comments
 
   FIXTURES_PATH = File.dirname(__FILE__) + '/../fixtures/mail_handler'
 
@@ -1174,6 +1174,40 @@ class MailHandlerTest < ActiveSupport::TestCase
     end
   end
 
+  def test_reply_to_a_news
+    m = submit_email('news_reply.eml')
+    assert m.is_a?(Comment)
+    assert !m.new_record?
+    m.reload
+    assert_equal News.find(1), m.commented
+    assert_equal "This is a reply to a news.", m.content
+  end
+
+  def test_reply_to_a_news_comment
+    m = submit_email('news_comment_reply.eml')
+    assert m.is_a?(Comment)
+    assert !m.new_record?
+    m.reload
+    assert_equal News.find(1), m.commented
+    assert_equal "This is a reply to a comment.", m.content
+  end
+
+  def test_reply_to_a_nonexistant_news
+    News.find(1).destroy
+    assert_no_difference('Comment.count') do
+      assert_not submit_email('news_reply.eml')
+      assert_not submit_email('news_comment_reply.eml')
+    end
+  end
+
+  def test_reply_to_a_news_without_permission
+    Role.all.each {|r| r.remove_permission! :comment_news}
+    assert_no_difference('Comment.count') do
+      assert_not submit_email('news_reply.eml')
+      assert_not submit_email('news_comment_reply.eml')
+    end
+  end
+
   def test_should_convert_tags_of_html_only_emails
     with_settings :text_formatting => 'textile' do
       issue = submit_email('ticket_html_only.eml', :issue => {:project => 'ecookbook'})