Skip to content

feat: get and update group IdP Sync settings #14647

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 14 commits into from
Sep 16, 2024
Merged

feat: get and update group IdP Sync settings #14647

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
64aee46
🧹
aslilac Sep 11, 2024
a7cc9a2
sort of testing
aslilac Sep 13, 2024
497d7e6
:(
aslilac Sep 13, 2024
2c243d1
🧹
aslilac Sep 13, 2024
3e1087e
make gen
Emyrk Sep 16, 2024
86ba866
add unit test to enforce unauthorized
Emyrk Sep 16, 2024
c8982a4
update unit test
Emyrk Sep 16, 2024
1093e83
add unit test for legacy group settings
Emyrk Sep 16, 2024
4ac097b
update unit test to use EntryNotFoundErr
Emyrk Sep 16, 2024
812a786
update unit tests and linting
Emyrk Sep 16, 2024
93d916c
add rbac authz test
Emyrk Sep 16, 2024
8449c9d
revert ui change
Emyrk Sep 16, 2024
0045409
gen
Emyrk Sep 16, 2024
ec83e80
fmt
Emyrk Sep 16, 2024
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
Prev Previous commit
Next Next commit
update unit test
  • Loading branch information
@Emyrk
Emyrk committed Sep 16, 2024
commit c8982a484104fed257fd9b92d61ff120b346170d
5 changes: 3 additions & 2 deletions enterprise/coderd/idpsync_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@
string(codersdk.ExperimentMultiOrganization),
}

client, db, user := coderdenttest.NewWithDatabase(t, &coderdenttest.Options{
owner, db, user := coderdenttest.NewWithDatabase(t, &coderdenttest.Options{
Options: &coderdtest.Options{
DeploymentValues: dv,
},
Expand All @@ -40,14 +40,15 @@
},
},
})
orgAdmin, _ := coderdtest.CreateAnotherUser(t, owner, user.OrganizationID, rbac.ScopedRoleOrgAdmin(user.OrganizationID))

ctx := testutil.Context(t, testutil.WaitShort)
dbresv := runtimeconfig.OrganizationResolver(user.OrganizationID, runtimeconfig.NewStoreResolver(db))
entry := runtimeconfig.MustNew[*idpsync.GroupSyncSettings]("group-sync-settings")
err := entry.SetRuntimeValue(dbauthz.AsSystemRestricted(ctx), dbresv, &idpsync.GroupSyncSettings{Field: "august"})

Check failure on line 48 in enterprise/coderd/idpsync_test.go

View workflow job for this annotation

GitHub Actions / lint 

ruleguard: Using 'AsSystemRestricted' is dangerous and should be accompanied by a comment explaining why it's ok and a nolint. (gocritic)
require.NoError(t, err)

settings, err := client.GroupIDPSyncSettings(ctx, user.OrganizationID.String())
settings, err := orgAdmin.GroupIDPSyncSettings(ctx, user.OrganizationID.String())
require.NoError(t, err)
require.Equal(t, "august", settings.Field)
})
Expand Down
Loading