Skip to content

Preserve CTest target identity in JUnit results - #1002

Open
Quchaosheng wants to merge 2 commits into
ros2:rollingfrom
Quchaosheng:fix-unique-junit-test-names
Open

Quchaosheng wants to merge 2 commits into
ros2:rollingfrom
Quchaosheng:fix-unique-junit-test-names

Conversation

@Quchaosheng

Copy link
Copy Markdown
Contributor

Description

Pass the unique CTest target name from add_launch_test() to the JUnit serializer and prefix each generated testcase.classname with it. This keeps distinct generated or parameterized launch tests from collapsing into the same Jenkins test history.

Fixes ros2/ros2#1855
Related to osrf/buildfarm-tools#262

Testing

  • colcon build for launch_testing and launch_testing_ament_cmake
  • colcon test: 99 launch_testing tests passed
  • colcon test: 2 launch_testing_ament_cmake tests passed
  • Verified generated XML contains the CTest target in every testcase.classname

Generative AI

Codex was used to investigate the issue, implement the change, and run the tests.

Signed-off-by: 渠超胜 <quchaosheng000406@163.com>
Signed-off-by: 渠超胜 <quchaosheng000406@163.com>
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.

🧑‍🌾 Buildfarm: Investigate non-unique test names in JUnit results causing incorrect Jenkins test history and age

2 participants