FIX: allow site_name to be present in rejection email subject

This commit is contained in:
Arpit Jalan 2015-05-04 11:08:17 +05:30
parent 4dfc1907ac
commit d9ec6888e1
2 changed files with 30 additions and 1 deletions

View File

@ -3,7 +3,7 @@ require_dependency 'email/message_builder'
class RejectionMailer < ActionMailer::Base
include Email::BuildEmailHelper
DISALLOWED_TEMPLATE_ARGS = [:to, :from, :site_name, :base_url,
DISALLOWED_TEMPLATE_ARGS = [:to, :from, :base_url,
:user_preferences_url,
:include_respond_instructions, :html_override,
:add_unsubscribe_link, :respond_instructions,

View File

@ -0,0 +1,29 @@
require "spec_helper"
describe RejectionMailer do
describe "send_rejection" do
context 'sends rejection email' do
let (:user) { Fabricate(:user) }
let (:template_args) { {former_title: "Mail Subject", destination: user.email, site_name: SiteSetting.title} }
let (:reject_mail) { RejectionMailer.send_rejection("email_reject_topic_not_found", user.email, template_args) }
it 'renders the senders email' do
expect(reject_mail.to).to eql([user.email])
end
it 'renders the subject' do
expect(reject_mail.subject).to be_present
end
it 'renders site title in subject' do
expect(reject_mail.subject).to match(SiteSetting.title)
end
it 'renders the body' do
expect(reject_mail.body).to be_present
end
end
end
end