fix: appease the linter gods

johnstcn · johnstcn · commit ccd67cce52ff · 2022-04-06T20:05:21.000Z
diff --git a/coderd/database/databasefake/databasefake.go b/coderd/database/databasefake/databasefake.go
@@ -1210,7 +1210,7 @@ func (q *fakeQuerier) UpdateProvisionerJobWithCompleteByID(_ context.Context, ar
 	return sql.ErrNoRows
 }
 
-func (q *fakeQuerier) UpdateWorkspaceAutostart(ctx context.Context, arg database.UpdateWorkspaceAutostartParams) error {
+func (q *fakeQuerier) UpdateWorkspaceAutostart(_ context.Context, arg database.UpdateWorkspaceAutostartParams) error {
 	q.mutex.Lock()
 	defer q.mutex.Unlock()
 
@@ -1226,7 +1226,7 @@ func (q *fakeQuerier) UpdateWorkspaceAutostart(ctx context.Context, arg database
 	return sql.ErrNoRows
 }
 
-func (q *fakeQuerier) UpdateWorkspaceAutostop(ctx context.Context, arg database.UpdateWorkspaceAutostopParams) error {
+func (q *fakeQuerier) UpdateWorkspaceAutostop(_ context.Context, arg database.UpdateWorkspaceAutostopParams) error {
 	q.mutex.Lock()
 	defer q.mutex.Unlock()
 
diff --git a/coderd/workspaces.go b/coderd/workspaces.go
@@ -321,8 +321,6 @@ func (api *api) putWorkspaceAutostart(rw http.ResponseWriter, r *http.Request) {
 		})
 		return
 	}
-
-	return
 }
 
 // TODO(cian): api.updateWorkspaceAutostop
diff --git a/coderd/workspaces_test.go b/coderd/workspaces_test.go
@@ -187,6 +187,7 @@ func TestWorkspaceBuildByName(t *testing.T) {
 }
 
 func TestWorkspaceUpdateAutostart(t *testing.T) {
+	t.Parallel()
 	var dublinLoc = mustLocation(t, "Europe/Dublin")
 
 	testCases := []struct {
@@ -251,6 +252,7 @@ func TestWorkspaceUpdateAutostart(t *testing.T) {
 	for _, testCase := range testCases {
 		testCase := testCase
 		t.Run(testCase.name, func(t *testing.T) {
+			t.Parallel()
 			var (
 				ctx       = context.Background()
 				client    = coderdtest.New(t, nil)

Original file line number	Diff line number	Diff line change
`@@ -1210,7 +1210,7 @@ func (q *fakeQuerier) UpdateProvisionerJobWithCompleteByID(_ context.Context, ar`
`1210`	`1210`	`return sql.ErrNoRows`
`1211`	`1211`	`}`
`1212`	`1212`
`1213`		`-func (q *fakeQuerier) UpdateWorkspaceAutostart(ctx context.Context, arg database.UpdateWorkspaceAutostartParams) error {`
	`1213`	`+func (q *fakeQuerier) UpdateWorkspaceAutostart(_ context.Context, arg database.UpdateWorkspaceAutostartParams) error {`
`1214`	`1214`	`q.mutex.Lock()`
`1215`	`1215`	`defer q.mutex.Unlock()`
`1216`	`1216`
`@@ -1226,7 +1226,7 @@ func (q *fakeQuerier) UpdateWorkspaceAutostart(ctx context.Context, arg database`
`1226`	`1226`	`return sql.ErrNoRows`
`1227`	`1227`	`}`
`1228`	`1228`
`1229`		`-func (q *fakeQuerier) UpdateWorkspaceAutostop(ctx context.Context, arg database.UpdateWorkspaceAutostopParams) error {`
	`1229`	`+func (q *fakeQuerier) UpdateWorkspaceAutostop(_ context.Context, arg database.UpdateWorkspaceAutostopParams) error {`
`1230`	`1230`	`q.mutex.Lock()`
`1231`	`1231`	`defer q.mutex.Unlock()`
`1232`	`1232`
Original file line number	Diff line number	Diff line change
`@@ -321,8 +321,6 @@ func (api api) putWorkspaceAutostart(rw http.ResponseWriter, r http.Request) {`
`321`	`321`	`})`
`322`	`322`	`return`
`323`	`323`	`}`
`324`		`-`
`325`		`- return`
`326`	`324`	`}`
`327`	`325`
`328`	`326`	`// TODO(cian): api.updateWorkspaceAutostop`