dblock / grape / 33993564537

99%

master: 97%

push

github

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
  `&.` can never see nil.

Raises the full-matrix branch coverage from 95.58% (1278/1337) to
96.85% (1324/1367), while keeping line coverage at 100%. Not aiming for
100% branch coverage; only the easy, high-signal gaps were closed.

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

1291 of 1334 branches covered (96.78%)

Branch coverage included in aggregate %.

4013 of 4013 relevant lines covered (100.0%)

27271.76 hits per line