fix: make status type on filter query

BrunoQuaresma · BrunoQuaresma · commit 3f25967cda26 · 2022-04-28T14:27:43.000Z
diff --git a/coderd/database/databasefake/databasefake.go b/coderd/database/databasefake/databasefake.go
@@ -229,7 +229,7 @@ func (q *fakeQuerier) GetUsers(_ context.Context, params database.GetUsersParams
 	if params.Status != "" {
 		usersFilteredByStatus := make([]database.User, 0, len(users))
 		for i, user := range users {
-			if user.Status == params.Status {
+			if params.Status == string(user.Status) {
 				usersFilteredByStatus = append(usersFilteredByStatus, users[i])
 			}
 		}
diff --git a/coderd/database/queries.sql.go b/coderd/database/queries.sql.go
diff --git a/coderd/database/queries/users.sql b/coderd/database/queries/users.sql
@@ -87,7 +87,7 @@ WHERE
 	END
 	-- Filter by status
 	AND CASE
-		WHEN @status :: user_status != '' THEN (
+		WHEN @status :: text != '' THEN (
 			status = @status
 		)
 		ELSE true
diff --git a/coderd/users.go b/coderd/users.go
@@ -137,7 +137,7 @@ func (api *api) users(rw http.ResponseWriter, r *http.Request) {
 		OffsetOpt: int32(offset),
 		LimitOpt:  int32(pageLimit),
 		Search:    searchName,
-		Status:    database.UserStatus(statusFilter),
+		Status:    statusFilter,
 	})
 
 	if err != nil {

Original file line number	Diff line number	Diff line change
`@@ -229,7 +229,7 @@ func (q *fakeQuerier) GetUsers(_ context.Context, params database.GetUsersParams`
`229`	`229`	`if params.Status != "" {`
`230`	`230`	`usersFilteredByStatus := make([]database.User, 0, len(users))`
`231`	`231`	`for i, user := range users {`
`232`		`- if user.Status == params.Status {`
	`232`	`+ if params.Status == string(user.Status) {`
`233`	`233`	`usersFilteredByStatus = append(usersFilteredByStatus, users[i])`
`234`	`234`	`}`
`235`	`235`	`}`