Skip to content
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

Jump to bottom

polish tests #900

Closed
shunjizhan opened this issue Dec 8, 2023 · 1 comment
Closed

polish tests #900

shunjizhan opened this issue Dec 8, 2023 · 1 comment
Labels
Lv. 3 need some research to find out root cause or/and solution test test related

Comments

@shunjizhan
Copy link
Collaborator

currently endpoints.test is a little too mixed and complicated, it's not easy to run locally. Can probably do some refactor to

  • use some helpers to setup the deterministic env
  • break tests down and move out these that don't require deterministic setup
@shunjizhan shunjizhan added test test related Lv. 2 relatively clear issue, might need some small digging labels Dec 8, 2023
@shunjizhan shunjizhan added Lv. 3 need some research to find out root cause or/and solution and removed Lv. 2 relatively clear issue, might need some small digging labels Feb 28, 2024
@shunjizhan
Copy link
Collaborator Author

fixed by #1032

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Lv. 3 need some research to find out root cause or/and solution test test related
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@shunjizhan