]> source.dussan.org Git - redmine.git/commitdiff
fix source indent of EmailAddressesControllerTest
authorToshi MARUYAMA <marutosijp2@yahoo.co.jp>
Fri, 6 Nov 2020 12:59:50 +0000 (12:59 +0000)
committerToshi MARUYAMA <marutosijp2@yahoo.co.jp>
Fri, 6 Nov 2020 12:59:50 +0000 (12:59 +0000)
git-svn-id: http://svn.redmine.org/redmine/trunk@20270 e93f8b46-1217-0410-a6f0-8f06a7374b81

test/functional/email_addresses_controller_test.rb

index deb9961059288f55d79a8e26c6165085f2118735..65a9af15013cfbe0ec68df3eeb74633930d0e23d 100644 (file)
@@ -28,9 +28,7 @@ class EmailAddressesControllerTest < Redmine::ControllerTest
 
   def test_index_with_no_additional_emails
     @request.session[:user_id] = 2
-    get :index, :params => {
-        :user_id => 2
-      }
+    get(:index, :params => {:user_id => 2})
     assert_response :success
   end
 
@@ -38,9 +36,7 @@ class EmailAddressesControllerTest < Redmine::ControllerTest
     @request.session[:user_id] = 2
     EmailAddress.create!(:user_id => 2, :address => 'another@somenet.foo')
 
-    get :index, :params => {
-        :user_id => 2
-      }
+    get(:index, :params => {:user_id => 2})
     assert_response :success
     assert_select '.email', :text => 'another@somenet.foo'
   end
@@ -49,39 +45,35 @@ class EmailAddressesControllerTest < Redmine::ControllerTest
     @request.session[:user_id] = 2
     EmailAddress.create!(:user_id => 2, :address => 'another@somenet.foo')
 
-    get :index, :params => {
-        :user_id => 2
-      },
-      :xhr => true
+    get(:index, :params => {:user_id => 2}, :xhr => true)
     assert_response :success
     assert_include 'another@somenet.foo', response.body
   end
 
   def test_index_by_admin_should_be_allowed
     @request.session[:user_id] = 1
-    get :index, :params => {
-        :user_id => 2
-      }
+    get(:index, :params => {:user_id => 2})
     assert_response :success
   end
 
   def test_index_by_another_user_should_be_denied
     @request.session[:user_id] = 3
-    get :index, :params => {
-        :user_id => 2
-      }
+    get(:index, :params => {:user_id => 2})
     assert_response 403
   end
 
   def test_create
     @request.session[:user_id] = 2
     assert_difference 'EmailAddress.count' do
