From bd63d3bc7163edd70c98056768b10c037a4041da Mon Sep 17 00:00:00 2001 From: Toshi MARUYAMA Date: Sun, 24 Nov 2019 16:44:18 +0000 Subject: [PATCH] cleanup: rubocop: fix Lint/ParenthesesAsGroupedExpression in test/functional/email_addresses_controller_test.rb git-svn-id: http://svn.redmine.org/redmine/trunk@19267 e93f8b46-1217-0410-a6f0-8f06a7374b81 --- .rubocop_todo.yml | 1 - test/functional/email_addresses_controller_test.rb | 12 ++++++------ 2 files changed, 6 insertions(+), 7 deletions(-) diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml index df5bd12c7..332554305 100644 --- a/.rubocop_todo.yml +++ b/.rubocop_todo.yml @@ -384,7 +384,6 @@ Lint/Loop: Lint/ParenthesesAsGroupedExpression: Exclude: - - 'test/functional/email_addresses_controller_test.rb' - 'test/functional/my_controller_test.rb' - 'test/functional/settings_controller_test.rb' - 'test/functional/users_controller_test.rb' diff --git a/test/functional/email_addresses_controller_test.rb b/test/functional/email_addresses_controller_test.rb index ac11ec383..08932236f 100644 --- a/test/functional/email_addresses_controller_test.rb +++ b/test/functional/email_addresses_controller_test.rb @@ -127,8 +127,8 @@ class EmailAddressesControllerTest < Redmine::ControllerTest :address => 'something@example.fr' } } - - assert_not_nil (mail = ActionMailer::Base.deliveries.last) + mail = ActionMailer::Base.deliveries.last + assert_not_nil mail assert_mail_body_match '0.0.0.0', mail assert_mail_body_match I18n.t(:mail_body_security_notification_add, field: I18n.t(:field_mail), value: 'something@example.fr'), mail assert_select_email do @@ -179,8 +179,8 @@ class EmailAddressesControllerTest < Redmine::ControllerTest :notify => '0' }, :xhr => true - - assert_not_nil (mail = ActionMailer::Base.deliveries.last) + mail = ActionMailer::Base.deliveries.last + assert_not_nil mail assert_mail_body_match I18n.t(:mail_body_security_notification_notify_disabled, value: 'another@somenet.foo'), mail # The changed address should be notified for security purposes @@ -237,8 +237,8 @@ class EmailAddressesControllerTest < Redmine::ControllerTest :id => email.id }, :xhr => true - - assert_not_nil (mail = ActionMailer::Base.deliveries.last) + mail = ActionMailer::Base.deliveries.last + assert_not_nil mail assert_mail_body_match I18n.t(:mail_body_security_notification_remove, field: I18n.t(:field_mail), value: 'another@somenet.foo'), mail # The removed address should be notified for security purposes -- 2.39.5