From 88f99a54f7f62bf74b8662739422a0d219643c0f Mon Sep 17 00:00:00 2001 From: Ben Irvin Date: Tue, 18 Jul 2023 17:40:11 +0200 Subject: [PATCH] Revert "fix defaultRole and comment" This reverts commit f33ad9cc2966f3de465426c7be45d16aa874159b. --- api-tests/core/admin/ee/provider-options.test.api.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/api-tests/core/admin/ee/provider-options.test.api.js b/api-tests/core/admin/ee/provider-options.test.api.js index f06fcff90e..3dd0b17cd6 100644 --- a/api-tests/core/admin/ee/provider-options.test.api.js +++ b/api-tests/core/admin/ee/provider-options.test.api.js @@ -103,7 +103,7 @@ describeOnCondition(edition === 'EE')('SSO Provider Options', () => { ])('can be %s', async (name, value) => { const newData = { ssoLockedRoles: value, - defaultRole: localData.restrictedRole.id, // TODO: there seems to be a bug with not setting a default role + defaultRole: 1, // TODO: there seems to be a bug with not setting a default role autoRegister: false, }; const res = await requests.admin.put('/admin/providers/options', { @@ -127,7 +127,7 @@ describeOnCondition(edition === 'EE')('SSO Provider Options', () => { const res = await requests.admin.put('/admin/providers/options', { body: { ssoLockedRoles: value, - defaultRole: localData.restrictedRole.id, // TODO: there seems to be a bug with not setting a default role + defaultRole: localData.restrictedRole.id, autoRegister: false, }, });