Skip to content

Conversation

sloretz
Copy link
Contributor

@sloretz sloretz commented Aug 25, 2020

This cherry-pick backports #201 to ROS Foxy

* Add function for same-package pluginlib tests

Adds pluginlib_enable_plugin_testing()
This CMake function handles creating a fake install environment for
testing pluginlib plugins in the same package that built them.
This replaces manually creating this folder structure just to run unit
tests.

Signed-off-by: Shane Loretz<sloretz@openrobotics.org>
Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>

* Grammar and unused variable

Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>

* Typo in comment

Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>
Signed-off-by: Shane Loretz <sloretz@openrobotics.org>
@sloretz sloretz self-assigned this Aug 25, 2020
@sloretz sloretz changed the base branch from ros2 to foxy August 25, 2020 16:50
@sloretz
Copy link
Contributor Author

sloretz commented Aug 27, 2020

Should cherry-pick #203 before backporting

@sloretz sloretz marked this pull request as draft August 27, 2020 16:53
Signed-off-by: Karsten Knese <karsten@openrobotics.org>
Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>
@sloretz
Copy link
Contributor Author

sloretz commented Aug 27, 2020

Cherry-picked 8b5a1a2 - ready for review again.

@sloretz sloretz marked this pull request as ready for review August 27, 2020 18:24
Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>
@sloretz
Copy link
Contributor Author

sloretz commented Oct 19, 2020

Cherry-picked #211 onto this branch too

@jacobperron
Copy link
Contributor

Friendly ping. What's the status of this PR?

@sloretz
Copy link
Contributor Author

sloretz commented Aug 31, 2021

I definitely forgot about it. It looks like feedback has been addressed, so it needs a review and CI

Copy link
Contributor

@jacobperron jacobperron left a comment

Choose a reason for hiding this comment

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

LGTM

@sloretz
Copy link
Contributor Author

sloretz commented Dec 6, 2021

CI (Foxy repos, build: --packages-up-to pluginlib test: --packages-select pluginlib)

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

@sloretz sloretz merged commit aca1372 into foxy Dec 7, 2021
@sloretz sloretz deleted the foxy_backport_pluginlib_enable_plugin_testing branch December 7, 2021 16:49
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.

4 participants