-      post :create, :params => {
+      post(
+        :create,
+        :params => {
           :user_id => 2,
           :email_address => {
             :address => 'another@somenet.foo'
           }
         }
+      )
       assert_response 302
       assert_redirected_to '/users/2/email_addresses'
     end
@@ -93,13 +85,16 @@ class EmailAddressesControllerTest < Redmine::ControllerTest
   def test_create_as_js
     @request.session[:user_id] = 2
     assert_difference 'EmailAddress.count' do
-      post :create, :params => {
+      post(
+        :create,
+        :params => {
           :user_id => 2,
           :email_address => {
             :address => 'another@somenet.foo'
           }
         },
         :xhr => true
+      )
       assert_response 200
     end
   end
@@ -107,12 +102,15 @@ class EmailAddressesControllerTest < Redmine::ControllerTest
   def test_create_with_failure
     @request.session[:user_id] = 2
     assert_no_difference 'EmailAddress.count' do
-      post :create, :params => {
+      post(
+        :create,
+        :params => {
           :user_id => 2,
           :email_address => {
             :address => 'invalid'
           }
         }
+      )
       assert_response :success
       assert_select_error /email is invalid/i
     end
@@ -123,12 +121,15 @@ class EmailAddressesControllerTest < Redmine::ControllerTest
 
     with_settings :email_domains_denied => 'black.example' do
       assert_no_difference 'EmailAddress.count' do
-        post :create, :params => {
+        post(
+          :create,
+          :params => {
             :user_id => 2,
             :email_address => {
               :address => 'another@black.example'
             }
           }
+        )
         assert_response :success
         assert_select_error 'Email is invalid'
       end
@@ -136,12 +137,15 @@ class EmailAddressesControllerTest < Redmine::ControllerTest
 
     with_settings :email_domains_allowed => 'white.example' do
       assert_no_difference 'EmailAddress.count' do
-        post :create, :params => {
+        post(
+          :create,
+          :params => {
             :user_id => 2,
             :email_address => {
               :address => 'something@example.fr'
             }
           }
+        )
         assert_response :success
         assert_select_error 'Email is invalid'
       end
@@ -151,12 +155,15 @@ class EmailAddressesControllerTest < Redmine::ControllerTest
   def test_create_should_send_security_notification
     @request.session[:user_id] = 2
     ActionMailer::Base.deliveries.clear
-    post :create, :params => {
+    post(
+      :create,
+      :params => {
         :user_id => 2,
         :email_address => {
           :address => 'something@example.fr'
         }
       }
+    )
     mail = ActionMailer::Base.deliveries.last
     assert_not_nil mail
     assert_mail_body_match '0.0.0.0', mail
@@ -173,11 +180,14 @@ class EmailAddressesControllerTest < Redmine::ControllerTest
     @request.session[:user_id] = 2
     email = EmailAddress.create!(:user_id => 2, :address => 'another@somenet.foo')
 
-    put :update, :params => {
+    put(
+      :update,
+      :params => {
         :user_id => 2,
         :id => email.id,
         :notify => '0'
       }
+    )
     assert_response 302
 
     assert_equal false, email.reload.notify
@@ -187,12 +197,15 @@ class EmailAddressesControllerTest < Redmine::ControllerTest
     @request.session[:user_id] = 2
     email = EmailAddress.create!(:user_id => 2, :address => 'another@somenet.foo')
 
-    put :update, :params => {
+    put(
+      :update,
+      :params => {
         :user_id => 2,
         :id => email.id,
         :notify => '0'
       },
       :xhr => true
+    )
     assert_response 200
 
     assert_equal false, email.reload.notify
@@ -203,12 +216,15 @@ class EmailAddressesControllerTest < Redmine::ControllerTest
     email = EmailAddress.create!(:user_id => 2, :address => 'another@somenet.foo')
 
     ActionMailer::Base.deliveries.clear
-    put :update, :params => {
+    put(
+      :update,
+      :params => {
         :user_id => 2,
         :id => email.id,
         :notify => '0'
       },
       :xhr => true
+    )
     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
@@ -222,10 +238,13 @@ class EmailAddressesControllerTest < Redmine::ControllerTest
     email = EmailAddress.create!(:user_id => 2, :address => 'another@somenet.foo')
 
     assert_difference 'EmailAddress.count', -1 do
-      delete :destroy, :params => {
+      delete(
+        :destroy,
+        :params => {
           :user_id => 2,
           :id => email.id
         }
+      )
       assert_response 302
       assert_redirected_to '/users/2/email_addresses'
     end
@@ -236,11 +255,14 @@ class EmailAddressesControllerTest < Redmine::ControllerTest
     email = EmailAddress.create!(:user_id => 2, :address => 'another@somenet.foo')
 
     assert_difference 'EmailAddress.count', -1 do
-      delete :destroy, :params => {
+      delete(
+        :destroy,
+        :params => {
           :user_id => 2,
           :id => email.id
         },
         :xhr => true
+      )
       assert_response 200
     end
   end
@@ -249,10 +271,13 @@ class EmailAddressesControllerTest < Redmine::ControllerTest
     @request.session[:user_id] = 2
 
     assert_no_difference 'EmailAddress.count' do
-      delete :destroy, :params => {
+      delete(
+        :destroy,
+        :params => {
           :user_id => 2,
           :id => User.find(2).email_address.id
         }
+      )
       assert_response 404
     end
   end
@@ -262,11 +287,14 @@ class EmailAddressesControllerTest < Redmine::ControllerTest
     email = EmailAddress.create!(:user_id => 2, :address => 'another@somenet.foo')
 
     ActionMailer::Base.deliveries.clear
-    delete :destroy, :params => {
+    delete(
+      :destroy,
+      :params => {
         :user_id => 2,
         :id => email.id
       },
       :xhr => true
+    )
     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