From 3a3bf19cf8372dc7b4b45067b33c317f8cd84241 Mon Sep 17 00:00:00 2001
From: Rohan Mitchell <rohan@rohanmitchell.com>
Date: Wed, 29 Apr 2015 15:31:42 +1000
Subject: [PATCH] Clean up duplication

---
 spec/mailers/producer_mailer_spec.rb | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/spec/mailers/producer_mailer_spec.rb b/spec/mailers/producer_mailer_spec.rb
index b059a2c85c..98d7171f0d 100644
--- a/spec/mailers/producer_mailer_spec.rb
+++ b/spec/mailers/producer_mailer_spec.rb
@@ -17,9 +17,11 @@ describe ProducerMailer do
     order.save
     order
   end
+  let(:mail) { ActionMailer::Base.deliveries.last }
 
   before do
     ActionMailer::Base.deliveries.clear
+    ProducerMailer.order_cycle_report(s1, order_cycle).deliver
   end
 
   after do
@@ -27,26 +29,22 @@ describe ProducerMailer do
   end
 
   it "should send an email when an order cycle is closed" do
-    ProducerMailer.order_cycle_report(s1, order_cycle).deliver
     ActionMailer::Base.deliveries.count.should == 1
   end
 
   it "sets a reply-to of the enterprise email" do
-    ProducerMailer.order_cycle_report(s1, order_cycle).deliver
-    ActionMailer::Base.deliveries.last.reply_to.should == [s1.email]
+    mail.reply_to.should == [s1.email]
   end
 
   it "cc's the enterprise" do
-    ProducerMailer.order_cycle_report(s1, order_cycle).deliver
-    ActionMailer::Base.deliveries.last.cc.should == [s1.email]
+    mail.cc.should == [s1.email]
   end
 
   it "contains an aggregated list of produce" do
-    ProducerMailer.order_cycle_report(s1, order_cycle).deliver
-    email_body = ActionMailer::Base.deliveries.last.body
+    email_body = mail.body
     email_body.to_s.each_line do |line|
       if line.include? p1.name
-        line.include?('QTY: 2').should == true
+        line.should include 'QTY: 2'
       end
     end
   end