fktn-k / fkYAML / 28906424346

100%

push

github

Handle empty mapping key separator as parse error (#525)

* Handle empty mapping key separator as parse error (#514)

Replace an assertion-only guard in KEY_SEPARATOR handling with a parse_error
when the parser sees a mapping value separator without an active context.

This prevents malformed input such as `: foo` from aborting on
FK_YAML_ASSERT(!m_context_stack.empty()) and lets callers handle the failure
through the normal exception path.

Add regression coverage for the empty-key separator case and for the issue #514
input to ensure it does not abort during deserialization.

* Handle root empty mapping keys

Parse a root-level mapping value separator as an empty string key instead of
throwing a parse_error.

This supports YAML such as `: foo` as a mapping with `""` as the key, while
leaving empty mapping keys inside block sequences unsupported for now.

Update the key separator regression tests accordingly.

* Clarify empty key sequence parse-error test

1402 of 1404 branches covered (99.86%)

Branch coverage included in aggregate %.

12 of 13 new or added lines in 1 file covered. (92.31%)

5152 of 5154 relevant lines covered (99.96%)

403.52 hits per line