diff options
author | Yatish Mehta <yatish.mehta@coupa.com> | 2016-10-26 00:08:53 +0300 |
---|---|---|
committer | Yatish Mehta <yatish.mehta@coupa.com> | 2016-11-08 23:04:05 +0300 |
commit | a0aaf93fe591215a7fc29a52ff6cbd38604c8dcb (patch) | |
tree | 8742a213a18bdae2f7de15b4851c60ed2c898394 /spec/requests/api/users_spec.rb | |
parent | c6d010986724faf10b46453d66eaca38a948fabf (diff) |
Add query param to filter users on 'external' & 'blocked' type on API
Diffstat (limited to 'spec/requests/api/users_spec.rb')
-rw-r--r-- | spec/requests/api/users_spec.rb | 21 |
1 files changed, 21 insertions, 0 deletions
diff --git a/spec/requests/api/users_spec.rb b/spec/requests/api/users_spec.rb index ae8639d78d5..34d1f557e4b 100644 --- a/spec/requests/api/users_spec.rb +++ b/spec/requests/api/users_spec.rb @@ -48,6 +48,17 @@ describe API::API, api: true do end['username']).to eq(username) end + it "returns an array of blocked users" do + ldap_blocked_user + create(:user, state: 'blocked') + + get api("/users?blocked=true", user) + + expect(response).to have_http_status(200) + expect(json_response).to be_an Array + expect(json_response).to all(include('state' => /(blocked|ldap_blocked)/)) + end + it "returns one user" do get api("/users?username=#{omniauth_user.username}", user) expect(response).to have_http_status(200) @@ -69,6 +80,16 @@ describe API::API, api: true do expect(json_response.first.keys).to include 'last_sign_in_at' expect(json_response.first.keys).to include 'confirmed_at' end + + it "returns an array of external users" do + create(:user, external: true) + + get api("/users?external=true", admin) + + expect(response).to have_http_status(200) + expect(json_response).to be_an Array + expect(json_response).to all(include('external' => true)) + end end end |