ruby-grape / grape / 30238951520

97%

push

github

Stop entity autodetection from picking up a top-level ::Entity constant (#2825)

entity_class_for_obj used klass.const_defined?(:Entity), which walks the
ancestor chain up to Object — where top-level constants live. Any app
defining a class named Entity had it silently used as the presenter for
every presented object without its own nested Entity (also reachable for
empty collections, whose lookup class resolves to NilClass).

Walk the ancestors explicitly with const_defined?(:Entity, false),
skipping Object, so only a genuine Klass::Entity (on the class or a real
ancestor) is auto-detected. Same family as the ::Options fix in #2773.

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>

1183 of 1274 branches covered (92.86%)

Branch coverage included in aggregate %.

3817 of 3884 relevant lines covered (98.27%)

25070.51 hits per line