Don't require secrets in the test environment

Author: reidab

app/models/speaker_mailer.rb

@@ -7,7 +7,7 @@ class SpeakerMailer < ActionMailer::Base
   include ActionView::Helpers::SanitizeHelper
 
   def self.configured?
-    if SECRETS.email.blank? || SECRETS.email['action_mailer'].blank? || SECRETS.email['default_from_address'] == 'test'
+    if !Rails.env.test? && (SECRETS.email.blank? || SECRETS.email['action_mailer'].blank? || SECRETS.email['default_from_address'] == 'test')
       return false
     else
       return true

spec/models/speaker_mailer_spec.rb

@@ -25,7 +25,7 @@ def deliver_email(proposal)
 
   context "when sending email" do
     it "should raise error if speaker_mailer is not configured" do
-      SECRETS.stub(:email).and_return({})
+      SpeakerMailer.stub(:configured?).and_return(false)
       lambda { deliver_email(@proposal) }.should raise_error(ArgumentError)
     end