Skip to content

Remove some broken tests #3513

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

Merged
merged 1 commit into from
Oct 31, 2022
Merged

Remove some broken tests #3513

merged 1 commit into from
Oct 31, 2022

Conversation

dkhalanskyjb
Copy link
Collaborator

@dkhalanskyjb dkhalanskyjb commented Oct 31, 2022

These tests haven't been reliable for a very long time for some reason. We could ensure that they pass with a manual check, but it feels like nobody actually relies on this behavior, and it's not clear who even would want to.

https://github.com/search?p=3&q=%22dispatchTimeoutMs+%3D+0%22&type=Code https://grep.app/search?q=dispatchTimeoutMs%20%3D%200

These tests haven't been reliable for a very long time for some
reason. We could ensure them passing by a manual check, but it
feels like nobody actually relies on this behavior, and it's not
clear who even would want to.

https://github.com/search?p=3&q=%22dispatchTimeoutMs+%3D+0%22&type=Code
https://grep.app/search?q=dispatchTimeoutMs%20%3D%200
@dkhalanskyjb dkhalanskyjb requested a review from qwwdfsad October 31, 2022 13:47
@dkhalanskyjb dkhalanskyjb merged commit 6c9d358 into develop Oct 31, 2022
@dkhalanskyjb dkhalanskyjb deleted the fix-runTest-with-0-timeout branch October 31, 2022 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants