diff options
author | Rémy Coutable <remy@rymai.me> | 2017-08-11 14:08:20 +0200 |
---|---|---|
committer | Rémy Coutable <remy@rymai.me> | 2017-08-11 15:14:32 +0200 |
commit | 09a348eb139178be534d181273a360a3125df9f9 (patch) | |
tree | f39b5b1969454a80a2cd0ae03368ff92aa968a8f /spec/requests | |
parent | 810c44ae7468fff1934ba1d54bf2b895d98f0842 (diff) | |
download | gitlab-ce-09a348eb139178be534d181273a360a3125df9f9.tar.gz |
Include the `is_admin` field in the `GET /users/:id` API when current user is an admin
Signed-off-by: Rémy Coutable <remy@rymai.me>
Diffstat (limited to 'spec/requests')
-rw-r--r-- | spec/requests/api/users_spec.rb | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/spec/requests/api/users_spec.rb b/spec/requests/api/users_spec.rb index 2dc7be22f8f..49739a1601a 100644 --- a/spec/requests/api/users_spec.rb +++ b/spec/requests/api/users_spec.rb @@ -217,9 +217,19 @@ describe API::Users do it "does not return the user's `is_admin` flag" do get api("/users/#{user.id}", user) + expect(response).to have_http_status(200) expect(json_response['is_admin']).to be_nil end + context 'when authenticated as admin' do + it 'includes the `is_admin` field' do + get api("/users/#{user.id}", admin) + + expect(response).to have_http_status(200) + expect(json_response['is_admin']).to be(false) + end + end + context 'for an anonymous user' do it "returns a user by id" do get api("/users/#{user.id}") |