-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Assertion fails during nlohmann/json tests #1241
Comments
We haven't seen a Catch assertion fail in years - this seems to be an issue related to FreeBSD. For reference:
That's this line. Note we are still using Catch v1.9.7 as porting all tests to Catch 2 is still an issue. Nevertheless, it would be interesting to understand what the reported assertion means and if there is a bug somewhere. |
@yurivict Can you check what happens when you run Catch's self-test on FreeBSD? |
Yes, there are assertions:
Version 2.2.1 |
Now I am trying to update the FreeBSD port Any hope to resolve this? |
sections, see also catchorg/Catch2#816 (comment) As a result, when built with gcc, loop iterations were skipped. When built with clang, the test aborted with an assertion in catch.hpp line 6222. This also addresses the issues discussed here: nlohmann#1032 (comment) and here: catchorg/Catch2#1241 Please note that this introduces new problems, as some of the unit tests fail now - the library stores keys in lexographical order, while the cbor/msgpack/ubjson examples store them in original order.
This is caused by nlohmann-json using duplicate section names in loops, basically the problem described here: #816 (comment) You can probably close this issue, as this is not caused by catch, but by an invalid use of it. |
The fix in nlohmann-json has been merged. This is neither a catch2 nor a FreeBSD issue, you can really close it now. |
I have been a bit busy so I didn't notice that it has been resolved since. I am going to tag this with a "Revisit" label, to check if we can report this to the user somehow. Thank you for reporting back. |
Tests in https://github.com/nlohmann/json version 3.1.2 fail on FreeBSD due to assertions in catch2: nlohmann/json#1032 (comment)
The text was updated successfully, but these errors were encountered: