Improve test suite layering - #93
Draft
orting wants to merge 21 commits into
Draft
Conversation
Recommend the platform Makefile targets as the local developer entrypoints for Daisy tests, clarify the prerequisites for direct ctest usage, and keep the distinction between local usage and explicit CI workflow steps clear in the documentation. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Add suite-level CTest labels for C++ unit tests, .dai unit tests, and .dai system tests; document the selectors; restore the C++ unit-test harness so the suite builds under explicit registration; and update the CI workflows to configure and run the C++ unit tests on Linux, macOS, and Windows. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Make the C++ unit-test support library static for Windows linkage, only link Boost::system when that imported target exists so macOS config-package builds succeed, and add missing Daisy registration declarations to eliminate missing-declaration warnings for explicit registration functions. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Time::now() is only used by the C++ unit tests in this repository, but it should still convert tm_year to a full calendar year. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Keep the PR workflows as one explicit job per platform while retaining the safer dependency and archive lookup hardening. Also stop rewriting the copied managed Python dylib install name on macOS, which avoids install_name_tool failures on current runner images. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Update the Flatpak launcher so persisted lib and sample data are refreshed when the packaged content changes, avoiding stale runtime files after reinstalls or upgrades. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Add a standalone packaged-test bundle and runner for install verification outside the build tree. Bundle dai tests, baselines, and Python helpers; align packaged discovery with disabled CI tests; and document the Flatpak data and temp-directory behavior needed for packaged runs. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Keep build-time C++ checks in the build jobs, then verify the packaged artifacts in downstream package-test jobs using the standalone test bundle. Linux now tests the installed .deb, while macOS and Windows test their packaged zip outputs. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Use the built .deb for Linux package verification, install make in the Windows build job for test-bundle creation, and fix staged macOS dylib references so packaged tests can load SuiteSparse at runtime. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
orting
force-pushed
the
improve-testing
branch
from
August 28, 2026 07:40
6e82b01 to
068196f
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
cxx-unit,dai-unit, anddai-systemNotes
explicit-registration-phase1