Build the trade digest tests again
Table of Contents
This page is a capture in the inbox bucket of the product backlog — a pre-sprint idea, not yet pulled into a sprint as a story.
1. What
Commit 41346d023c, "Model the parts' link libraries and lift the CMake guard",
regenerated projects/ores.trading/core/CMakeLists.txt and removed
add_subdirectory(${CMAKE_CURRENT_SOURCE_DIR}/digest_tests). The folder
projects/ores.trading/core/digest_tests/ still holds main.cpp,
trade_economic_digest_write_tests.cpp and its CMakeLists.txt, but no target
builds them. The trading core model must state the extra test directory, so
regeneration keeps it, and the suite must build and pass again.
2. Why
Found on 2026-11-02 in brave_hopper. publish/bin still held a binary from
2026-10-17, which crashed with SIGSEGV in a copy of database::context
because it loaded today's libraries. build.ninja has no
ores.trading.core.digest.tests target. The tests from 7998cd1956, "test the
digest for every family", have not run since the regeneration, so a digest
regression would pass CI. The stale binary also misleads anyone who runs every
binary in publish/bin.
3. References
- 41346d023c, the regeneration that dropped the directory
projects/ores.trading/core/digest_tests/