From 3080d1c55993ec37590c8a76da569ef3a34dc6fd Mon Sep 17 00:00:00 2001 From: Hank Grabowski Date: Tue, 1 Jan 2019 11:05:55 -0500 Subject: [PATCH] Authentication cucumber tests fixed by updating to for new scopes names --- features/step_definitions/auth_code_steps.rb | 2 +- features/step_definitions/implicit_flow_steps.rb | 4 ++-- features/step_definitions/oidc_common_steps.rb | 4 ++-- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/features/step_definitions/auth_code_steps.rb b/features/step_definitions/auth_code_steps.rb index 723b913cb..a4f71475d 100644 --- a/features/step_definitions/auth_code_steps.rb +++ b/features/step_definitions/auth_code_steps.rb @@ -3,7 +3,7 @@ O_AUTH_QUERY_PARAMS_WITH_CODE = { redirect_uri: "http://example.org/", response_type: "code", - scope: "openid profile read", + scope: "openid profile", nonce: "hello", state: "hi" } diff --git a/features/step_definitions/implicit_flow_steps.rb b/features/step_definitions/implicit_flow_steps.rb index 316ecfa74..ee4d4bbfb 100644 --- a/features/step_definitions/implicit_flow_steps.rb +++ b/features/step_definitions/implicit_flow_steps.rb @@ -3,7 +3,7 @@ O_AUTH_QUERY_PARAMS = { redirect_uri: "http://example.org/", response_type: "id_token token", - scope: "openid profile read", + scope: "openid profile", nonce: "hello", state: "hi", prompt: "login" @@ -12,7 +12,7 @@ O_AUTH_QUERY_PARAMS = { O_AUTH_QUERY_PARAMS_WITH_MAX_AGE = { redirect_uri: "http://example.org/", response_type: "id_token token", - scope: "openid profile read", + scope: "openid profile", nonce: "hello", state: "hi", prompt: "login", diff --git a/features/step_definitions/oidc_common_steps.rb b/features/step_definitions/oidc_common_steps.rb index 971f7c2e0..0952a0293 100644 --- a/features/step_definitions/oidc_common_steps.rb +++ b/features/step_definitions/oidc_common_steps.rb @@ -3,12 +3,12 @@ Given /^a client with a provided picture exists for user "([^\"]*)"$/ do |email| app = FactoryGirl.create(:o_auth_application, logo_uri: "/assets/user/default.png") user = User.find_by(email: email) - FactoryGirl.create(:auth_with_read, user: user, o_auth_application: app) + FactoryGirl.create(:auth_with_read_scopes, user: user, o_auth_application: app) end Given /^a client exists for user "([^\"]*)"$/ do |email| user = User.find_by(email: email) - FactoryGirl.create(:auth_with_read, user: user) + FactoryGirl.create(:auth_with_read_scopes, user: user) end When /^I register a new client$/ do