Skip to content

[23638] Fix blackbox test paths when compiling against Fast DDS Pro#6035

Merged
MiguelCompany merged 3 commits intomasterfrom
feature/compile-fastdds-tests-in-pro
Sep 19, 2025
Merged

[23638] Fix blackbox test paths when compiling against Fast DDS Pro#6035
MiguelCompany merged 3 commits intomasterfrom
feature/compile-fastdds-tests-in-pro

Conversation

@Carlosespicur
Copy link
Copy Markdown
Contributor

@Carlosespicur Carlosespicur commented Sep 16, 2025

Description

This PR changes the blackbox test paths to be relative to current directory, instead of the project root. This way, compiling blackbox tests agains Fast DDS Pro is supported.

Contributor Checklist

  • Commit messages follow the project guidelines.
  • The code follows the style guidelines of this project.
  • N/A Tests that thoroughly check the new feature have been added/Regression tests checking the bug and its fix have been added; the added tests pass locally
  • N/A Any new/modified methods have been properly documented using Doxygen.
  • N/A Any new configuration API has an equivalent XML API (with the corresponding XSD extension)
  • N/A Changes are backport compatible: they do NOT break ABI nor change library core behavior.
  • N/A Changes are API compatible.
  • N/A New feature has been added to the versions.md file (if applicable).
  • N/A New feature has been documented/Current behavior is correctly described in the documentation.
  • N/A Applicable backports have been included in the description.

Reviewer Checklist

  • The PR has a milestone assigned.
  • The title and description correctly express the PR's purpose.
  • Check contributor checklist is correct.
  • If this is a critical bug fix, backports to the critical-only supported branches have been requested.
  • Check CI results: changes do not issue any warning.
  • Check CI results: failing tests are unrelated with the changes.

Signed-off-by: Carlosespicur <carlosespicur@proton.me>
@MiguelCompany MiguelCompany added this to the v3.4.0 milestone Sep 17, 2025
@MiguelCompany MiguelCompany self-requested a review September 17, 2025 11:09
@github-actions github-actions bot added the ci-pending PR which CI is running label Sep 17, 2025
Signed-off-by: Carlosespicur <carlosespicur@proton.me>
@Carlosespicur Carlosespicur requested review from MiguelCompany and removed request for MiguelCompany September 17, 2025 12:41
MiguelCompany
MiguelCompany previously approved these changes Sep 18, 2025
Signed-off-by: Carlosespicur <carlosespicur@proton.me>
@MiguelCompany MiguelCompany merged commit d0e2ee4 into master Sep 19, 2025
37 of 41 checks passed
@MiguelCompany MiguelCompany deleted the feature/compile-fastdds-tests-in-pro branch September 19, 2025 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-pending PR which CI is running fast dds pro

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants