summaryrefslogtreecommitdiffstats
path: root/test/unit/mailer_test.rb
diff options
context:
space:
mode:
Diffstat (limited to 'test/unit/mailer_test.rb')
-rw-r--r--test/unit/mailer_test.rb33
1 files changed, 19 insertions, 14 deletions
diff --git a/test/unit/mailer_test.rb b/test/unit/mailer_test.rb
index b020e5303..1c732674b 100644
--- a/test/unit/mailer_test.rb
+++ b/test/unit/mailer_test.rb
@@ -19,7 +19,7 @@ require File.expand_path('../../test_helper', __FILE__)
class MailerTest < ActiveSupport::TestCase
include Redmine::I18n
- include ActionDispatch::Assertions::SelectorAssertions
+ include Rails::Dom::Testing::Assertions
fixtures :projects, :enabled_modules, :issues, :users, :email_addresses, :members,
:member_roles, :roles, :documents, :attachments, :news,
:tokens, :journals, :journal_details, :changesets,
@@ -61,11 +61,14 @@ class MailerTest < ActiveSupport::TestCase
# link to a changeset
assert_select 'a[href=?][title=?]',
'https://mydomain.foo/projects/ecookbook/repository/revisions/2',
- 'This commit fixes #1, #2 and references #1 &amp; #3',
+ 'This commit fixes #1, #2 and references #1 & #3',
:text => 'r2'
# link to a description diff
- assert_select 'a[href=?][title=?]',
- 'https://mydomain.foo/journals/diff/3?detail_id=4',
+ assert_select 'a[href^=?][title=?]',
+ # should be https://mydomain.foo/journals/diff/3?detail_id=4
+ # but the Rails 4.2 DOM assertion doesn't handle the ? in the
+ # attribute value
+ 'https://mydomain.foo/journals/diff/3',
'View differences',
:text => 'diff'
# link to an attachment
@@ -100,11 +103,14 @@ class MailerTest < ActiveSupport::TestCase
# link to a changeset
assert_select 'a[href=?][title=?]',
'http://mydomain.foo/rdm/projects/ecookbook/repository/revisions/2',
- 'This commit fixes #1, #2 and references #1 &amp; #3',
+ 'This commit fixes #1, #2 and references #1 & #3',
:text => 'r2'
# link to a description diff
- assert_select 'a[href=?][title=?]',
- 'http://mydomain.foo/rdm/journals/diff/3?detail_id=4',
+ assert_select 'a[href^=?][title=?]',
+ # should be http://mydomain.foo/rdm/journals/diff/3?detail_id=4
+ # but the Rails 4.2 DOM assertion doesn't handle the ? in the
+ # attribute value
+ 'http://mydomain.foo/rdm/journals/diff/3',
'View differences',
:text => 'diff'
# link to an attachment
@@ -150,11 +156,14 @@ class MailerTest < ActiveSupport::TestCase
# link to a changeset
assert_select 'a[href=?][title=?]',
'http://mydomain.foo/rdm/projects/ecookbook/repository/revisions/2',
- 'This commit fixes #1, #2 and references #1 &amp; #3',
+ 'This commit fixes #1, #2 and references #1 & #3',
:text => 'r2'
# link to a description diff
- assert_select 'a[href=?][title=?]',
- 'http://mydomain.foo/rdm/journals/diff/3?detail_id=4',
+ assert_select 'a[href^=?][title=?]',
+ # should be http://mydomain.foo/rdm/journals/diff/3?detail_id=4
+ # but the Rails 4.2 DOM assertion doesn't handle the ? in the
+ # attribute value
+ 'http://mydomain.foo/rdm/journals/diff/3',
'View differences',
:text => 'diff'
# link to an attachment
@@ -762,10 +771,6 @@ class MailerTest < ActiveSupport::TestCase
end
end
- def test_mail_should_return_a_mail_message
- assert_kind_of ::Mail::Message, Mailer.test_email(User.find(1))
- end
-
def test_with_synched_deliveries_should_yield_with_synced_deliveries
ActionMailer::Base.delivery_method = :async_smtp
ActionMailer::Base.async_smtp_settings = {:foo => 'bar'}