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

quotest occasionally gets stuck after completing all tests #493

Closed
KitsuneRal opened this issue Aug 9, 2021 · 2 comments
Closed

quotest occasionally gets stuck after completing all tests #493

KitsuneRal opened this issue Aug 9, 2021 · 2 comments
Assignees
@KitsuneRal
Labels
bug/fix The library doesn't work as expected

Comments

@KitsuneRal
Copy link
Member

The internal watchdog force-finishes it but it would be great to fix that.
@KitsuneRal KitsuneRal added the bug/fix The library doesn't work as expected label Aug 9, 2021
@KitsuneRal KitsuneRal self-assigned this Oct 4, 2021
@KitsuneRal
Copy link
Member Author

Aside from overlapping quotest runs, quotest may get stuck when rate-limiting occurs. It's not quite clear what is happening but it's pretty clear that waiting for messageSent() and checking pending messages is not the most reliable way to track the sending queue expiry. Queueing events should definitely help (and further reduce the amount of server-driven rate-limiting).

@KitsuneRal
Copy link
Member Author

It doesn't happen much (if at all) recently - seems that overlapping quotest runs were the main cause, and this is already tracked in #435.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@KitsuneRal KitsuneRal

Labels
bug/fix The library doesn't work as expected
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@KitsuneRal