Skip to content

Basic integration test#30

Closed
ghost wants to merge 4 commits intoros2_develfrom
unknown repository
Closed

Basic integration test#30
ghost wants to merge 4 commits intoros2_develfrom
unknown repository

Conversation

@ghost
Copy link

@ghost ghost commented Aug 24, 2018

This is my idea of what an integration/system test written in pytest could look like.

  • The call to ros2 run would be replaced by a ros2 launch in more complicated configurations.
  • Partitioning functionality between the Node and the test will probably vary and depend on the complexity of the tests.
  • The scope of the test fixtures will probably need to vary depending on the test as well.

@ghost ghost requested review from bpwilcox, mhpanah, mjeronimo, mkhansenbot and orduno August 24, 2018 20:26
@ghost ghost closed this Aug 28, 2018
ghost referenced this pull request in logivations/navigation2 Mar 7, 2022
* after planning handle

* removed helpers
mizdan added a commit to modulab/navigation2 that referenced this pull request Jun 12, 2022
doisyg pushed a commit to doisyg/navigation2 that referenced this pull request Apr 30, 2024
…tf_lookups

Extend util function by also allowing to get the transform msg instea…
turtlewizard73 added a commit to turtlewizard73/navigation2 that referenced this pull request Dec 14, 2024
…gation#3871) (ros-navigation#30)

* change duration tyype from int to double

* change "1" to "1.0" as default wait_duration value

* modify wait_action tests

* fix

* change int values to double in trees

* add usage of Duration::from_seconds()

* fix build failing issue

* delete whitespace

Co-authored-by: maksymdidukh <142302397+maksymdidukh@users.noreply.github.com>
mini-1235 pushed a commit to mini-1235/navigation2 that referenced this pull request Feb 5, 2025
* adding costmaps param descriptions

* split into multiple pages
This pull request was closed.
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.

0 participants