fixup test

Emyrk · Emyrk · commit 8967a42c466d · 2024-09-16T12:31:46.000-05:00
diff --git a/coderd/idpsync/group_test.go b/coderd/idpsync/group_test.go
@@ -210,9 +210,12 @@ func TestGroupSyncTable(t *testing.T) {
 			},
 		},
 		{
-			Name:          "NoSettingsNoUser",
+			Name:          "NoSettings",
 			GroupSettings: nil,
 			Groups:        map[uuid.UUID]bool{},
+			assertGroups: &orgGroupAssert{
+				ExpectedGroups: []uuid.UUID{},
+			},
 		},
 		{
 			Name: "LegacyMapping",
diff --git a/enterprise/coderd/enidpsync/role.go b/enterprise/coderd/enidpsync/role.go
@@ -60,7 +60,7 @@ func (e EnterpriseIDPSync) ParseRoleClaims(ctx context.Context, mergedClaims jwt
 				slog.F("raw_value", rawType),
 				slog.Error(err),
 			)
-			// TODO: Deterine a static page or not
+			// TODO: Determine a static page or not
 			return idpsync.RoleParams{}, &idpsync.HTTPError{
 				Code:             http.StatusInternalServerError,
 				Msg:              "Login disabled until site wide OIDC config is fixed",

Original file line number	Diff line number	Diff line change
`@@ -210,9 +210,12 @@ func TestGroupSyncTable(t *testing.T) {`
`210`	`210`	`},`
`211`	`211`	`},`
`212`	`212`	`{`
`213`		`- Name: "NoSettingsNoUser",`
	`213`	`+ Name: "NoSettings",`
`214`	`214`	`GroupSettings: nil,`
`215`	`215`	`Groups: map[uuid.UUID]bool{},`
	`216`	`+ assertGroups: &orgGroupAssert{`
	`217`	`+ ExpectedGroups: []uuid.UUID{},`
	`218`	`+ },`
`216`	`219`	`},`
`217`	`220`	`{`
`218`	`221`	`Name: "LegacyMapping",`
Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ func (e EnterpriseIDPSync) ParseRoleClaims(ctx context.Context, mergedClaims jwt`
`60`	`60`	`slog.F("raw_value", rawType),`
`61`	`61`	`slog.Error(err),`
`62`	`62`	`)`
`63`		`- // TODO: Deterine a static page or not`
	`63`	`+ // TODO: Determine a static page or not`
`64`	`64`	`return idpsync.RoleParams{}, &idpsync.HTTPError{`
`65`	`65`	`Code: http.StatusInternalServerError,`
`66`	`66`	`Msg: "Login disabled until site wide OIDC config is fixed",`