yarpc / yarpc-go

A message passing platform for Go
MIT License
405 stars 103 forks source link

Added integration test for inbound and outbound with any resolver #2158

Closed gandhikrishna closed 2 years ago

gandhikrishna commented 2 years ago

This PR aims to add integration for inbound and outbound with anyresolver. The test simulates (un)marshalling with anytypes provided no resolver and resolver.

codecov[bot] commented 2 years ago

Codecov Report

Base: 84.82% // Head: 85.00% // Increases project coverage by +0.18% :tada:

Coverage data is based on head (b5b8898) compared to base (ffc30dd). Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## gopal/GoGoDeprecation #2158 +/- ## ========================================================= + Coverage 84.82% 85.00% +0.18% ========================================================= Files 263 263 Lines 14883 14883 ========================================================= + Hits 12625 12652 +27 + Misses 1843 1820 -23 + Partials 415 411 -4 ``` | [Impacted Files](https://codecov.io/gh/yarpc/yarpc-go/pull/2158?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=yarpc) | Coverage Δ | | |---|---|---| | [encoding/protobuf/v2/marshal.go](https://codecov.io/gh/yarpc/yarpc-go/pull/2158/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=yarpc#diff-ZW5jb2RpbmcvcHJvdG9idWYvdjIvbWFyc2hhbC5nbw==) | `94.91% <100.00%> (+18.64%)` | :arrow_up: | | [dispatcher\_startup.go](https://codecov.io/gh/yarpc/yarpc-go/pull/2158/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=yarpc#diff-ZGlzcGF0Y2hlcl9zdGFydHVwLmdv) | `91.07% <0.00%> (-1.79%)` | :arrow_down: | | [encoding/protobuf/v2/error.go](https://codecov.io/gh/yarpc/yarpc-go/pull/2158/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=yarpc#diff-ZW5jb2RpbmcvcHJvdG9idWYvdjIvZXJyb3IuZ28=) | `97.29% <0.00%> (+3.60%)` | :arrow_up: | | [encoding/protobuf/v2/protobuf.go](https://codecov.io/gh/yarpc/yarpc-go/pull/2158/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=yarpc#diff-ZW5jb2RpbmcvcHJvdG9idWYvdjIvcHJvdG9idWYuZ28=) | `71.13% <0.00%> (+4.12%)` | :arrow_up: | | [encoding/protobuf/v2/outbound.go](https://codecov.io/gh/yarpc/yarpc-go/pull/2158/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=yarpc#diff-ZW5jb2RpbmcvcHJvdG9idWYvdjIvb3V0Ym91bmQuZ28=) | `58.76% <0.00%> (+6.18%)` | :arrow_up: | | [encoding/protobuf/v2/inbound.go](https://codecov.io/gh/yarpc/yarpc-go/pull/2158/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=yarpc#diff-ZW5jb2RpbmcvcHJvdG9idWYvdjIvaW5ib3VuZC5nbw==) | `55.17% <0.00%> (+6.89%)` | :arrow_up: | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=yarpc). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=yarpc)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.