Merge branch 'feature/add-open-id-discovery-tests' into 'master'

add values to OpenID configuration test, not only checking for issuer key See merge request gitlab-org/gitlab-ce!18773

Merge branch 'feature/add-open-id-discovery-tests' into 'master'
35d0b63e · Rémy Coutable · 11b7f911 · a827f676 · 35d0b63e
Commit 35d0b63e authored May 15, 2018 by Rémy Coutable
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

openid_connect_spec.rb spec/requests/openid_connect_spec.rb +3 -1

No files found.
--- a/spec/requests/openid_connect_spec.rb
+++ b/spec/requests/openid_connect_spec.rb
@@ -159,7 +159,9 @@ describe 'OpenID Connect requests' do
      get '/.well-known/openid-configuration'
      expect(response).to have_gitlab_http_status(200)
-      expect(json_response).to have_key('issuer')
+      expect(json_response['issuer']).to eq('http://localhost')
+      expect(json_response['jwks_uri']).to eq('http://www.example.com/oauth/discovery/keys')
+      expect(json_response['scopes_supported']).to eq(%w[api read_user sudo read_repository openid])
    end
  end
 end