Skip to content

Add Client methods for checking and waiting for service availability - #85

Merged
esteve merged 3 commits into
dashingfrom
wait_for_service
May 7, 2020
Merged

Add Client methods for checking and waiting for service availability#85
esteve merged 3 commits into
dashingfrom
wait_for_service

Conversation

@jacobperron

Copy link
Copy Markdown
Contributor

Depends on #76 in order for tests to pass.

Re-enable the service-related tests with some modifications.

Signed-off-by: Jacob Perron <jacob@openrobotics.org>
These methods are very useful for allowing a client to wait for a service to be available before making requests.
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
Signed-off-by: Jacob Perron <jacob@openrobotics.org>

@clalancetteclalancette left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me.

@esteve

Copy link
Copy Markdown
Member

@jacobperron thanks, LGTM

@esteve
esteve merged commit 7bddf8e into dashingMay 7, 2020
@esteve
esteve deleted the wait_for_service branch May 7, 2020 12:28
jacobperron referenced this pull request in osrf/ros2_java Nov 10, 2020
…85)
* Re-enable tests related to services
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
* Add Client methods for checking and waiting for service availability
These methods are very useful for allowing a client to wait for a service to be available before making requests.
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
* Refactor ClientTest to avoid repeatedly sending requests
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
jacobperron referenced this pull request in osrf/ros2_java Nov 10, 2020
…85)
* Re-enable tests related to services
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
* Add Client methods for checking and waiting for service availability
These methods are very useful for allowing a client to wait for a service to be available before making requests.
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
* Refactor ClientTest to avoid repeatedly sending requests
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
jacobperron referenced this pull request in osrf/ros2_java Nov 10, 2020
…85)
* Re-enable tests related to services
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
* Add Client methods for checking and waiting for service availability
These methods are very useful for allowing a client to wait for a service to be available before making requests.
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
* Refactor ClientTest to avoid repeatedly sending requests
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
jacobperron referenced this pull request in osrf/ros2_java Nov 10, 2020
…85)
* Re-enable tests related to services
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
* Add Client methods for checking and waiting for service availability
These methods are very useful for allowing a client to wait for a service to be available before making requests.
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
* Refactor ClientTest to avoid repeatedly sending requests
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
jacobperron added a commit that referenced this pull request May 17, 2021
…85)
* Re-enable tests related to services
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
* Add Client methods for checking and waiting for service availability
These methods are very useful for allowing a client to wait for a service to be available before making requests.
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
* Refactor ClientTest to avoid repeatedly sending requests
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
jacobperron added a commit that referenced this pull request May 17, 2021
…85)
* Re-enable tests related to services
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
* Add Client methods for checking and waiting for service availability
These methods are very useful for allowing a client to wait for a service to be available before making requests.
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
* Refactor ClientTest to avoid repeatedly sending requests
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@jacobperron@esteve@clalancette