Skip to content

Test ros2 lifecycle output streams - #1266

Open
Miko997 wants to merge 1 commit into
ros2:rollingfrom
Miko997:test-lifecycle-set-streams
Open

Test ros2 lifecycle output streams#1266
Miko997 wants to merge 1 commit into
ros2:rollingfrom
Miko997:test-lifecycle-set-streams

Conversation

@Miko997

@Miko997 Miko997 commented Aug 12, 2026

Copy link
Copy Markdown

Description

Verify that ros2 lifecycle set writes command errors to stderr and successful transitions to stdout. Also require the unused stream to remain empty so regressions cannot pass through combined process output.

Addresses #484

Is this user-facing behavior change?

No.

Did you use Generative AI?

No.

Additional Information

Validation:

  • python3 -m compileall -q ros2lifecycle/test/test_cli.py
  • git diff --check origin/rolling...HEAD

The full ROS integration test was not run locally because this environment does not contain ROS 2 or colcon.

Signed-off-by: Miko Parkkinen <141511110+Miko997@users.noreply.github.com>
@mergify

mergify Bot commented Aug 12, 2026

Copy link
Copy Markdown

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@Miko997

Miko997 commented Aug 12, 2026

Copy link
Copy Markdown
Author

Validated the current head 6b6e8e27e7b0919df85610f9fb3c1af507767ff5 in the official ros:rolling-ros-base container using rmw_fastrtps_cpp. Both packages built successfully, and the ros2lifecycle test suite passed: 5 tests, 0 errors, 0 failures, 0 skipped.

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.

1 participant