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

Added rmw_client, rmw_destroy_client and rmw_service_server_is_available tests #140

Merged
merged 5 commits into from
Sep 30, 2020

Conversation

ahcorde
Copy link
Contributor

@ahcorde ahcorde commented Sep 24, 2020

Testing:

  • rmw_client
  • rmw_destroy_client
  • rmw_service_server_is_available

Related to this PR

Signed-off-by: ahcorde ahcorde@gmail.com

@ahcorde ahcorde added the tests Failing or missing tests label Sep 24, 2020
@ahcorde ahcorde requested a review from hidmic September 24, 2020 14:23
@ahcorde ahcorde self-assigned this Sep 24, 2020
@ahcorde
Copy link
Contributor Author

ahcorde commented Sep 24, 2020

Related with this PR ros2/rmw_fastrtps#447

Copy link
Contributor

@hidmic hidmic left a comment

Choose a reason for hiding this comment

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

There's one overlap with #138 we have to sort out.

test_rmw_implementation/test/test_client.cpp Show resolved Hide resolved
test_rmw_implementation/test/test_client.cpp Show resolved Hide resolved
@ahcorde
Copy link
Contributor Author

ahcorde commented Sep 25, 2020

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@ahcorde
Copy link
Contributor Author

ahcorde commented Sep 25, 2020

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@hidmic
Copy link
Contributor

hidmic commented Sep 29, 2020

Alright, PR rebased on top of #138.

@hidmic
Copy link
Contributor

hidmic commented Sep 29, 2020

CI up to test_rmw_implementation and rcl:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@hidmic
Copy link
Contributor

hidmic commented Sep 29, 2020

@ros-pull-request-builder retest this please.

Copy link
Contributor

@Lobotuerk Lobotuerk left a comment

Choose a reason for hiding this comment

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

LGTM

@hidmic
Copy link
Contributor

hidmic commented Sep 29, 2020

@ros-pull-request-builder retest this please.

…ble tests

Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
@Lobotuerk
Copy link
Contributor

Depends on ros2/rosidl_typesupport_connext#60 to be released and rolling to be upped

@ahcorde
Copy link
Contributor Author

ahcorde commented Sep 30, 2020

@ros-pull-request-builder retest this please.

3 similar comments
@hidmic
Copy link
Contributor

hidmic commented Sep 30, 2020

@ros-pull-request-builder retest this please.

@hidmic
Copy link
Contributor

hidmic commented Sep 30, 2020

@ros-pull-request-builder retest this please.

@hidmic
Copy link
Contributor

hidmic commented Sep 30, 2020

@ros-pull-request-builder retest this please.

@hidmic
Copy link
Contributor

hidmic commented Sep 30, 2020

It passed ! Finally all green. Merging!

@hidmic hidmic merged commit e8746fe into master Sep 30, 2020
@delete-merged-branch delete-merged-branch bot deleted the ahcorde/test/rmw_client branch September 30, 2020 19:26
ahcorde added a commit that referenced this pull request Oct 9, 2020
Signed-off-by: ahcorde <ahcorde@gmail.com>
ahcorde added a commit that referenced this pull request Oct 21, 2020
Signed-off-by: ahcorde <ahcorde@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests Failing or missing tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants