-
Notifications
You must be signed in to change notification settings - Fork 51
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.
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
Increase test coverage backend #1828
Comments
|
The Tests are now sequential by 6878c2d I'm trying to make tests independent from REDIS (Results backend) if possible. |
The indicators async implementation was done 8c6f732. And there were a concern about the client for if the async/await of JavaScript still makes CPU busy and hot. Fortunately as ChatGPT says, it will not block the CPU. So we can implement the indicators without set/result endpoints, and at the client side we can get the results asynchronously on one request. From Chat GPT:
|
Goal of this issue
Add the tests which are mentioned.
These tests are mentioned by TODO inside tests directory.
Resources
List of TODOs:
Tasks
Deliverables
Branch to derive
dev
The text was updated successfully, but these errors were encountered: