cst1/openstreetmap-website
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Don't link to trace pages from failed import emails

Browse source
This commit is contained in:
Anton Khorev 2024-01-11 17:11:09 +03:00
parent 02aa14ffea
commit 51f331e8cc
2 changed files with 12 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

11
test/mailers/user_mailer_test.rb
View file

@ -18,7 +18,7 @@ class UserMailerTest < ActionMailer::TestCase
assert_match(/one two three/, email.html_part.body.to_s)
end
def test_gpx_trace_link
def test_gpx_success_trace_link
trace = create(:trace)
email = UserMailer.gpx_success(trace, 100)
body = Rails::Dom::Testing.html_document_fragment.parse(email.html_part.body)
@ -27,6 +27,15 @@ class UserMailerTest < ActionMailer::TestCase
assert_select body, "a[href='#{url}']", :text => trace.name
end
def test_gpx_failure_no_trace_link
trace = create(:trace)
email = UserMailer.gpx_failure(trace, "some error")
body = Rails::Dom::Testing.html_document_fragment.parse(email.html_part.body)
url = Rails.application.routes.url_helpers.show_trace_url(trace.user, trace, :host => Settings.server_url, :protocol => Settings.server_protocol)
assert_select body, "a[href='#{url}']", :count => 0
end
def test_html_encoding
user = create(:user, :display_name => "Jack & Jill <br>")
message = create(:message, :sender => user)