Merge pull request #73 from Serraview/bugfix/message-id-error

Mail expects message_id to be encapsulated within <>

lib/aws/ses/send_email.rb

@@ -101,7 +101,7 @@ def send_raw_email(mail, args = {})
 
         raw_email = build_raw_email(message, args)
         result = request('SendRawEmail', raw_email)
-        message.message_id = "#{result.parsed['SendRawEmailResult']['MessageId']}@#{message_id_domain}"
+        message.message_id = "<#{result.parsed['SendRawEmailResult']['MessageId']}@#{message_id_domain}>"
         result
       end
 

test/send_email_test.rb

@@ -82,6 +82,27 @@ class SendEmailTest < Test::Unit::TestCase
       assert_equal 'xyz-123', result.request_id
     end
 
+    should 'send a raw e-mail with a mail object' do
+      mock_connection(@base, :body => %{
+        <SendRawEmailResponse xmlns="http://ses.amazonaws.com/doc/2010-12-01/">
+          <SendRawEmailResult>
+            <MessageId>abc-123</MessageId>
+          </SendRawEmailResult>
+          <ResponseMetadata>
+            <RequestId>xyz-123</RequestId>
+          </ResponseMetadata>
+        </SendRawEmailResponse>
+      })
+
+      message = Mail.new(@basic_email)
+      result = @base.send_raw_email(message)
+      assert result.success?
+      assert_equal 'abc-123', result.message_id
+      assert_equal 'xyz-123', result.request_id
+      assert message.errors.empty?
+      assert_equal 'abc-123@email.amazonses.com', message.message_id
+    end
+
     context "with a standard email"  do
       setup do
         @message = Mail.new({:from => 'jon@example.com', :to => 'dave@example.com', :cc => 'sally@example.com', :subject => 'Subject1', :body => "test body"})