linting

Emyrk · Emyrk · commit 0ecdab5ff47b · 2024-10-31T11:56:08.000-04:00
diff --git a/coderd/database/dbauthz/dbauthz_test.go b/coderd/database/dbauthz/dbauthz_test.go
@@ -152,7 +152,10 @@ func TestDBAuthzRecursive(t *testing.T) {
 		for i := 2; i < method.Type.NumIn(); i++ {
 			ins = append(ins, reflect.New(method.Type.In(i)).Elem())
 		}
-		if method.Name == "InTx" || method.Name == "Ping" || method.Name == "Wrappers" {
+		if method.Name == "InTx" ||
+			method.Name == "Ping" ||
+			method.Name == "Wrappers" ||
+			method.Name == "PGLocks" {
 			continue
 		}
 		// Log the name of the last method, so if there is a panic, it is
diff --git a/coderd/database/dbmem/dbmem.go b/coderd/database/dbmem/dbmem.go
@@ -339,7 +339,7 @@ func (*FakeQuerier) Ping(_ context.Context) (time.Duration, error) {
 	return 0, nil
 }
 
-func (*FakeQuerier) PGLocks(ctx context.Context) (database.PGLocks, error) {
+func (*FakeQuerier) PGLocks(_ context.Context) (database.PGLocks, error) {
 	return []database.PGLock{}, nil
 }
 
diff --git a/coderd/database/pglocks.go b/coderd/database/pglocks.go
@@ -103,9 +103,9 @@ func (l PGLocks) String() string {
 	var out strings.Builder
 	for i, lock := range l {
 		if i != 0 {
-			out.WriteString("\n")
+			_, _ = out.WriteString("\n")
 		}
-		out.WriteString(lock.String())
+		_, _ = out.WriteString(lock.String())
 	}
 	return out.String()
 }

Original file line number	Diff line number	Diff line change
`@@ -152,7 +152,10 @@ func TestDBAuthzRecursive(t *testing.T) {`
`152`	`152`	`for i := 2; i < method.Type.NumIn(); i++ {`
`153`	`153`	`ins = append(ins, reflect.New(method.Type.In(i)).Elem())`
`154`	`154`	`}`
`155`		`- if method.Name == "InTx" \|\| method.Name == "Ping" \|\| method.Name == "Wrappers" {`
	`155`	`+ if method.Name == "InTx" \|\|`
	`156`	`+ method.Name == "Ping" \|\|`
	`157`	`+ method.Name == "Wrappers" \|\|`
	`158`	`+ method.Name == "PGLocks" {`
`156`	`159`	`continue`
`157`	`160`	`}`
`158`	`161`	`// Log the name of the last method, so if there is a panic, it is`
Original file line number	Diff line number	Diff line change
`@@ -339,7 +339,7 @@ func (*FakeQuerier) Ping(_ context.Context) (time.Duration, error) {`
`339`	`339`	`return 0, nil`
`340`	`340`	`}`
`341`	`341`
`342`		`-func (*FakeQuerier) PGLocks(ctx context.Context) (database.PGLocks, error) {`
	`342`	`+func (*FakeQuerier) PGLocks(_ context.Context) (database.PGLocks, error) {`
`343`	`343`	`return []database.PGLock{}, nil`
`344`	`344`	`}`
`345`	`345`