Skip to content

catch up with boost-ci#74

Merged
jll63 merged 3 commits into
boostorg:developfrom
jll63:fix/b2-alias
May 27, 2026
Merged

catch up with boost-ci#74
jll63 merged 3 commits into
boostorg:developfrom
jll63:fix/b2-alias

Conversation

@jll63
Copy link
Copy Markdown
Collaborator

@jll63 jll63 commented May 25, 2026

Catch up with boost-ci:

  • fix cmake test
  • handle gcc-16 warning-as-error when building unit_test
  • improve coverage

@cppalliance-bot
Copy link
Copy Markdown

cppalliance-bot commented May 25, 2026

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
Copy link
Copy Markdown

codecov Bot commented May 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.36%. Comparing base (814a8e0) to head (87afa9c).
⚠️ Report is 7 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@             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     
Files with missing lines Coverage Δ
test/test_rolex.cpp 100.00% <100.00%> (+57.57%) ⬆️

... and 25 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3632b00...87afa9c. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jll63 jll63 force-pushed the fix/b2-alias branch 2 times, most recently from e4a3aa1 to 4b50749 Compare May 25, 2026 19:07
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>
@jll63 jll63 changed the title build.jam: fix malformed alias so b2 install generates cmake config catch up with boost-ci May 26, 2026
@jll63 jll63 requested a review from pdimov May 26, 2026 02:33
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 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::openmethod interface alias.
  • Keeps warnings visible while disabling warnings-as-errors for Boost.Test in B2 tests.
  • Converts test_rolex.cpp into 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.

@jll63 jll63 merged commit de258ad into boostorg:develop May 27, 2026
56 checks passed
@jll63 jll63 deleted the fix/b2-alias branch May 27, 2026 11:59
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.

3 participants