Fix a few tests

BrunoQuaresma · BrunoQuaresma · commit 7be61cdc93cc · 2023-03-21T19:44:34.000Z
diff --git a/coderd/database/dbauthz/querier_test.go b/coderd/database/dbauthz/querier_test.go
@@ -721,7 +721,7 @@ func (s *MethodTestSuite) TestTemplate() {
 		check.Args(database.UpdateTemplateVersionByIDParams{
 			ID:         tv.ID,
 			TemplateID: uuid.NullUUID{UUID: t1.ID, Valid: true},
-		}).Asserts(t1, rbac.ActionUpdate).Returns()
+		}).Asserts(t1, rbac.ActionUpdate).Returns(tv)
 	}))
 	s.Run("UpdateTemplateVersionDescriptionByJobID", s.Subtest(func(db database.Store, check *expects) {
 		jobID := uuid.New()
diff --git a/coderd/templateversions.go b/coderd/templateversions.go
@@ -66,6 +66,7 @@ func (api *API) templateVersion(rw http.ResponseWriter, r *http.Request) {
 // @Summary Patch template version by ID
 // @ID patch-template-version-by-id
 // @Security CoderSessionToken
+// @Accept json
 // @Produce json
 // @Tags Templates
 // @Param templateversion path string true "Template version ID" format(uuid)
diff --git a/codersdk/templateversions.go b/codersdk/templateversions.go
@@ -296,8 +296,8 @@ func (c *Client) PreviousTemplateVersion(ctx context.Context, organization uuid.
 	return version, json.NewDecoder(res.Body).Decode(&version)
 }
 
-func (c *Client) UpdateTemplateVersion(ctx context.Context, versionId uuid.UUID, req PatchTemplateVersionRequest) (TemplateVersion, error) {
-	res, err := c.Request(ctx, http.MethodPatch, fmt.Sprintf("/api/v2/templateversions/%s", versionId), req)
+func (c *Client) UpdateTemplateVersion(ctx context.Context, versionID uuid.UUID, req PatchTemplateVersionRequest) (TemplateVersion, error) {
+	res, err := c.Request(ctx, http.MethodPatch, fmt.Sprintf("/api/v2/templateversions/%s", versionID), req)
 	if err != nil {
 		return TemplateVersion{}, err
 	}

Original file line number	Diff line number	Diff line change
`@@ -296,8 +296,8 @@ func (c *Client) PreviousTemplateVersion(ctx context.Context, organization uuid.`
`296`	`296`	`return version, json.NewDecoder(res.Body).Decode(&version)`
`297`	`297`	`}`
`298`	`298`
`299`		`-func (c *Client) UpdateTemplateVersion(ctx context.Context, versionId uuid.UUID, req PatchTemplateVersionRequest) (TemplateVersion, error) {`
`300`		`- res, err := c.Request(ctx, http.MethodPatch, fmt.Sprintf("/api/v2/templateversions/%s", versionId), req)`
	`299`	`+func (c *Client) UpdateTemplateVersion(ctx context.Context, versionID uuid.UUID, req PatchTemplateVersionRequest) (TemplateVersion, error) {`
	`300`	`+ res, err := c.Request(ctx, http.MethodPatch, fmt.Sprintf("/api/v2/templateversions/%s", versionID), req)`
`301`	`301`	`if err != nil {`
`302`	`302`	`return TemplateVersion{}, err`
`303`	`303`	`}`