You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As a FE developer at Coder, I would like to have full test coverage and I would also like to be able to run my test suite without seeing flaking tests.
In #3207 we removed a flaking user test that was frequently failing on main (both locally and in CI). This test was holding up PRs and causing angst when testing locally.
When we have time, we should investigate this test failure more thoroughly and add it back in a more stable way.
Details
../site/src/pages/UsersPage/UsersPage.test.tsx
● Users Page › suspend user › when it is success › shows a success message and refresh the page
expect(jest.fn()).toBeCalledTimes(expected)
Expected number of calls: 1
Received number of calls: 2
195 |
196 | // Check if the users list was reload
> 197 | await waitFor(() => expect(API.getUsers).toBeCalledTimes(1))
| ^
198 | })
199 | })
200 |
AC
test is added back in
test runs consistently locally and in CI
Definition of done
AC is completed
The text was updated successfully, but these errors were encountered:
As a FE developer at Coder, I would like to have full test coverage and I would also like to be able to run my test suite without seeing flaking tests.
In #3207 we removed a flaking user test that was frequently failing on main (both locally and in CI). This test was holding up PRs and causing angst when testing locally.
When we have time, we should investigate this test failure more thoroughly and add it back in a more stable way.
Details
../site/src/pages/UsersPage/UsersPage.test.tsx
AC
Definition of done
The text was updated successfully, but these errors were encountered: