diff options
author | Yves Senn <yves.senn@gmail.com> | 2012-09-29 22:29:29 +0200 |
---|---|---|
committer | Yves Senn <yves.senn@gmail.com> | 2012-10-28 20:45:43 +0100 |
commit | b786f065d3fc6b0869f7f485042e8667e71ff66d (patch) | |
tree | 703e918d99de02f946898f019a74c1be5eff359e /actionmailer/test | |
parent | 2e44dda27ae8df9301b1a83bfaa74708678bfcd4 (diff) | |
download | rails-b786f065d3fc6b0869f7f485042e8667e71ff66d.tar.gz rails-b786f065d3fc6b0869f7f485042e8667e71ff66d.tar.bz2 rails-b786f065d3fc6b0869f7f485042e8667e71ff66d.zip |
Do not render views when mail() isn't called. (NullMail refactoring)
Diffstat (limited to 'actionmailer/test')
-rw-r--r-- | actionmailer/test/base_test.rb | 6 | ||||
-rw-r--r-- | actionmailer/test/fixtures/base_mailer/without_mail_call.erb | 1 | ||||
-rw-r--r-- | actionmailer/test/mailers/base_mailer.rb | 3 |
3 files changed, 10 insertions, 0 deletions
diff --git a/actionmailer/test/base_test.rb b/actionmailer/test/base_test.rb index 1cb3ce63fe..b30ec2ddc9 100644 --- a/actionmailer/test/base_test.rb +++ b/actionmailer/test/base_test.rb @@ -499,6 +499,12 @@ class BaseTest < ActiveSupport::TestCase end end + test 'the view is not rendered when mail was never called' do + mail = BaseMailer.without_mail_call + assert_equal('', mail.body.to_s.strip) + mail.deliver + end + # Before and After hooks class MyObserver diff --git a/actionmailer/test/fixtures/base_mailer/without_mail_call.erb b/actionmailer/test/fixtures/base_mailer/without_mail_call.erb new file mode 100644 index 0000000000..290379d5fb --- /dev/null +++ b/actionmailer/test/fixtures/base_mailer/without_mail_call.erb @@ -0,0 +1 @@ +<% raise 'the template should not be rendered' %>
\ No newline at end of file diff --git a/actionmailer/test/mailers/base_mailer.rb b/actionmailer/test/mailers/base_mailer.rb index f25d9b9aff..52342bc59e 100644 --- a/actionmailer/test/mailers/base_mailer.rb +++ b/actionmailer/test/mailers/base_mailer.rb @@ -115,4 +115,7 @@ class BaseMailer < ActionMailer::Base def email_with_translations mail body: render("email_with_translations", formats: [:html]) end + + def without_mail_call + end end |