catch up with boost-ci#74
Conversation
|
An automated preview of the documentation is available at https://74.openmethod.prtest3.cppalliance.org/libs/openmethod/doc/html/index.html If more commits are pushed to the pull request, the docs will rebuild at the same URL. 2026-05-26 23:20:49 UTC |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #74 +/- ##
===========================================
+ Coverage 94.34% 94.36% +0.02%
===========================================
Files 43 40 -3
Lines 2916 2874 -42
===========================================
- Hits 2751 2712 -39
+ Misses 165 162 -3
... and 25 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
e4a3aa1 to
4b50749
Compare
Use <warnings-as-errors>off instead of <warnings>off for the boost_unit_test_framework dependency. GCC 16 fires a spurious -Warray-bounds in Boost.Test's junit_log_formatter.hpp when compiling unit_test_log.cpp in shared-link mode; <warnings>off does not suppress -Werror, so the .so fails to build. Same pattern used by Boost.Redis and Boost.MySQL. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
This PR updates the test and CI integration to align with newer boost-ci behavior, particularly for installed header-only CMake testing, Boost.Test warning handling, and expanded test coverage.
Changes:
- Adds B2-installed Boost handling in the CMake install test by creating a
Boost::openmethodinterface alias. - Keeps warnings visible while disabling warnings-as-errors for Boost.Test in B2 tests.
- Converts
test_rolex.cppinto a Boost.Test module with explicit test cases and removes unused mix-release-debug override code.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
test/test_rolex.cpp |
Converts the Rolex example into Boost.Test cases with broader dispatch assertions. |
test/mix_release_debug/lib.cpp |
Removes an unused override body from the mixed runtime-checks test fixture. |
test/Jamfile |
Disables warnings-as-errors specifically for Boost.Test framework builds. |
test/cmake_test/CMakeLists.txt |
Adds installed-by-B2 handling for the header-only OpenMethod CMake target. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Catch up with boost-ci: