Stiffstream / restinio

Cross-platform, efficient, customizable, and robust asynchronous HTTP(S)/WebSocket server C++ library with the right balance between performance and ease of use
Other
1.13k stars 92 forks source link

Upgrade Catch2 to v3 #158

Closed eao197 closed 8 months ago

eao197 commented 2 years ago

Catch2 library is used as unit-test framework in the RESTinio development.

Since v3 Catch2 is no more header-only library: https://github.com/catchorg/Catch2/releases/tag/v3.0.1

RESTinio have to be updated to Catch2 after Catch2-v3 will be stabilized and got a few correcting releases.

barracuda156 commented 11 months ago

And currently tests fail due to non-existent header:

fatal error: catch2/catch.hpp: No such file or directory
    5 | #include <catch2/catch.hpp>
      |          ^~~~~~~~~~~~~~~~~~
compilation terminated.

Correct name is catch_all.h.

eao197 commented 10 months ago

It seem that we can't add support for Catch2-v3 for CMake and MxxRu: Catch2-v3 requires generation of catch_user_config.hpp from catch_user_config.hpp.in during the configure phase. This is simply not possible with MxxRu.

So we have to drop MxxRu support.

It's not a problem. But I'm afraid we can't do that in 0.6 branch. So the support for Catch2-v3 will wait until the work on 0.7 branch will be started :(

eao197 commented 8 months ago

Catch2 v3 is used since v.0.7.0.