Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion app/forms/idp/create.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -133,7 +133,7 @@ export function CreateIdpSideModalForm() {
<TextField
name="groupAttributeName"
label="Group attribute name"
description="Name of SAML attribute where we can find a comma-separated list of names of groups the user belongs to"
description="Name of the SAML attribute in the IdP response listing the user’s groups"
control={form.control}
/>
{/* TODO: Email field, probably */}
Expand Down
16 changes: 8 additions & 8 deletions app/forms/idp/edit.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -104,14 +104,14 @@ export function EditIdpSideModalForm() {
control={form.control}
disabled
/>
{/* TODO: add group attribute name when it is added to the API
<TextField
name="groupAttributeName"
label="Group attribute name"
description="Name of SAML attribute where we can find a comma-separated list of names of groups the user belongs to"
control={form.control}
disabled
/> */}
<TextField
name="groupAttributeName"
label="Group attribute name"
description="Name of the SAML attribute in the IdP response listing the user’s groups"
required
control={form.control}
disabled
/>
{/* TODO: Email field, probably */}
<TextField
name="technicalContactEmail"
Expand Down
1 change: 1 addition & 0 deletions mock-api/silo.ts
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,7 @@ export const samlIdp: Json<SamlIdentityProvider> = {
slo_url: '',
sp_client_id: '',
technical_contact_email: '',
group_attribute_name: 'groups',
}

// This works differently from Nexus, but the result is the same. In Nexus,
Expand Down
4 changes: 4 additions & 0 deletions test/e2e/silos.e2e.ts
Original file line number Diff line number Diff line change
Expand Up @@ -186,6 +186,10 @@ test('Identity providers', async ({ page }) => {
'text="Single Logout (SLO) URL"',
])

await expect(page.getByRole('textbox', { name: 'Group attribute name' })).toHaveValue(
'groups'
)

await page.getByRole('button', { name: 'Cancel' }).click()
await expectNotVisible(page, ['role=dialog[name="Identity provider"]'])
})
Expand Down