aboutsummaryrefslogtreecommitdiffstats
path: root/actionmailbox/test/unit/inbound_email_test.rb
diff options
context:
space:
mode:
authorPratik Naik <pratiknaik@gmail.com>2019-01-17 11:13:40 -0600
committerPratik Naik <pratiknaik@gmail.com>2019-01-17 11:13:40 -0600
commit5cd733a334846669d6435517f7d9913c4a9b1eb1 (patch)
treeb79c74e9f818c53c8b2a800ef500d1334f3b574f /actionmailbox/test/unit/inbound_email_test.rb
parent2dee59fed1e78b983aed4db53dc8fc59e49b9200 (diff)
downloadrails-5cd733a334846669d6435517f7d9913c4a9b1eb1.tar.gz
rails-5cd733a334846669d6435517f7d9913c4a9b1eb1.tar.bz2
rails-5cd733a334846669d6435517f7d9913c4a9b1eb1.zip
Ensure Action Mailbox processes an email only once when received multiple times
This also adds a new column, message_checksum, to the action_mailbox_inbound_emails table for storing SHA1 digest of the email source. Additionally, it makes generating the missing message id deterministic and adds a unique index on message_checksum and message_id to detect duplicate emails.
Diffstat (limited to 'actionmailbox/test/unit/inbound_email_test.rb')
-rw-r--r--actionmailbox/test/unit/inbound_email_test.rb22
1 files changed, 22 insertions, 0 deletions
diff --git a/actionmailbox/test/unit/inbound_email_test.rb b/actionmailbox/test/unit/inbound_email_test.rb
index 993423406f..76f047eb61 100644
--- a/actionmailbox/test/unit/inbound_email_test.rb
+++ b/actionmailbox/test/unit/inbound_email_test.rb
@@ -11,5 +11,27 @@ module ActionMailbox
test "source returns the contents of the raw email" do
assert_equal file_fixture("welcome.eml").read, create_inbound_email_from_fixture("welcome.eml").source
end
+
+ test "email with message id is processed only once when received multiple times" do
+ mail = Mail.from_source(file_fixture("welcome.eml").read)
+ assert mail.message_id
+
+ inbound_email_1 = create_inbound_email_from_source(mail.to_s)
+ assert inbound_email_1
+
+ inbound_email_2 = create_inbound_email_from_source(mail.to_s)
+ assert_nil inbound_email_2
+ end
+
+ test "email with missing message id is processed only once when received multiple times" do
+ mail = Mail.from_source("Date: Fri, 28 Sep 2018 11:08:55 -0700\r\nTo: a@example.com\r\nMime-Version: 1.0\r\nContent-Type: text/plain\r\nContent-Transfer-Encoding: 7bit\r\n\r\nHello!")
+ assert_nil mail.message_id
+
+ inbound_email_1 = create_inbound_email_from_source(mail.to_s)
+ assert inbound_email_1
+
+ inbound_email_2 = create_inbound_email_from_source(mail.to_s)
+ assert_nil inbound_email_2
+ end
end
end