chore(grpc-mock): restructure and add mock grpc tests

[ethanoroshiba]

Summary

Divided GRPC mock tests by component and added a few more.

Background

Some parts of the GRPC mock still weren't tested, and the crate also needed a test for the new delay capabilities.

Changes

• Divided tests up by mock, response, and matcher.
• Added field to HealthCheckRequest so that we could more accurately test matcher::partialpbjson.
• Added tests for the and mock combinator, matchers not matching incorrect requests, up_to_n_times, DynamicResponse, and delay.

Testing

All tests passing.

Related Issues

closes #902
closes #1559

[Fraser999]

I'd suggest doing let start = Instant::now(); just before this, and after the .await.unwrap() below, you can assert that start.elapsed() >= const delay.

[ethanoroshiba]

Same comment as above, but happy to put it in still if needed

[SuperFluffy]

I'd prefer to not change this file because it's taken from a well known service.

[ethanoroshiba]

Reverted in 4b15aa0, but needed a type with more fields to adequately test message_partial_pbjson so I added a new struct MockMessage to test_utils. Let me know if this works for you!

[SuperFluffy]

I didn't mean for you to implement this manually (although I am sure it was a good exercise to learn about prost and manual serde impls).

What if you you copy the echo server over (from here: https://github.com/hyperium/tonic/blob/master/examples/proto/echo/echo.proto) and just generate extra code?

That way we don't have to worry about potential breakage should we ever update prost or tonic.

[SuperFluffy]

This is looking good. I would not do the manual the manual mock message - just because we might end up in a situation where we bump either of prost, tonic, or pbjson, subtly breaking the derived code.

[SuperFluffy]

I didn't mean for you to implement this manually (although I am sure it was a good exercise to learn about prost and manual serde impls).

What if you you copy the echo server over (from here: https://github.com/hyperium/tonic/blob/master/examples/proto/echo/echo.proto) and just generate extra code?

That way we don't have to worry about potential breakage should we ever update prost or tonic.

[SuperFluffy]

Was this added by you or by prost?

[ethanoroshiba]

Added by prost, I think at some point I may have bumped the dep and this is a new thing that the builder does? If I remember correctly, I believe I've noticed it elsewhere as well.

[ethanoroshiba]

@SuperFluffy in RE to the comment ^^above (since I can't reply directly), I'm not sure the echo server would be sufficient as is. To adequately test the partial_pbjson matcher, we need a message with at least two fields, which the echo request/response does not have. Were you meaning to copy it over and add another field?

[joroshiba]

This PR is stale because it has been open 45 days with no activity. Remove stale label or this PR will be
closed in 7 days.

[SuperFluffy]

@SuperFluffy in RE to the comment ^^above (since I can't reply directly), I'm not sure the echo server would be sufficient as is. To adequately test the partial_pbjson matcher, we need a message with at least two fields, which the echo request/response does not have. Were you meaning to copy it over and add another field?

I didn't realize I never answered. Sorry about that.

What I meant to say was that I rather you rename the health service to something else, or introduce a new service rather than extending a "well known service" (to avoid confusion).