nlohmann / json

JSON for Modern C++
https://json.nlohmann.me
MIT License
41.25k stars 6.57k forks source link

exclude some parts of the tests from the build #4385

Closed EfesX closed 1 month ago

EfesX commented 1 month ago

I've excluded the test parts that only work if JSON_DisableEnumSerialization option is OFF.

4384


Pull request checklist

Read the Contribution Guidelines for detailed information.

Please don't

github-actions[bot] commented 1 month ago

πŸ”΄ Amalgamation check failed! πŸ”΄

The source code has not been amalgamated. @EfesX Please read and follow the Contribution Guidelines.

coveralls commented 1 month ago

Coverage Status

coverage: 100.0%. remained the same when pulling 6d75a9fb0b1dd79abace90859a7a3a8fe86ddf5e on EfesX:issue_4384_fix_build_noenum_test into 8c391e04fe4195d8be862c97f38cfe10e2a3472e on nlohmann:develop.

github-actions[bot] commented 1 month ago

πŸ”΄ Amalgamation check failed! πŸ”΄

The source code has not been amalgamated. @EfesX Please read and follow the Contribution Guidelines.

github-actions[bot] commented 1 month ago

πŸ”΄ Amalgamation check failed! πŸ”΄

The source code has not been amalgamated. @EfesX Please read and follow the Contribution Guidelines.

github-actions[bot] commented 1 month ago

πŸ”΄ Amalgamation check failed! πŸ”΄

The source code has not been amalgamated. @EfesX Please read and follow the Contribution Guidelines.

EfesX commented 1 month ago

Hmm, this seems to be the expected behavior... I should have read the docs more carefully )

JSON_DISABLE_ENUM_SERIALIZATION When defined to `1`, default serialization and deserialization functions for enums are excluded and have to be provided by the user, for example, using [`NLOHMANN_JSON_SERIALIZE_ENUM`](nlohmann_json_serialize_enum.md) (see [arbitrary type conversions](../../features/arbitrary_types.md) for more details).