ruby-grape / grape / 34029220227

99%

push

github

Improve branch test coverage (#2897)

* Bring test suite line coverage to 100%

Add specs for previously-untested classes/methods and branches across
the middleware, router, serve_stream, util, validations and
exceptions namespaces.

Also remove a structurally unreachable "format not supported" throw
in Grape::Middleware::Error#format_message: formatter_for always
falls back to a registered Txt formatter (registered via
ErrorFormatter::Base.inherited), so `formatter` can never be falsy
and the 406 branch could never execute.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Improve branch coverage for exceptions, namespace, router, testing and translation utilities

Adds direct unit tests to exercise previously-uncovered branches:
- Grape::Exceptions::Base#translate_message (Proc/Hash-pattern/plain-value branches, and the Hash-without-:key error case)
- Grape::Exceptions::RequestError (new spec file)
- Grape::Exceptions::Validation (message omitted case)
- Grape::Middleware::Base (instance-level #default_options fallback)
- Grape::Middleware::Error (#resolved_backtrace fallback chain, InvalidVersionHeader precedence)
- Grape::Middleware::Stack::Middleware#== (non-Middleware/non-Class comparison)
- Grape::Middleware::Versioner::Header (skips media-type matching without a vendor)
- Grape::Namespace.joined_space
- Grape::Router::BaseRoute#initialize (ActiveSupport::OrderedOptions wrapping)
- Grape::Router::MustermannPattern (:param capture syntax, Integer vs. default constraint)
- Grape::Router::Pattern::Path#suffix (non-path versioning)
- Grape::Testing (missing block / no registered hooks)
- Grape::Util::Translation#translate (explicit locale / explicit default / fallback-locale retry)

Also marks one genuinely unreachable branch (comment-only, not deleted,
per project convention):
- Grape::DSL::Routing#version's trailing `@versions&.last` — @versions
  is unconditionally reassigned to an Array a few lines above, so the
  `&.` ca... (continued)

1291 of 1334 branches covered (96.78%)

Branch coverage included in aggregate %.

4013 of 4013 relevant lines covered (100.0%)

27273.27 hits per line