Skip to content

Basic move to Catch2 3+ - #53

Open
Arniiiii wants to merge 3 commits into
arximboldi:masterfrom
Arniiiii:master
Open

Arniiiii wants to merge 3 commits into
arximboldi:masterfrom
Arniiiii:master

Conversation

@Arniiiii

Copy link
Copy Markdown

Fixes: #47

@loqs

loqs commented Apr 15, 2026

Copy link
Copy Markdown

Following on from the basic move all instances of the combined header catch2/catch_all.hpp can be replaced bycatch2/catch_test_macros.hpp with the following needing catch2/matchers/catch_matchers_floating_point.hpp for the WithinAbs matcher:

  • test/transducer/range.cpp
  • test/transducer/unzip.cpp
  • test/transducer/zip.cpp
    Additionally the CATCH_CONFIG_MAIN define can be dropped from test/CMakeLists.txt and test/__inline_var_tests/main.cpp as Catch2::Catch2WithMain provides main().

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.

Update to use the current catch2

2 participants