multi_mail 0.1.0 → 0.1.1

data/spec/mandrill/message_spec.rb

@@ -1,5 +1,4 @@
 require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
-require 'multi_mail/mandrill/message'
 
 describe MultiMail::Message::Mandrill do
   let :message do
@@ -121,6 +120,19 @@ describe MultiMail::Message::Mandrill do
     MultiMail::Message::Mandrill.new
   end
 
+  let :message_with_one_tag do
+    MultiMail::Message::Mandrill.new do
+      tag 'foo'
+    end
+  end
+
+  let :message_with_many_tags do
+    MultiMail::Message::Mandrill.new do
+      tag 'foo'
+      tag 'bar'
+    end
+  end
+
   describe '#mandrill_to' do
     it 'should return the recipients with names' do
       message.mandrill_to.should == [
@@ -301,5 +313,17 @@ describe MultiMail::Message::Mandrill do
     it 'should convert an empty message' do
       empty_message.to_mandrill_hash.should == {}
     end
+
+    it 'should convert the message with one tag' do
+      message_with_one_tag.to_mandrill_hash.should == {
+        :tags => ['foo'],
+      }
+    end
+
+    it 'should convert the message with many tags' do
+      message_with_many_tags.to_mandrill_hash.should == {
+        :tags => ['foo', 'bar'],
+      }
+    end
   end
 end

data/spec/mandrill/receiver_spec.rb

@@ -2,106 +2,104 @@ require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 require 'multi_mail/mandrill/receiver'
 
 describe MultiMail::Receiver::Mandrill do
-  context 'after initialization' do
-    def params(fixture)
-      MultiMail::Receiver::Mandrill.parse(response('mandrill', fixture))
+  def params(fixture)
+    MultiMail::Receiver::Mandrill.parse(response('mandrill', fixture))
+  end
+
+  context 'without optional arguments' do
+    let :service do
+      MultiMail::Receiver.new(:provider => :mandrill)
     end
 
-    context 'without optional arguments' do
-      let :service do
-        MultiMail::Receiver.new(:provider => :mandrill)
+    describe '#valid?' do
+      it 'should return true if the response is valid' do
+        service.valid?(params('valid')).should == true
       end
 
-      describe '#valid?' do
-        it 'should return true if the response is valid' do
-          service.valid?(params('valid')).should == true
-        end
-
-        it 'should return true if the response is invalid' do
-          service.valid?(params('invalid')).should == true
-        end
+      it 'should return true if the response is invalid' do
+        service.valid?(params('invalid')).should == true
+      end
 
-        it 'should return true if parameters are missing' do
-          service.valid?(params('missing')).should == true
-        end
+      it 'should return true if parameters are missing' do
+        service.valid?(params('missing')).should == true
       end
     end
+  end
+
+  context 'with optional arguments' do
+    let :service do
+      MultiMail::Receiver.new({
+        :provider => :mandrill,
+        :mandrill_webhook_key => 'rth_rywL9CWIIZBuwPQIWw',
+        :mandrill_webhook_url => 'http://rackbin.herokuapp.com/',
+      })
+    end
 
-    context 'with optional arguments' do
-      let :service do
-        MultiMail::Receiver.new({
-          :provider => :mandrill,
-          :mandrill_webhook_key => 'rth_rywL9CWIIZBuwPQIWw',
-          :mandrill_webhook_url => 'http://rackbin.herokuapp.com/',
-        })
+    describe '#valid?' do
+      it 'should return true if the response is valid' do
+        service.valid?(params('valid')).should == true
       end
 
-      describe '#valid?' do
-        it 'should return true if the response is valid' do
-          service.valid?(params('valid')).should == true
-        end
+      it 'should return false if the response is invalid' do
+        service.valid?(params('invalid')).should == false
+      end
 
-        it 'should return false if the response is invalid' do
-          service.valid?(params('invalid')).should == false
-        end
+      it 'should raise an error if parameters are missing' do
+        expect{ service.valid?(params('missing')) }.to raise_error(IndexError)
+      end
+    end
 
-        it 'should raise an error if parameters are missing' do
-          expect{ service.valid?(params('missing')) }.to raise_error(IndexError)
-        end
+    describe '#transform' do
+      it 'should return a mail message' do
+        messages = service.transform(params('valid'))
+        messages.size.should == 1
+        message = messages[0]
+
+        # Headers
+        message.date.should    == DateTime.parse('Mon, 29 May 2013 16:51:45 -04:00')
+        message.from.should    == ['james@opennorth.ca']
+        message.to.should      == ['foo+bar@govkit.org']
+        message.subject.should == 'Test'
+
+        # Body
+        message.multipart?.should            == true
+        message.parts.size.should            == 4
+        message.parts[0].content_type.should == 'text/plain'
+        message.parts[1].content_type.should == 'text/html; charset=UTF-8'
+        # @note Mandrill adds a newline at the end of each part.
+        message.parts[0].body.decoded.should == "bold text\n\n\n\nsome more bold text\n\n\n\nsome italic text\n\n> multiline\n> quoted\n> text\n\n\n--\nSignature block\n"
+        message.parts[1].body.decoded.should == %(<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><b>bold text</b><div><br></div><div></div></body></html><html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><head></head><br><div></div><div><br></div><div><b>some more bold text</b></div><div><b><br></b></div><div><b></b></div></body></html><html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><b></b></div><div><b><span class="Apple-style-span" style="font-weight: normal; "><br></span></b></div><div><b><span class="Apple-style-span" style="font-weight: normal; "><i>some italic text</i></span></b></div><div><b><span class="Apple-style-span" style="font-weight: normal; "><br></span></b></div><div><blockquote type="cite">multiline</blockquote><blockquote type="cite">quoted</blockquote><blockquote type="cite">text</blockquote></div><div><br></div><div>--</div><div>Signature block</div></body></html>)
+
+        # Attachments
+        attachment0 = message.attachments.find{|attachment| attachment.filename == 'foo.txt'}
+        attachment1 = message.attachments.find{|attachment| attachment.filename == 'bar.txt'}
+        attachment0.read.should == "Lorem ipsum dolor sit amet, consectetur adipiscing elit.\n\n"
+        attachment1.read.should == "Nam accumsan euismod eros et rhoncus.\n\n"
+
+        # Extra Mandrill parameters
+        message['ts'].value.should == '1369860716'
+        message['email'].value.should == 'foo+bar@govkit.org'
+        message['dkim-signed'].value.should == 'false'
+        message['dkim-valid'].value.should == 'false'
+        message['spam_report-score'].value.should == '-0.7'
+        message['spf-result'].value.should == 'pass'
       end
 
-      describe '#transform' do
-        it 'should return a mail message' do
-          messages = service.transform(params('valid'))
-          messages.size.should == 1
-          message = messages[0]
-
-          # Headers
-          message.date.should    == DateTime.parse('Mon, 29 May 2013 16:51:45 -04:00')
-          message.from.should    == ['james@opennorth.ca']
-          message.to.should      == ['foo+bar@govkit.org']
-          message.subject.should == 'Test'
-
-          # Body
-          message.multipart?.should            == true
-          message.parts.size.should            == 4
-          message.parts[0].content_type.should == 'text/plain'
-          message.parts[1].content_type.should == 'text/html; charset=UTF-8'
-          # @note Mandrill adds a newline at the end of each part.
-          message.parts[0].body.decoded.should == "bold text\n\n\n\nsome more bold text\n\n\n\nsome italic text\n\n> multiline\n> quoted\n> text\n\n\n--\nSignature block\n"
-          message.parts[1].body.decoded.should == %(<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><b>bold text</b><div><br></div><div></div></body></html><html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><head></head><br><div></div><div><br></div><div><b>some more bold text</b></div><div><b><br></b></div><div><b></b></div></body></html><html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><b></b></div><div><b><span class="Apple-style-span" style="font-weight: normal; "><br></span></b></div><div><b><span class="Apple-style-span" style="font-weight: normal; "><i>some italic text</i></span></b></div><div><b><span class="Apple-style-span" style="font-weight: normal; "><br></span></b></div><div><blockquote type="cite">multiline</blockquote><blockquote type="cite">quoted</blockquote><blockquote type="cite">text</blockquote></div><div><br></div><div>--</div><div>Signature block</div></body></html>)
-
-          # Attachments
-          attachment0 = message.attachments.find{|attachment| attachment.filename == 'foo.txt'}
-          attachment1 = message.attachments.find{|attachment| attachment.filename == 'bar.txt'}
-          attachment0.read.should == "Lorem ipsum dolor sit amet, consectetur adipiscing elit.\n\n"
-          attachment1.read.should == "Nam accumsan euismod eros et rhoncus.\n\n"
-
-          # Extra Mandrill parameters
-          message['ts'].value.should == '1369860716'
-          message['email'].value.should == 'foo+bar@govkit.org'
-          message['dkim-signed'].value.should == 'false'
-          message['dkim-valid'].value.should == 'false'
-          message['spam_report-score'].value.should == '-0.7'
-          message['spf-result'].value.should == 'pass'
-        end
-
-        it 'should return multiple messages if there are multiple events' do
-          messages = service.transform(params('multiple'))
-          messages.size.should == 2
-        end
+      it 'should return multiple messages if there are multiple events' do
+        messages = service.transform(params('multiple'))
+        messages.size.should == 2
       end
+    end
 
-      describe '#spam?' do
-        it 'should return true if the response is spam' do
-          message = service.transform(params('spam'))[0]
-          service.spam?(message).should == true
-        end
+    describe '#spam?' do
+      it 'should return true if the response is spam' do
+        message = service.transform(params('spam'))[0]
+        service.spam?(message).should == true
+      end
 
-        it 'should return false if the response is ham' do
-          message = service.transform(params('valid'))[0]
-          service.spam?(message).should == false
-        end
+      it 'should return false if the response is ham' do
+        message = service.transform(params('valid'))[0]
+        service.spam?(message).should == false
       end
     end
   end

data/spec/mandrill/sender_spec.rb

@@ -1,11 +1,10 @@
 require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
-require 'multi_mail/mandrill/sender'
 
 describe MultiMail::Sender::Mandrill do 
   let :message do
     Mail.new do
       from    'foo@example.com'
-      to      'bar@example.com'
+      to      'bit-bucket@test.smtp.org'
       subject 'test'
       body    'hello'
     end
@@ -19,14 +18,12 @@ describe MultiMail::Sender::Mandrill do
     it 'should raise an error if :api_key is missing' do
       expect{
         message.delivery_method MultiMail::Sender::Mandrill
-        message.deliver # request not sent
       }.to raise_error(ArgumentError, "Missing required arguments: api_key")
     end
 
     it 'should raise an error if :api_key is nil' do
       expect{
         message.delivery_method MultiMail::Sender::Mandrill, :api_key => nil
-        message.deliver # request not sent
       }.to raise_error(ArgumentError, "Missing required arguments: api_key")
     end
 
@@ -37,6 +34,16 @@ describe MultiMail::Sender::Mandrill do
       }.to raise_error(MultiMail::InvalidAPIKey, 'Invalid API key')
     end
 
+    it 'should transform send_at to a string if it is not a string' do
+      sender = MultiMail::Sender::Mandrill.new(:api_user => '', :api_key => '', 'send_at' => Time.at(981203696))
+      sender.send_at.should == '2001-02-03 12:34:56'
+    end
+
+    it 'should not transform send_at to a string if it is a string' do
+      sender = MultiMail::Sender::Mandrill.new(:api_user => '', :api_key => '', 'send_at' => '2001-02-03 12:34:56')
+      sender.send_at.should == '2001-02-03 12:34:56'
+    end
+
     it 'should have default settings' do
       sender = MultiMail::Sender::Mandrill.new(:api_key => '')
 
@@ -126,8 +133,8 @@ describe MultiMail::Sender::Mandrill do
       result.size.should == 4
 
       result['reject_reason'].should == nil
-      result['status'].should == "sent"
-      result['email'].should == "bar@example.com"
+      result['status'].should == 'sent'
+      result['email'].should == 'bit-bucket@test.smtp.org'
       result['_id'].should match(/\A[0-9a-f]{32}\z/)
     end
 

data/spec/message/base_spec.rb

@@ -65,7 +65,7 @@ describe MultiMail::Message::Base do
     end
   end
 
-  describe '#text_part' do
+  describe '#body_text' do
     it 'should return the body if the message is text only' do
       text_message.body_text.should == 'hello'
     end
