linting

Emyrk · Emyrk · commit 33c0bc4d8c57 · 2024-11-18T12:09:53.000-06:00
diff --git a/coderd/database/dbmem/dbmem.go b/coderd/database/dbmem/dbmem.go
@@ -8409,7 +8409,7 @@ func (q *FakeQuerier) ListWorkspaceAgentPortShares(_ context.Context, workspaceI
 	return shares, nil
 }
 
-func (q *FakeQuerier) OIDCClaimFields(ctx context.Context, organizationID uuid.UUID) ([]string, error) {
+func (q *FakeQuerier) OIDCClaimFields(_ context.Context, organizationID uuid.UUID) ([]string, error) {
 	orgMembers := q.getOrganizationMemberNoLock(organizationID)
 
 	var fields []string
diff --git a/coderd/database/oidcclaims_test.go b/coderd/database/oidcclaims_test.go
@@ -137,14 +137,7 @@ type userGenerator struct {
 }
 
 func (g userGenerator) noLink(lt database.LoginType) database.User {
-	return g.user(lt, false, nil)
-}
-
-func (g userGenerator) withLink(lt database.LoginType, rawJSON json.RawMessage) database.User {
-	return g.user(lt, true, rawJSON)
-}
 
-func (g userGenerator) user(lt database.LoginType, createLink bool, rawJSON json.RawMessage) database.User {
 	t := g.t
 	db := g.db
 
@@ -153,13 +146,17 @@ func (g userGenerator) user(lt database.LoginType, createLink bool, rawJSON json
 	u := dbgen.User(t, db, database.User{
 		LoginType: lt,
 	})
+	return u
+}
 
-	if !createLink {
-		return u
-	}
+func (g userGenerator) withLink(lt database.LoginType, rawJSON json.RawMessage) database.User {
+	t := g.t
+	db := g.db
+
+	user := g.noLink(lt)
 
 	link := dbgen.UserLink(t, db, database.UserLink{
-		UserID:    u.ID,
+		UserID:    user.ID,
 		LoginType: lt,
 	})
 
@@ -188,7 +185,7 @@ func (g userGenerator) user(lt database.LoginType, createLink bool, rawJSON json
 		require.NoError(t, err)
 	}
 
-	return u
+	return user
 }
 
 type rawUpdater interface {

Original file line number	Diff line number	Diff line change
`@@ -8409,7 +8409,7 @@ func (q *FakeQuerier) ListWorkspaceAgentPortShares(_ context.Context, workspaceI`
`8409`	`8409`	`return shares, nil`
`8410`	`8410`	`}`
`8411`	`8411`
`8412`		`-func (q *FakeQuerier) OIDCClaimFields(ctx context.Context, organizationID uuid.UUID) ([]string, error) {`
	`8412`	`+func (q *FakeQuerier) OIDCClaimFields(_ context.Context, organizationID uuid.UUID) ([]string, error) {`
`8413`	`8413`	`orgMembers := q.getOrganizationMemberNoLock(organizationID)`
`8414`	`8414`
`8415`	`8415`	`var fields []string`