gazebosim / gz-transport

Transport library for component communication based on publication/subscription and service calls.
https://gazebosim.org
Apache License 2.0
29 stars 36 forks source link

Use a default timeout when requesting a service from CLI. #486

Closed caguero closed 3 months ago

caguero commented 3 months ago

🦟 Bug fix

Fixes #475

This patch makes the timeout optional when requesting services from the command line.

How to test it?

Compile the transport examples and run:

./responser

And then type in a separate terminal:

gz service -s /echo --reqtype gz.msgs.StringMsg --reptype gz.msgs.StringMsg --req 'data: "Hello"'

You should receive:

data: "Hello"

Summary

Checklist

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.73%. Comparing base (eac2e69) to head (0f59b2f). Report is 6 commits behind head on gz-transport13.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## gz-transport13 #486 +/- ## ================================================== + Coverage 87.69% 87.73% +0.04% ================================================== Files 59 59 Lines 5704 5709 +5 ================================================== + Hits 5002 5009 +7 + Misses 702 700 -2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.