fixup test

Emyrk · Emyrk · commit ec12c4136f6f · 2024-09-16T08:44:39.000-05:00
diff --git a/coderd/idpsync/group_test.go b/coderd/idpsync/group_test.go
@@ -209,9 +209,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
`@@ -209,9 +209,12 @@ func TestGroupSyncTable(t *testing.T) {`
`209`	`209`	`},`
`210`	`210`	`},`
`211`	`211`	`{`
`212`		`- Name: "NoSettingsNoUser",`
	`212`	`+ Name: "NoSettings",`
`213`	`213`	`GroupSettings: nil,`
`214`	`214`	`Groups: map[uuid.UUID]bool{},`
	`215`	`+ assertGroups: &orgGroupAssert{`
	`216`	`+ ExpectedGroups: []uuid.UUID{},`
	`217`	`+ },`
`215`	`218`	`},`
`216`	`219`	`{`
`217`	`220`	`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",`