ruby-grape / grape / 25583726063 / 32

97%

master: 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>

383 of 1161 branches covered (32.99%)

Branch coverage included in aggregate %.

2515 of 3457 relevant lines covered (72.75%)

8.36 hits per line