8000 feat: add count to get users endpoint by f0ssel · Pull Request #5016 · coder/coder · GitHub
[go: up one dir, main page]
Skip to content

feat: add count to get users endpoint #5016

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 17 commits into from
Nov 14, 2022
+420 −478
Merged

feat: add count to get users endpoint #5016

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
f771959
feat: add count to queries in window function
f0ssel Nov 10, 2022
ed5a147
make initial tests pass
f0ssel Nov 14, 2022
5da633e
fix more tests
f0ssel Nov 14, 2022
8a00932
idk
f0ssel Nov 14, 2022
206381d
get presleys help
f0ssel Nov 14, 2022
a1b7e4e
rename test
f0ssel Nov 14, 2022
482b59f
remove count endpoint
f0ssel Nov 14, 2022
0d738e8
remove more
f0ssel Nov 14, 2022
d5b5000
fmt
f0ssel Nov 14, 2022
ada3995
fix js tests
f0ssel Nov 14, 2022
bd34b05
keep fixing
f0ssel Nov 14, 2022
6249824
fmt
f0ssel Nov 14, 2022
8a92b2b
fmt
f0ssel Nov 14, 2022
ef32b99
fix all js tests
f0ssel Nov 14, 2022
7f43a6c
fix
f0ssel Nov 14, 2022
991b518
move states
f0ssel Nov 14, 2022
b86e0cb
fmt
f0ssel Nov 14, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fmt
  • Loading branch information
@f0ssel
f0ssel committed Nov 14, 2022
commit 8a92b2ba6707351474db1773e19b96622940e011
34 changes: 14 additions & 20 deletions site/src/pages/UsersPage/UsersPage.test.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -199,12 +199,10 @@ describe("UsersPage", () => {

await suspendUser(() => {
jest.spyOn(API, "suspendUser").mockResolvedValueOnce(MockUser)
jest
.spyOn(API, "getUsers")
.mockResolvedValueOnce({
users: [SuspendedMockUser, MockUser2],
count: 2,
})
jest.spyOn(API, "getUsers").mockResolvedValueOnce({
users: [SuspendedMockUser, MockUser2],
count: 2,
})
})

// Check if the success message is displayed
Expand Down Expand Up @@ -277,12 +275,10 @@ describe("UsersPage", () => {

await deleteUser(() => {
jest.spyOn(API, "deleteUser").mockResolvedValueOnce(undefined)
jest
.spyOn(API, "getUsers")
.mockResolvedValueOnce({
users: [MockUser, SuspendedMockUser],
count: 2,
})
jest.spyOn(API, "getUsers").mockResolvedValueOnce({
users: [MockUser, SuspendedMockUser],
count: 2,
})
})

// Check if the success message is displayed
Expand Down Expand Up @@ -326,14 +322,12 @@ describe("UsersPage", () => {
jest
.spyOn(API, "activateUser")
.mockResolvedValueOnce(SuspendedMockUser)
jest
.spyOn(API, "getUsers")
.mockImplementationOnce(() =>
Promise.resolve({
users: [MockUser, MockUser2, SuspendedMockUser],
count: 3,
}),
)
jest.spyOn(API, "getUsers").mockImplementationOnce(() =>
Promise.resolve({
users: [MockUser, MockUser2, SuspendedMockUser],
count: 3,
}),
)
})

// Check if the success message is displayed
Expand Down
0