Skip to content

Commit

Permalink
- Removed mail_interceptor use in tests
Browse files Browse the repository at this point in the history
- Fixed secrets config to remove mail interceptor data for test
- Reverted simplecov coverage method name
  • Loading branch information
vipulnsward committed May 18, 2015
1 parent 91d60a2 commit 49190cc
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 6 deletions.
2 changes: 1 addition & 1 deletion Gemfile
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ gem 'browser'
gem 'haml-rails'

# intercepts outgoing emails in non-production environment
gem 'mail_interceptor', github: 'bigbinary/mail_interceptor', group: [:development, :staging, :test]
gem 'mail_interceptor', github: 'bigbinary/mail_interceptor', group: [:development, :staging]

# Adds prefix to the subject in emails
gem 'email_prefixer'
Expand Down
3 changes: 0 additions & 3 deletions config/secrets.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,9 +50,6 @@ test:
<<: *default
secret_key_base: 08523cd7820d8e6232afb3a704c60a75feb024ab937de989322f4e64a3854b4d8dc8bb119faf8847adceb70aadce1288ff3193ce62de10f78da223e95760dd6c
mailer_delivery_method: :test
intercept_and_forward_emails_to:
- john@example.com
- adam@example.com
mailer:
smtp_settings:
user_name: <%= ENV['MANDRILLAPP_USERNAME'] %>
Expand Down
4 changes: 2 additions & 2 deletions test/test_helper.rb
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
require 'coveralls'
Coveralls.wear!

def enable_local_test_coverage
def enable_test_coverage
require 'simplecov'

SimpleCov.start do
Expand All @@ -17,7 +17,7 @@ def enable_local_test_coverage
end
end

enable_local_test_coverage if ENV['COVERAGE']
enable_test_coverage if ENV['COVERAGE']

ENV["RAILS_ENV"] ||= "test"

Expand Down

0 comments on commit 49190cc

Please sign in to comment.