Skip to content

Commit

Permalink
Merge pull request #15151 from imtayadeway/api/bug/request-admin-access
Browse files Browse the repository at this point in the history
Requests should allow access to users with admin role
  • Loading branch information
abellotti authored May 19, 2017
2 parents 0613afb + fe0485b commit 2c07aaa
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion app/controllers/api/requests_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ def find_requests(id)
end

def requests_search_conditions
return {} if User.current_user.admin?
return {} if User.current_user.admin_user?
{:requester => User.current_user}
end

Expand Down
2 changes: 1 addition & 1 deletion app/controllers/api/service_requests_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ def find_service_requests(id)
end

def service_requests_search_conditions
return {} if User.current_user.admin?
return {} if User.current_user.admin_user?
{:requester => User.current_user}
end

Expand Down
2 changes: 1 addition & 1 deletion spec/requests/api/requests_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@
end

it "lists all the service requests if you are admin" do
allow_any_instance_of(User).to receive(:admin?).and_return(true)
@group.miq_user_role = @role = FactoryGirl.create(:miq_user_role, :role => "administrator")
other_user = FactoryGirl.create(:user)
service_request_1 = FactoryGirl.create(:service_template_provision_request,
:requester => other_user,
Expand Down
2 changes: 1 addition & 1 deletion spec/requests/api/service_requests_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -234,7 +234,7 @@ def expect_result_to_have_user_email(email)
end

it "lists all the service requests if you are admin" do
allow_any_instance_of(User).to receive(:admin?).and_return(true)
@group.miq_user_role = @role = FactoryGirl.create(:miq_user_role, :role => "administrator")
other_user = FactoryGirl.create(:user)
service_request_1 = FactoryGirl.create(:service_template_provision_request,
:requester => other_user,
Expand Down

0 comments on commit 2c07aaa

Please sign in to comment.