From 4436099e31c8581248602e423644458d6d0d1616 Mon Sep 17 00:00:00 2001 From: Andy Allan Date: Wed, 12 Jan 2022 16:42:03 +0000 Subject: [PATCH 1/1] Remove params from user deletion test They have no effect, and are likely a copy-paste error from when the test was first written in 39a54f8c14d739c95495e0d2c0ca56826eddfe52 --- test/controllers/users_controller_test.rb | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/controllers/users_controller_test.rb b/test/controllers/users_controller_test.rb index 0a9a82f1c..e979e5f6d 100644 --- a/test/controllers/users_controller_test.rb +++ b/test/controllers/users_controller_test.rb @@ -568,18 +568,18 @@ class UsersControllerTest < ActionDispatch::IntegrationTest user = create(:user, :home_lat => 12.1, :home_lon => 12.1, :description => "test") # Try without logging in - delete user_path(user), :params => { :status => "suspended" } + delete user_path(user) assert_response :forbidden # Now try as a normal user session_for(user) - delete user_path(user), :params => { :status => "suspended" } + delete user_path(user) assert_response :redirect assert_redirected_to :controller => :errors, :action => :forbidden # Finally try as an administrator session_for(create(:administrator_user)) - delete user_path(user), :params => { :status => "suspended" } + delete user_path(user) assert_response :redirect assert_redirected_to :action => :show, :display_name => user.display_name -- 2.39.5