feat: getWorkspaces filter site api (#1564)

greyscaled · kylecarbs · commit d457f93db562 · 2022-06-10T17:01:40.000Z
diff --git a/coderd/workspaces.go b/coderd/workspaces.go
@@ -151,7 +151,7 @@ func (api *api) workspaces(rw http.ResponseWriter, r *http.Request) {
 
 	// Empty strings mean no filter
 	orgFilter := r.URL.Query().Get("organization_id")
-	ownerFilter := r.URL.Query().Get("owner_id")
+	ownerFilter := r.URL.Query().Get("owner")
 
 	filter := database.GetWorkspacesWithFilterParams{Deleted: false}
 	if orgFilter != "" {
diff --git a/codersdk/workspaces.go b/codersdk/workspaces.go
@@ -194,7 +194,7 @@ func (f WorkspaceFilter) asRequestOption() requestOption {
 			q.Set("organization_id", f.OrganizationID.String())
 		}
 		if f.Owner != "" {
-			q.Set("owner_id", f.Owner)
+			q.Set("owner", f.Owner)
 		}
 		r.URL.RawQuery = q.Encode()
 	}
diff --git a/site/src/api/api.test.ts b/site/src/api/api.test.ts
@@ -1,5 +1,5 @@
 import axios from "axios"
-import { getApiKey, login, logout } from "./api"
+import { getApiKey, getWorkspacesURL, login, logout } from "./api"
 import * as TypesGen from "./typesGenerated"
 
 describe("api.ts", () => {
@@ -113,4 +113,17 @@ describe("api.ts", () => {
       }
     })
   })
+
+  describe("getWorkspacesURL", () => {
+    it.each<[TypesGen.WorkspaceFilter | undefined, string]>([
+      [undefined, "/api/v2/workspaces"],
+
+      [{ OrganizationID: "1", Owner: "" }, "/api/v2/workspaces?organization_id=1"],
+      [{ OrganizationID: "", Owner: "1" }, "/api/v2/workspaces?owner=1"],
+
+      [{ OrganizationID: "1", Owner: "me" }, "/api/v2/workspaces?organization_id=1&owner=me"],
+    ])(`getWorkspacesURL(%p) returns %p`, (filter, expected) => {
+      expect(getWorkspacesURL(filter)).toBe(expected)
+    })
+  })
 })
diff --git a/site/src/api/api.ts b/site/src/api/api.ts
@@ -120,10 +120,25 @@ export const getWorkspace = async (workspaceId: string): Promise<TypesGen.Worksp
   return response.data
 }
 
-// TODO: @emyrk add query params as arguments. Supports 'organization_id' and 'owner'
-//  'owner' can be a username, user_id, or 'me'
-export const getWorkspaces = async (): Promise<TypesGen.Workspace[]> => {
-  const response = await axios.get<TypesGen.Workspace[]>(`/api/v2/workspaces`)
+export const getWorkspacesURL = (filter?: TypesGen.WorkspaceFilter): string => {
+  const basePath = "/api/v2/workspaces"
+  const searchParams = new URLSearchParams()
+
+  if (filter?.OrganizationID) {
+    searchParams.append("organization_id", filter.OrganizationID)
+  }
+  if (filter?.Owner) {
+    searchParams.append("owner", filter.Owner)
+  }
+
+  const searchString = searchParams.toString()
+
+  return searchString ? `${basePath}?${searchString}` : basePath
+}
+
+export const getWorkspaces = async (filter?: TypesGen.WorkspaceFilter): Promise<TypesGen.Workspace[]> => {
+  const url = getWorkspacesURL(filter)
+  const response = await axios.get<TypesGen.Workspace[]>(url)
   return response.data
 }
 
diff --git a/site/src/testHelpers/handlers.ts b/site/src/testHelpers/handlers.ts
@@ -36,9 +36,6 @@ export const handlers = [
   rest.post("/api/v2/users/me/workspaces", async (req, res, ctx) => {
     return res(ctx.status(200), ctx.json(M.MockWorkspace))
   }),
-  rest.get("/api/v2/workspaces", async (req, res, ctx) => {
-    return res(ctx.status(200), ctx.json([M.MockWorkspace]))
-  }),
   rest.get("/api/v2/users/me/organizations", (req, res, ctx) => {
     return res(ctx.status(200), ctx.json([M.MockOrganization]))
   }),
@@ -79,6 +76,12 @@ export const handlers = [
   }),
 
   // workspaces
+
+  // REMARK: This endpoint works with query parameters, but they won't be
+  //         reflected in the return.
+  rest.get("/api/v2/workspaces", async (req, res, ctx) => {
+    return res(ctx.status(200), ctx.json([M.MockWorkspace]))
+  }),
   rest.get("/api/v2/organizations/:organizationId/workspaces/:userName/:workspaceName", (req, res, ctx) => {
     if (req.params.workspaceName !== M.MockWorkspace.name) {
       return res(

Original file line number	Diff line number	Diff line change
`@@ -194,7 +194,7 @@ func (f WorkspaceFilter) asRequestOption() requestOption {`
`194`	`194`	`q.Set("organization_id", f.OrganizationID.String())`
`195`	`195`	`}`
`196`	`196`	`if f.Owner != "" {`
`197`		`- q.Set("owner_id", f.Owner)`
	`197`	`+ q.Set("owner", f.Owner)`
`198`	`198`	`}`
`199`	`199`	`r.URL.RawQuery = q.Encode()`
`200`	`200`	`}`