-
-
Notifications
You must be signed in to change notification settings - Fork 767
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
Fix stream replay in validators #1678
Conversation
Pull Request Test Coverage Report for Build 4568397653
💛 - Coveralls |
dce8262
to
607c1c6
Compare
tests/test_validation.py
Outdated
more_body = message.get("more_body", False) | ||
|
||
assert len(replay) <= len(messages), ( | ||
"Replayed more messages than received, " "break out of while loop" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this can be a single string instead of two?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, black did this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The solution is ugly though 🤮
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yikes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, one minor comment on the assertion message
Fantastic work! Can't wait for the next release 🥇 |
The current implementation of replaying the stream will always replay the first message. This PR fixes this by progressing through the messages with each call.