ruby-grape / grape / 25583726063

97%

push

github

Fix Array/Set custom-type coercer dropping symbolized hash keys (#2699)

The Array/Set branch of `enforce_symbolized_keys` built a symbolized
copy via non-mutating `deep_symbolize_keys` and discarded it, so the
returned collection still carried string-keyed hashes. Switch to
`map!` so the symbolized values replace the originals, matching the
Hash branch's contract.

While here, refactor the class for readability: hoist the
`[Array, Set]` and `[:coerced?, :parsed?]` arrays to private frozen
constants, lift trailing `if/else` chains into guard clauses, extract
`hash_symbolizer` / `collection_symbolizer` / `enumerable_type_check`
helpers, collapse the `infer_coercion_method` + symbolize-wrap chain
into a single `build_coercion_method` entry point, and reorder the
private methods to follow call order from `initialize`.

Add `spec/grape/validations/types/custom_type_coercer_spec.rb`
covering the Array and Set symbolization paths through the public
API. The Array case is the regression test for the bug above.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

1079 of 1163 branches covered (92.78%)

Branch coverage included in aggregate %.

25 of 25 new or added lines in 1 file covered. (100.0%)

3411 of 3466 relevant lines covered (98.41%)

32165.27 hits per line