Nonblocking nits:
We should fix it so that MirConnection::channel is always valid. If some tests don't provide a valid channel, we should fix the tests.
It's a bit weird that complete_response searches through pending_calls twice, but I can see why you've done it that way. Doing it better probably requires deeper changes.
Looks good.
Nonblocking nits: :channel is always valid. If some tests don't provide a valid channel, we should fix the tests.
We should fix it so that MirConnection:
It's a bit weird that complete_response searches through pending_calls twice, but I can see why you've done it that way. Doing it better probably requires deeper changes.