aboutsummaryrefslogtreecommitdiffstats
path: root/actionmailer/test
diff options
context:
space:
mode:
authorclaudiob <claudiob@gmail.com>2015-01-11 11:50:29 -0800
committerclaudiob <claudiob@gmail.com>2015-01-11 11:50:29 -0800
commite4613c97c3760c5298ab19ec5120f93ac83f03b2 (patch)
tree29b4335a2e62f6042bcc84408f117cbfd127dd63 /actionmailer/test
parent7b47f422cf367db4918be76c417d9db592a04adf (diff)
downloadrails-e4613c97c3760c5298ab19ec5120f93ac83f03b2.tar.gz
rails-e4613c97c3760c5298ab19ec5120f93ac83f03b2.tar.bz2
rails-e4613c97c3760c5298ab19ec5120f93ac83f03b2.zip
Fix assertion that was never run
In order to run whether the `welcome` method of the ActionMailer::Base subclass raises an error, `message` must be called, otherwise the method is not executed at all. You could just replace with `def welcome; raise StandardError; end` and you would still see a passing test. This commit fixes the test so the assertion is actually executed, just like any other tests in the file, where `.message` is called.
Diffstat (limited to 'actionmailer/test')
-rw-r--r--actionmailer/test/base_test.rb2
1 files changed, 1 insertions, 1 deletions
diff --git a/actionmailer/test/base_test.rb b/actionmailer/test/base_test.rb
index 5d9eda2555..b218d923e5 100644
--- a/actionmailer/test/base_test.rb
+++ b/actionmailer/test/base_test.rb
@@ -287,7 +287,7 @@ class BaseTest < ActiveSupport::TestCase
end
end
- assert_nothing_raised { LateAttachmentAccessorMailer.welcome }
+ assert_nothing_raised { LateAttachmentAccessorMailer.welcome.message }
end
# Implicit multipart