Skip to content

add CTest support and enable testing in CMake configuration#68

Merged
nbassler merged 5 commits into
mainfrom
64-modernize-repository
Mar 28, 2026
Merged

add CTest support and enable testing in CMake configuration#68
nbassler merged 5 commits into
mainfrom
64-modernize-repository

Conversation

@nbassler
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings March 28, 2026 17:57
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR wires up CTest at the top-level CMake configuration and runs the test suite in GitHub Actions, aiming to make tests discoverable and executable from the build directory.

Changes:

  • Enable CTest in the root CMakeLists.txt so ctest can discover tests from the build root.
  • Update test/CMakeLists.txt (rename the test executable to dedx_test, rely on top-level enable_testing()).
  • Run ctest in CI on Linux; adjust a custom command dependency line in buildbins.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
CMakeLists.txt Enables testing at the top level for CTest discovery.
test/CMakeLists.txt Renames a test executable and relies on root-level test enablement.
buildbins/CMakeLists.txt Removes a dependency declaration from a custom command associated with binary_table.
.github/workflows/docs.yml Adds a Linux-only step to execute ctest in CI.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread buildbins/CMakeLists.txt
Comment thread test/CMakeLists.txt Outdated
@nbassler nbassler merged commit d33bee4 into main Mar 28, 2026
5 checks passed
@nbassler nbassler deleted the 64-modernize-repository branch March 28, 2026 18:27
@nbassler nbassler restored the 64-modernize-repository branch March 28, 2026 18:27
@nbassler nbassler linked an issue Mar 28, 2026 that may be closed by this pull request
41 tasks
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.

Modernize repository

2 participants