Skip to content

test: wire up CTest integration and fix set -e abort in run_tests.sh#276

Merged
spe-ciellt merged 1 commit intodevelopfrom
test/cmake-ctest-integration
Feb 22, 2026
Merged

test: wire up CTest integration and fix set -e abort in run_tests.sh#276
spe-ciellt merged 1 commit intodevelopfrom
test/cmake-ctest-integration

Conversation

@spe-ciellt
Copy link
Copy Markdown
Contributor

  • Add enable_testing() + add_subdirectory(test) to root CMakeLists.txt
  • New test/CMakeLists.txt: registers gerbv_regression_tests CTest test, sets GERBV, GERBV_SCHEMEINIT, and LD_LIBRARY_PATH via generator expressions so tests work from the build tree without installing
  • Populate testPresets in cmake/preset/base.json and linux-gnu-gcc.json so ctest --preset linux-gnu-gcc works
  • Fix set -e bug in run_tests.sh: append || true to the three ImageMagick diagnostic commands in the failure block so the script no longer aborts at the first failing test
  • Add imagemagick to CI apt dependencies; add non-blocking Run tests step (continue-on-error: true) to the standard (non-coverage) CI job
  • Replace test/README.txt (CVS/Makefile.am references) with test/README.md documenting CTest usage, env vars, Cairo version note, and how to add or regenerate tests

- Add enable_testing() + add_subdirectory(test) to root CMakeLists.txt
- New test/CMakeLists.txt: registers gerbv_regression_tests CTest test,
  sets GERBV, GERBV_SCHEMEINIT, and LD_LIBRARY_PATH via generator
  expressions so tests work from the build tree without installing
- Populate testPresets in cmake/preset/base.json and linux-gnu-gcc.json
  so `ctest --preset linux-gnu-gcc` works
- Fix set -e bug in run_tests.sh: append `|| true` to the three
  ImageMagick diagnostic commands in the failure block so the script
  no longer aborts at the first failing test
- Add imagemagick to CI apt dependencies; add non-blocking Run tests
  step (continue-on-error: true) with -V so the full sub-test summary
  is always visible in CI logs
- Replace test/README.txt (CVS/Makefile.am references) with test/README.md
  documenting CTest usage, env vars, Cairo version note, and how to
  add or regenerate tests

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@spe-ciellt spe-ciellt force-pushed the test/cmake-ctest-integration branch from 06895c7 to 8f6a2eb Compare February 22, 2026 17:36
@spe-ciellt spe-ciellt merged commit 3500b17 into develop Feb 22, 2026
1 check passed
@spe-ciellt spe-ciellt deleted the test/cmake-ctest-integration branch February 23, 2026 18:19
@spe-ciellt spe-ciellt added the fix Solution for a potential problem or omission. label Feb 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Solution for a potential problem or omission.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant