Remove unnecessary token granting from the user_preferences tests
authorAndy Allan <git@gravitystorm.co.uk>
Wed, 31 Oct 2018 10:36:24 +0000 (11:36 +0100)
committerAndy Allan <git@gravitystorm.co.uk>
Wed, 31 Oct 2018 10:36:24 +0000 (11:36 +0100)
Sufficient permissions are granted by the basic authorisation, so this
isn't testing anything.

test/controllers/user_preferences_controller_test.rb
test/test_helper.rb

index 1a51779ed867a1653ad43f4e8f9b82866d672c69..3e5cbb36998b3b3f03b40037f4756adfd122361f 100644 (file)
@@ -35,7 +35,6 @@ class UserPreferencesControllerTest < ActionController::TestCase
 
     # authenticate as a user with no preferences
     basic_authorization create(:user).email, "test"
-    grant_oauth_token :allow_read_prefs
 
     # try the read again
     get :read
@@ -76,7 +75,6 @@ class UserPreferencesControllerTest < ActionController::TestCase
 
     # authenticate as a user with preferences
     basic_authorization user.email, "test"
-    grant_oauth_token :allow_read_prefs
 
     # try the read again
     get :read_one, :params => { :preference_key => "key" }
@@ -110,7 +108,6 @@ class UserPreferencesControllerTest < ActionController::TestCase
 
     # authenticate as a user with preferences
     basic_authorization user.email, "test"
-    grant_oauth_token :allow_write_prefs
 
     # try the put again
     assert_no_difference "UserPreference.count" do
@@ -162,7 +159,6 @@ class UserPreferencesControllerTest < ActionController::TestCase
 
     # authenticate as a user with preferences
     basic_authorization user.email, "test"
-    grant_oauth_token :allow_write_prefs
 
     # try adding a new preference
     assert_difference "UserPreference.count", 1 do
@@ -200,7 +196,6 @@ class UserPreferencesControllerTest < ActionController::TestCase
 
     # authenticate as a user with preferences
     basic_authorization user.email, "test"
-    grant_oauth_token :allow_write_prefs
 
     # try the delete again
     assert_difference "UserPreference.count", -1 do
index b7b93455206e1fb5d01090cdec53b5bca27f94f9..83cf909dd94f208c5b89c20e024f8e6941e040f4 100644 (file)
@@ -87,16 +87,6 @@ module ActiveSupport
       @request.env["HTTP_AUTHORIZATION"] = format("Basic %{auth}", :auth => Base64.encode64("#{user}:#{pass}"))
     end
 
-    ##
-    # set oauth token permissions
-    def grant_oauth_token(*tokens)
-      request.env["oauth.token"] = AccessToken.new do |token|
-        tokens.each do |t|
-          token.public_send("#{t}=", true)
-        end
-      end
-    end
-
     ##
     # set request readers to ask for a particular error format
     def error_format(format)