Skip to content
Snippets Groups Projects

fix: disable DD4Hep tests in cmake

Merged Christoph Hasse requested to merge chasse_dd4hep_tests into master

Came across this during reference bot developments.

Returning exit(77) from a test kind of works, but not perfectly. While the test registers as "skipped", validation is still executed and this results in newly created empty reference files. This would then result in automatic reference updates erasing the ref files of DD4Hep tests.

In this case, it is known at configuration time if those tests should be run or not, so I've moved the skipping to the CMake level.

Edited by Christoph Hasse

Merge request reports

Pipeline #2995429 passed

Pipeline passed for ba49fb47 on chasse_dd4hep_tests

Approval is optional

Merged by Christoph HasseChristoph Hasse 3 years ago (Sep 8, 2021 7:27am UTC)

Merge details

  • Changes merged into master with e5aaeef9 (commits were squashed).
  • Deleted the source branch.

Pipeline #2998146 passed

Pipeline passed for e5aaeef9 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading