0
@@ -32,6 +32,8 @@
0
     # Loads ActionMailer settings from a Capistrano variable called "cap_gun_action_mailer_config"
0
     def load_mailer_config(cap) 
0
       raise ArgumentError, "You must define ActionMailer settings in 'cap_gun_action_mailer_config'" unless cap.cap_gun_action_mailer_config 
0
+      raise ArgumentError, "Need at least one recipient." if !cap.exists?(:cap_gun_options) || cap.cap_gun_options[:recipients].blank?
0
+      
0
       ActionMailer::Base.smtp_settings = cap.cap_gun_action_mailer_config
0
     end
0
   

0
@@ -23,10 +23,23 @@
0
     end
0
   
0
     it "gets action mailer config from capistrano" do
0
-      capistrano = stub(:cap_gun_action_mailer_config => {:account => "foo@gmail.com", :password => "password"}"})
0
+      capistrano = stub(:cap_gun_action_mailer_config => {:account => "foo@gmail.com", :password => "password"}, :exists? => true, :cap_gun_options => {:recipients => "foo"})
0
       CapGun::Helper.load_mailer_config(capistrano)
0
       ActionMailer::Base.smtp_settings.should == {:account => "foo@gmail.com", :password => "password"}
0
     end
0
+    
0
+    it "raises if we have no cap gun options" do
0
+      capistrano = stub_everything(:cap_gun_action_mailer_config => {}, :exists? => false)
0
+      lambda { CapGun::Helper.load_mailer_config capistrano }.should.raise(ArgumentError)
0
+    end
0
+    
0
+    it "raises if we dont have at least one recipient" do
0
+      capistrano = stub_everything(:cap_gun_action_mailer_config => {}, :cap_gun_options => {})
0
+      lambda { CapGun::Helper.load_mailer_config capistrano }.should.raise(ArgumentError)
0
+      capistrano = stub_everything(:cap_gun_action_mailer_config => {}, :cap_gun_options => {:recipients => []})
0
+      lambda { CapGun::Helper.load_mailer_config capistrano }.should.raise(ArgumentError)
0
+    end
0
+    
0
   end
0
   
0
   describe "misc helpers" do
0
0
@@ -53,11 +66,8 @@
0
   end
0
   
0
   describe "Mailer" do
0
-    it "raises if we dont have at least one recipient" do
0
-    end
0
-    
0
     it "passes capistrano into create body" do
0
-      capistrano = { :current_release => "/data/foo", :previous_release => "/data/foo", :cap_gun_options => {:recipients => "joe@example.com"} }
0
+      capistrano = { :current_release => "/data/foo", :previous_release => "/data/foo", :cap_gun_options => {:recipients => ["joe@example.com"]} }
0
       CapGun::Mailer.any_instance.expects(:create_body).with(capistrano).returns("foo")
0
       CapGun::Mailer.create_deployment_notification capistrano
0
       

0
@@ -1,5 +1,5 @@
0
 # desc "Explaining what the task does"
0
-# task :cap_bot do
0
+# task :cap_gun do
0
 #   # Task goes here
0
 # end