ruby-grape / grape / 24693461892

97%

master: 97%

Build Type

Pull #2683

github

Committed by ericproulx

Commit Message

Introduce Grape::Util::Lazy::Base for unified lazy-type dispatch

Adds a shared abstract parent class for Grape::Util::Lazy::Value and
Grape::Util::Lazy::Block so call sites can type-check with a single
is_a?(Grape::Util::Lazy::Base) instead of duck-typing on respond_to?(:lazy?)
/ respond_to?(:evaluate) or enumerating the concrete subclasses.

- New lib/grape/util/lazy/base.rb with a shared #to_s (deduped from Value and Block).
- Value and Block now inherit from Base; their initialize methods call super().
- The internal #lazy? method is removed — no callers remain.
- Grape::API::evaluate_arguments collapses into a single case/when on
  Grape::Util::Lazy::Base / Hash / Array; argument_lazy? indirection is
  removed and any_lazy? uses Array#any?(Base) for a C-level class match.
- Grape::DSL::Routing#evaluate_as_instance_with_configuration and
  Grape::Validations::ParamsScope#configuration swap their respond_to?
  duck checks for explicit is_a?(Grape::Util::Lazy::Base).

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

Pull Request Pull Request #2683: Introduce Grape::Util::Lazy::Base for unified lazy-type dispatch

Coverage Stats

1070 of 1157 branches covered (92.48%)

Branch coverage included in aggregate %.

23 of 25 new or added lines in 6 files covered. (92.0%)

1 existing line in 1 file now uncovered.

3353 of 3407 relevant lines covered (98.42%)

32337.69 hits per line