Merge pull request #34967 from lifo/am-handle-duplicate-emails

Update test schema.rb to reflect the recent changes
rails · Jan 17, 2019 · cc0dd1d · cc0dd1d
2 parents 9d046eb + 57cb8d7
commit cc0dd1d
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 10 deletions.
diff --git a/actionmailbox/app/controllers/rails/conductor/action_mailbox/inbound_emails_controller.rb b/actionmailbox/app/controllers/rails/conductor/action_mailbox/inbound_emails_controller.rb
@@ -28,8 +28,7 @@ def new_mail
       end
 
       def create_inbound_email(mail)
-        ActionMailbox::InboundEmail.create! raw_email: \
-          { io: StringIO.new(mail.to_s), filename: "inbound.eml", content_type: "message/rfc822" }
+        ActionMailbox::InboundEmail.create_and_extract_message_id!(mail.to_s)
       end
   end
 end
diff --git a/actionmailbox/db/migrate/20180917164000_create_action_mailbox_tables.rb b/actionmailbox/db/migrate/20180917164000_create_action_mailbox_tables.rb
@@ -8,7 +8,7 @@ def change
       t.datetime :created_at, precision: 6, null: false
       t.datetime :updated_at, precision: 6, null: false
 
-      t.index [ :message_id, :message_checksum ], unique: true
+      t.index [ :message_id, :message_checksum ], name: "index_action_mailbox_inbound_emails_uniqueness", unique: true
     end
   end
 end
diff --git a/actionmailbox/test/dummy/db/migrate/20180208205311_create_action_mailroom_tables.rb b/actionmailbox/test/dummy/db/migrate/20180208205311_create_action_mailroom_tables.rb
@@ -5,10 +5,10 @@ def change
       t.string  :message_id, null: false
       t.string  :message_checksum, null: false
 
-      t.datetime :created_at, precision: 6
-      t.datetime :updated_at, precision: 6
+      t.datetime :created_at, precision: 6, null: false
+      t.datetime :updated_at, precision: 6, null: false
 
-      t.index [ :message_id, :message_checksum ], unique: true
+      t.index [ :message_id, :message_checksum ], name: "index_action_mailbox_inbound_emails_uniqueness", unique: true
     end
   end
 end
diff --git a/actionmailbox/test/dummy/db/schema.rb b/actionmailbox/test/dummy/db/schema.rb
@@ -14,10 +14,10 @@
 
   create_table "action_mailbox_inbound_emails", force: :cascade do |t|
     t.integer "status", default: 0, null: false
-    t.string "message_id"
-    t.string "message_checksum"
-    t.datetime "created_at", precision: 6
-    t.datetime "updated_at", precision: 6
+    t.string "message_id", null: false
+    t.string "message_checksum", null: false
+    t.datetime "created_at", precision: 6, null: false
+    t.datetime "updated_at", precision: 6, null: false
     t.index ["message_id", "message_checksum"], name: "index_action_mailbox_inbound_emails_uniqueness", unique: true
   end