Skip to content
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

Merging RPC Client / Server test to main #290

Merged
merged 2 commits into from
Aug 15, 2024
Merged

Merging RPC Client / Server test to main #290

merged 2 commits into from
Aug 15, 2024

Conversation

gregmedd
Copy link
Contributor

Closes #174

Previously reviewed and approved in #275

debruce and others added 2 commits August 14, 2024 13:56
This test demonstrates up-cpp RPC code in a round trip test using
UTransportMock. This test is not working with a single transport
handle. It will work with two transport handles sharing the same
identity. It not clear if this is a problem with UTransportMock or an
unexpected issue in up-cpp code.
RPC client/server test using UTransportMock
@gregmedd gregmedd self-assigned this Aug 14, 2024
Copy link

Code coverage report is ready! 📈

Copy link

@PLeVasseur PLeVasseur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@gregmedd gregmedd merged commit 6da74c8 into main Aug 15, 2024
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Write extra RpcClientTest for up-cpp
3 participants