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

[ws client]: fix faulty poll in test. #330

Merged

dvdplm merged 1 commit into master from na-remove-faulty-poll-in-test

May 21, 2021

Member

niklasad1 commented May 21, 2021 •

edited

Loading

After the notification stream returned None it shouldn't be polled again because it will panic, which this PR fixes.

Also added a check that client is still connected.


          [ws client]: fix faulty poll in test.

047b16d

dvdplm approved these changes

View reviewed changes

dvdplm merged commit 3329e9c into master

dvdplm deleted the na-remove-faulty-poll-in-test branch

May 21, 2021 10:55

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

Labels

None yet