@@ -78,4 +78,36 @@ describe MultiMail::Message::Base do
       html_and_text_message.body_text.should == 'hello'
     end
   end
+
+  let :many_tags do
+    MultiMail::Message::Base.new do
+      tag 'foo'
+      tag 'bar'
+    end
+  end
+
+  let :one_tag do
+    MultiMail::Message::Base.new do
+      tag 'foo'
+    end
+  end
+
+  let :no_tag do
+    MultiMail::Message::Base.new do
+    end
+  end
+
+  describe '#tags' do
+    it 'should return a multi-value array if many tags are set' do
+      many_tags.tags.should == ['foo', 'bar']
+    end
+
+    it 'should return a single value array if one tag is set' do
+      one_tag.tags.should == ['foo']
+    end
+
+    it 'should return an empty array if no tags are set' do
+      no_tag.tags.should == []
+    end
+  end
 end

data/spec/postmark/message_spec.rb

@@ -0,0 +1,292 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe MultiMail::Message::Postmark do
+  let :message do
+    headers = {
+      'X-Autoreply'  => true,
+      'X-Precedence' => 'auto_reply',
+      'X-Numeric'    => 42,
+      'Delivered-To' => 'Autoresponder',
+    }
+
+    MultiMail::Message::Postmark.new do
+      date     Time.at(946702800)
+      headers  headers
+      from     %("John Doe" <foo@example.com>)
+      to       [%("Jane Doe" <bar@example.com>), '<baz@example.com>']
+      cc       'cc@example.com'
+      bcc      'bcc@example.com'
+      reply_to 'noreply@example.com'
+      subject  'test'
+
+      text_part do
+        body 'hello'
+      end
+
+      html_part do
+        content_type 'text/html; charset=UTF-8'
+        body '<p>hello</p>'
+      end
+
+      add_file  empty_gif_path
+      add_file  :filename => 'foo.txt', :content => 'hello world'
+    end
+  end
+
+  let :message_without_names do
+    MultiMail::Message::Postmark.new do
+      from    'foo@example.com'
+      to      ['bar@example.com', 'baz@example.com']
+      subject 'test'
+      body    'hello'
+    end
+  end
+
+  let :message_with_known_extension do
+    MultiMail::Message::Postmark.new do
+      from    'foo@example.com'
+      to      'bar@example.com'
+      subject 'test'
+      body    'hello'
+      add_file :filename => 'xxx.mov', :content => ''
+    end
+  end
+
+  let :message_with_unknown_extension do
+    MultiMail::Message::Postmark.new do
+      from    'foo@example.com'
+      to      'bar@example.com'
+      subject 'test'
+      body    'hello'
+      add_file :filename => 'xxx.xxx', :content => ''
+    end
+  end
+
+  let :message_without_extension do
+    MultiMail::Message::Postmark.new do
+      from    'foo@example.com'
+      to      'bar@example.com'
+      subject 'test'
+      body    'hello'
+      add_file :filename => 'xxx', :content => ''
+    end
+  end
+
+  let :message_with_empty_file do
+    MultiMail::Message::Postmark.new do
+      from    'foo@example.com'
+      to      'bar@example.com'
+      subject 'test'
+      body    'hello'
+      add_file :filename => '', :content => ''
+    end
+  end
+
+  let :message_with_empty_headers do
+    headers = {
+      'X-Autoreply' => nil,
+    }
+
+    MultiMail::Message::Postmark.new do
+      headers  headers
+      from    'foo@example.com'
+      to      'bar@example.com'
+      reply_to nil
+      subject  'test'
+      body     'hello'
+    end
+  end
+
+  let :message_without_html_body do
+    MultiMail::Message::Postmark.new do
+      from    'foo@example.com'
+      to      'bar@example.com'
+      subject 'test'
+      body    'hello'
+    end
+  end
+
+  let :message_without_text_body do
+    MultiMail::Message::Postmark.new do
+      from    'foo@example.com'
+      to      'bar@example.com'
+      subject 'test'
+      body    '<p>hello</p>'
+      content_type 'text/html; charset=UTF-8'
+    end
+  end
+
+  let :empty_message do
+    MultiMail::Message::Postmark.new
+  end
+
+  let :message_with_one_tag do
+    MultiMail::Message::Postmark.new do
+      tag 'foo'
+    end
+  end
+
+  let :message_with_many_tags do
+    MultiMail::Message::Postmark.new do
+      tag 'foo'
+      tag 'bar'
+    end
+  end
+
+  describe '#postmark_headers' do
+    it 'should return the headers' do
+      headers = message.postmark_headers
+      [ {'Name' => 'X-Autoreply', 'Value' => 'true'},
+        {'Name' => 'X-Precedence', 'Value' => 'auto_reply'},
+        {'Name' => 'X-Numeric', 'Value' => '42'},
+        {'Name' => 'Delivered-To', 'Value' => 'Autoresponder'},
+      ].each do |header|
+        headers.should include(header)
+      end
+    end
+
+    it 'should return empty X-* headers' do
+      message_with_empty_headers.postmark_headers.should == [
+        {'Name' => 'X-Autoreply', 'Value' => ''},
+      ]
+    end
+
+    it 'should return an empty hash' do
+      empty_message.postmark_headers.should == []
+    end
+  end
+
+  describe '#postmark_attachments' do
+    it 'should return the attachments' do
+      message.postmark_attachments.should == [
+        {
+          'ContentType' => 'image/gif; filename=empty.gif',
+          'Name' => 'empty.gif',
+          'Content' => "R0lGODlhAQABAPABAP///wAAACH5BAEKAAAALAAAAAABAAEAAAICRAEAOw==\n",
+          'ContentID' => 'empty.gif',
+        },
+        {
+          'ContentType' => 'text/plain; filename=foo.txt',
+          'Name' => 'foo.txt',
+          'Content' => Base64.encode64('hello world'),
+        },
+      ]
+    end
+
+    it 'should return an attachment with an known extension' do
+      message_with_known_extension.postmark_attachments.should == [
+        {
+          'ContentType' => 'video/quicktime; filename=xxx.mov',
+          'Name' => 'xxx.mov',
+          'Content' => '',
+        },
+      ]
+    end
+
+    it 'should return an attachment with an unknown extension' do
+      message_with_unknown_extension.postmark_attachments.should == [
+        {
+          'ContentType' => 'text/plain',
+          'Name' => 'xxx.xxx',
+          'Content' => '',
+        },
+      ]
+    end
+
+    it 'should return an attachment without an extension' do
+      message_without_extension.postmark_attachments.should == [
+        {
+          'ContentType' => 'text/plain',
+          'Name' => 'xxx',
+          'Content' => '',
+        },
+      ]
+    end
+
+    it 'should return an empty array if the attachment is blank' do
+      message_with_empty_file.postmark_attachments.should == []
+    end
+
+    it 'should return an empty array' do
+      empty_message.postmark_attachments.should == []
+    end
+  end
+
+  describe '#to_postmark_hash' do
+    it 'should return the message as Mandrill parameters' do
+      hash = message.to_postmark_hash
+      hash[:HtmlBody].should    == '<p>hello</p>'
+      hash[:TextBody].should    == 'hello'
+      hash[:Subject].should     == 'test'
+      hash[:From].should        == '"John Doe" <foo@example.com>'
+      hash[:To].should          == '"Jane Doe" <bar@example.com>, <baz@example.com>'
+      hash[:Cc].should          == "cc@example.com"
+      hash[:Bcc].should         == "bcc@example.com"
+      hash[:ReplyTo].should     == "noreply@example.com"
+      hash[:Attachments].should == [
+        {
+          'ContentType' => 'image/gif; filename=empty.gif',
+          'Name' => 'empty.gif',
+          'Content' => "R0lGODlhAQABAPABAP///wAAACH5BAEKAAAALAAAAAABAAEAAAICRAEAOw==\n",
+          'ContentID' => 'empty.gif',
+        },
+        {
+          'ContentType' => 'text/plain; filename=foo.txt',
+          'Name' => 'foo.txt',
+          'Content' => Base64.encode64('hello world'),
+        },
+      ]
+
+      [ {'Name' => 'X-Autoreply', 'Value' => 'true'},
+        {'Name' => 'X-Precedence', 'Value' => 'auto_reply'},
+        {'Name' => 'X-Numeric', 'Value' => '42'},
+        {'Name' => 'Delivered-To', 'Value' => 'Autoresponder'},
+      ].each do |header|
+        hash[:Headers].should include(header)
+      end
+    end
+
+    it 'should return the recipients without names' do
+      message_without_names.to_postmark_hash.should == {
+        :TextBody => 'hello',
+        :Subject  => 'test',
+        :From     => 'foo@example.com',
+        :To       => 'bar@example.com, baz@example.com',
+      }
+    end
+
+    it 'should convert the message without a text body' do
+      message_without_text_body.to_postmark_hash.should == {
+        :HtmlBody => '<p>hello</p>',
+        :Subject  => 'test',
+        :From     => 'foo@example.com',
+        :To       => 'bar@example.com',
+      }
+    end
+
+    it 'should convert the message without an HTML body' do
+      message_without_html_body.to_postmark_hash.should == {
+        :TextBody => 'hello',
+        :Subject  => 'test',
+        :From     => 'foo@example.com',
+        :To       => 'bar@example.com',
+      }
+    end
+
+    it 'should convert an empty message' do
+      empty_message.to_postmark_hash.should == {}
+    end
+
+    it 'should convert the message with one tag' do
+      message_with_one_tag.to_postmark_hash.should == {
+        :Tag => 'foo',
+      }
+    end
+
+    it 'should convert the message with many tags' do
+      message_with_many_tags.to_postmark_hash.should == {
+        :Tag => 'foo',
+      }
+    end
+  end
+end