-
Notifications
You must be signed in to change notification settings - Fork 62
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
Send busy/idle when an Effect is executing and output binding progress when an Output is recalculating #193
Conversation
89c5c4a
to
ac86826
Compare
9920f14
to
5ebea25
Compare
tests/test_poll.py
Outdated
async def _send_message(self, message: Dict[str, object]) -> None: | ||
pass | ||
|
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.
Is this needed?
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.
Yea, without it defined, unit tests will fail (so it's either define it here or check that the method exists before calling it in Effect_
)
It'll probably make more sense to revisit this "lack of a MockSession" problem in a future PR that I'm working on, so I wouldn't worry about it too much right now
5ebea25
to
0048759
Compare
0048759
to
48c6594
Compare
Co-authored-by: Winston Chang <winston@stdout.org>
Closes #185
Closes #16