Skip to content

Conversation

@NilsEb
Copy link
Collaborator

@NilsEb NilsEb commented Aug 5, 2025

  • Created from nlohmann_json's unit-class_parser.cpp an new test class
  • Replaced parse_helper and accept_helper with direct json::accept() and json::parse() calls
  • Simplification ensures test results directly validate accept() and parse()
  • Enables clear evidence for JSON well-formedness and parsing requirements

@coveralls
Copy link

coveralls commented Aug 5, 2025

Coverage Status

coverage: 99.186%. remained the same
when pulling ea3e06f on add_accept_and_parser_tests_without_test_helpers
into 243a9d6 on main.

@NilsEb NilsEb force-pushed the add_accept_and_parser_tests_without_test_helpers branch from 0b360e8 to 9025bed Compare August 5, 2025 12:32
@github-actions
Copy link

github-actions bot commented Aug 5, 2025

The created documentation from the pull request is available at: docu-html

@NilsEb NilsEb force-pushed the add_accept_and_parser_tests_without_test_helpers branch from abdae21 to e95b495 Compare August 6, 2025 12:55
@NilsEb NilsEb marked this pull request as ready for review August 6, 2025 12:57
@NilsEb NilsEb merged commit 9218972 into main Aug 7, 2025
72 checks passed
@NilsEb NilsEb deleted the add_accept_and_parser_tests_without_test_helpers branch August 7, 2025 07:31
Erikhu1 pushed a commit that referenced this pull request Nov 14, 2025
- Created from nlohmann_json's unit-class_parser.cpp an new test class
- Replaced parse_helper and accept_helper with direct json::accept() and json::parse() calls
- Simplification ensures test results directly validate accept() and parse()
- Enables clear evidence for JSON well-formedness and parsing requirements
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.

4 participants