Skip to content

[Foxy] backport pluginlib enable plugin testing#202

Merged
sloretz merged 3 commits intofoxyfrom
foxy_backport_pluginlib_enable_plugin_testing
Dec 7, 2021
Merged

[Foxy] backport pluginlib enable plugin testing#202
sloretz merged 3 commits intofoxyfrom
foxy_backport_pluginlib_enable_plugin_testing

Conversation

@sloretz
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor Author

sloretz commented Oct 19, 2020

Cherry-picked #211 onto this branch too

@jacobperron
Copy link
Copy Markdown
Contributor

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

@sloretz
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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