diff options
author | claudiob <claudiob@gmail.com> | 2015-01-11 11:50:29 -0800 |
---|---|---|
committer | claudiob <claudiob@gmail.com> | 2015-01-11 11:50:29 -0800 |
commit | e4613c97c3760c5298ab19ec5120f93ac83f03b2 (patch) | |
tree | 29b4335a2e62f6042bcc84408f117cbfd127dd63 /actionmailer/test | |
parent | 7b47f422cf367db4918be76c417d9db592a04adf (diff) | |
download | rails-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.rb | 2 |
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 |