spec/email_message_spec.rb in govdelivery-tms-0.8.0 vs spec/email_message_spec.rb in govdelivery-tms-0.8.1

@@ -1,101 +1,101 @@
 require 'spec_helper'
 
-describe TMS::EmailMessage do
+describe GovDelivery::TMS::EmailMessage do
   context "creating a new message" do
     let(:client) do
       double('client')
     end
     before do
-      @message = TMS::EmailMessage.new(client, '/messages/email', {
-        :body       => '12345678',
-        :subject    => 'blah',
-        :created_at => 'BAAAAAD',
-        :from_email => '[e-mail]',
-        :errors_to  => '[e-mail]',
-        :reply_to   => '[e-mail]'})
+      @message = GovDelivery::TMS::EmailMessage.new(client, '/messages/email', {
+        body:       '12345678',
+        subject:    'blah',
+        created_at: 'BAAAAAD',
+        from_email: '[e-mail]',
+        errors_to:  '[e-mail]',
+        reply_to:   '[e-mail]'})
     end
     it 'should not render readonly attrs in json hash' do
-      @message.to_json[:body].should == '12345678'
-      @message.to_json[:created_at].should == nil
+      expect(@message.to_json[:body]).to eq('12345678')
+      expect(@message.to_json[:created_at]).to eq(nil)
     end
     it 'should initialize with attrs and collections' do
-      @message.body.should             == '12345678'
-      @message.subject.should          == 'blah'
-      @message.from_email.should       == '[e-mail]'
-      @message.reply_to.should         == '[e-mail]'
-      @message.errors_to.should        == '[e-mail]'
-      @message.recipients.class.should == TMS::EmailRecipients
+      expect(@message.body).to             eq('12345678')
+      expect(@message.subject).to          eq('blah')
+      expect(@message.from_email).to       eq('[e-mail]')
+      expect(@message.reply_to).to         eq('[e-mail]')
+      expect(@message.errors_to).to        eq('[e-mail]')
+      expect(@message.recipients.class).to eq(GovDelivery::TMS::EmailRecipients)
     end
     it 'should post successfully' do
       response = {
-          :body       => 'processed',
-          :subject    => 'blah',
-          :from_email => '[e-mail]',
-          :errors_to  => '[e-mail]',
-          :reply_to   => '[e-mail]',
-          :recipients => [{:email => '[e-mail]'}],
-          :failed => [{:email => '[e-mail]'}],
-          :sent => [{:email => '[e-mail]'}],
-          :created_at => 'time'
+          body:       'processed',
+          subject:    'blah',
+          from_email: '[e-mail]',
+          errors_to:  '[e-mail]',
+          reply_to:   '[e-mail]',
+          recipients: [{email: '[e-mail]'}],
+          failed: [{email: '[e-mail]'}],
+          sent: [{email: '[e-mail]'}],
+          created_at: 'time'
       }
-      @message.client.should_receive('post').with(@message).and_return(double('response', :status => 201, :body => response))
+      expect(@message.client).to receive('post').with(@message).and_return(double('response', status: 201, body: response))
       @message.post
-      @message.body.should                              == 'processed'
-      @message.created_at.should                        == 'time'
-      @message.from_email.should                        == '[e-mail]'
-      @message.reply_to.should                          == '[e-mail]'
-      @message.errors_to.should                         == '[e-mail]'
-      @message.recipients.class.should                  == TMS::EmailRecipients
-      @message.recipients.collection.first.class.should == TMS::EmailRecipient
-      @message.sent.class.should == TMS::EmailRecipients
-      @message.sent.collection.first.class.should == TMS::EmailRecipient
-      @message.failed.class.should == TMS::EmailRecipients
-      @message.failed.collection.first.class.should == TMS::EmailRecipient
+      expect(@message.body).to                              eq('processed')
+      expect(@message.created_at).to                        eq('time')
+      expect(@message.from_email).to                        eq('[e-mail]')
+      expect(@message.reply_to).to                          eq('[e-mail]')
+      expect(@message.errors_to).to                         eq('[e-mail]')
+      expect(@message.recipients.class).to                  eq(GovDelivery::TMS::EmailRecipients)
+      expect(@message.recipients.collection.first.class).to eq(GovDelivery::TMS::EmailRecipient)
+      expect(@message.sent.class).to eq(GovDelivery::TMS::EmailRecipients)
+      expect(@message.sent.collection.first.class).to eq(GovDelivery::TMS::EmailRecipient)
+      expect(@message.failed.class).to eq(GovDelivery::TMS::EmailRecipients)
+      expect(@message.failed.collection.first.class).to eq(GovDelivery::TMS::EmailRecipient)
     end
     it 'should handle errors' do
-      response = {'errors' => {:body => "can't be nil"}}
-      @message.client.should_receive('post').with(@message).and_return(double('response', :status => 422, :body => response))
+      response = {'errors' => {body: "can't be nil"}}
+      expect(@message.client).to receive('post').with(@message).and_return(double('response', status: 422, body: response))
       @message.post
-      @message.body.should == '12345678'
-      @message.errors.should == {:body => "can't be nil"}
+      expect(@message.body).to eq('12345678')
+      expect(@message.errors).to eq({body: "can't be nil"})
     end
 
     it 'should handle 401 errors' do
-      @message.client.should_receive('post').with(@message).and_return(double('response', :status => 401))
+      expect(@message.client).to receive('post').with(@message).and_return(double('response', status: 401))
       expect {@message.post}.to raise_error(StandardError, "401 Not Authorized")
     end
 
     it 'should handle 404 errors' do
-      @message.client.should_receive('post').with(@message).and_return(double('response', :status => 404))
+      expect(@message.client).to receive('post').with(@message).and_return(double('response', status: 404))
       expect {@message.post}.to raise_error(StandardError, "Can't POST to /messages/email")
     end
   end
 
   context 'an existing message' do
     let(:client) do
       double('client')
     end
     before do
       # blank hash prevents the client from doing a GET in the initialize method
-      @message = TMS::EmailMessage.new(client, '/messages/99', {})
+      @message = GovDelivery::TMS::EmailMessage.new(client, '/messages/99', {})
     end
     it 'should GET cleanly' do
-      response = {:body => 'processed',
-                  :subject    => 'hey',
-                  :from_email => '[e-mail]',
-                  :errors_to  => '[e-mail]',
-                  :reply_to   => '[e-mail]',
-                  :recipients => [{:email => '[e-mail]'}],
-                  :created_at => 'time'}
-      @message.client.should_receive('get').with(@message.href).and_return(double('response', :status => 200, :body => response))
+      response = {body: 'processed',
+                  subject:    'hey',
+                  from_email: '[e-mail]',
+                  errors_to:  '[e-mail]',
+                  reply_to:   '[e-mail]',
+                  recipients: [{email: '[e-mail]'}],
+                  created_at: 'time'}
+      expect(@message.client).to receive('get').with(@message.href).and_return(double('response', status: 200, body: response))
       @message.get
-      @message.body.should       == 'processed'
-      @message.subject.should    == 'hey'
-      @message.from_email.should == '[e-mail]'
-      @message.reply_to.should   == '[e-mail]'
-      @message.errors_to.should  == '[e-mail]'
-      @message.created_at.should == 'time'
+      expect(@message.body).to       eq('processed')
+      expect(@message.subject).to    eq('hey')
+      expect(@message.from_email).to eq('[e-mail]')
+      expect(@message.reply_to).to   eq('[e-mail]')
+      expect(@message.errors_to).to  eq('[e-mail]')
+      expect(@message.created_at).to eq('time')
     end
   end
 
 
 end