Actions
Patch #34492
closedFix passing a wrong parameter to assert_select in API test for 'GET /users/:id'
Description
At source:tags/4.1.1/test/integration/api_test/users_test.rb#L121, assert_select there tests the element with User.find(1).status.to_s
. However, it must be User.find(2).status.to_s
because the preceding get
request retrieves data for user 2.
test "GET /users/:id should return status for administrators" do
get '/users/2.xml', :headers => credentials('admin')
assert_response :success
assert_select 'user status', :text => User.find(1).status.to_s
end
The following patch fixes this issue.
diff --git a/test/integration/api_test/users_test.rb b/test/integration/api_test/users_test.rb
index d54701ad5..bd58a39f2 100644
--- a/test/integration/api_test/users_test.rb
+++ b/test/integration/api_test/users_test.rb
@@ -158,7 +158,7 @@ class Redmine::ApiTest::UsersTest < Redmine::ApiTest::Base
test "GET /users/:id should return status for administrators" do
get '/users/2.xml', :headers => credentials('admin')
assert_response :success
- assert_select 'user status', :text => User.find(1).status.to_s
+ assert_select 'user status', :text => User.find(2).status.to_s
end
test "GET /users/:id should return admin status for current user" do
Actions