ruby-grape / grape / 30238951520 / 15

97%

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

224 of 1272 branches covered (17.61%)

Branch coverage included in aggregate %.

2392 of 3880 relevant lines covered (61.65%)

7.97 hits per line