Merge remote-tracking branch 'upstream/pull/1974'

[rails.git] / test / integration / user_roles_test.rb

diff --git a/test/integration/user_roles_test.rb b/test/integration/user_roles_test.rb
index 2613c402cea5988611b970423c8a111ed738d99b..7001abd3dc9c2a30c2860bf416a0be5ec682daea 100644 (file)
--- a/test/integration/user_roles_test.rb
+++ b/test/integration/user_roles_test.rb
@@ -1,20 +1,18 @@
 require "test_helper"
 
 class UserRolesTest < ActionDispatch::IntegrationTest
 require "test_helper"
 
 class UserRolesTest < ActionDispatch::IntegrationTest

-  fixtures :users, :user_roles
-
-  setup do
-    stub_signup_requests
+  def setup
+    stub_hostip_requests

   end
 
   test "grant" do
   end
 
   test "grant" do

-    check_fail(:grant, :public_user, :moderator)
+    check_fail(:grant, :user, :moderator)

     check_fail(:grant, :moderator_user, :moderator)
     check_success(:grant, :administrator_user, :moderator)
   end
 
   test "revoke" do
     check_fail(:grant, :moderator_user, :moderator)
     check_success(:grant, :administrator_user, :moderator)
   end
 
   test "revoke" do

-    check_fail(:revoke, :public_user, :moderator)
+    check_fail(:revoke, :user, :moderator)

     check_fail(:revoke, :moderator_user, :moderator)
     # this other user doesn't have moderator role, so this fails
     check_fail(:revoke, :administrator_user, :moderator)
     check_fail(:revoke, :moderator_user, :moderator)
     # this other user doesn't have moderator role, so this fails
     check_fail(:revoke, :administrator_user, :moderator)


@@ -28,13 +26,14 @@ class UserRolesTest < ActionDispatch::IntegrationTest
     assert_redirected_to "controller" => "user", "action" => "login", "cookie_test" => "true"
     follow_redirect!
     assert_response :success
     assert_redirected_to "controller" => "user", "action" => "login", "cookie_test" => "true"
     follow_redirect!
     assert_response :success

-    post "/login", "username" => users(user).email, "password" => "test", :referer => "/"
+    post "/login", :params => { "username" => create(user).email, "password" => "test", :referer => "/" }

     assert_response :redirect
     follow_redirect!
     assert_response :success
 
     assert_response :redirect
     follow_redirect!
     assert_response :success
 

-    post "/user/#{users(:second_public_user).display_name}/role/#{role}/#{action}"
-    assert_redirected_to :controller => "user", :action => "view", :display_name => users(:second_public_user).display_name
+    target_user = create(:user)
+    post "/user/#{ERB::Util.u(target_user.display_name)}/role/#{role}/#{action}"
+    assert_redirected_to user_path(target_user)

 
     reset!
   end
 
     reset!
   end


@@ -45,13 +44,14 @@ class UserRolesTest < ActionDispatch::IntegrationTest
     assert_redirected_to "controller" => "user", "action" => "login", "cookie_test" => "true"
     follow_redirect!
     assert_response :success
     assert_redirected_to "controller" => "user", "action" => "login", "cookie_test" => "true"
     follow_redirect!
     assert_response :success

-    post "/login", "username" => users(user).email, "password" => "test", :referer => "/"
+    post "/login", :params => { "username" => create(user).email, "password" => "test", :referer => "/" }

     assert_response :redirect
     follow_redirect!
     assert_response :success
 
     assert_response :redirect
     follow_redirect!
     assert_response :success
 

-    post "/user/#{users(:second_public_user).display_name}/role/#{role}/#{action}"
-    assert_redirected_to :controller => "user", :action => "view", :display_name => users(:second_public_user).display_name
+    target_user = create(:user)
+    post "/user/#{ERB::Util.u(target_user.display_name)}/role/#{role}/#{action}"
+    assert_redirected_to user_path(target_user)

 
     reset!
   end
 
     reset!
   end