ruby-grape / grape / 30486032579

97%

push

github

Set api.version when the version is the only path segment (#2826)

* Set api.version when the version is the only path segment

The path versioner only extracted the version when a second slash
followed it (path_info.index('/', 1)), so the root route of a
path-versioned API (GET /v1, or /v1.json with a format suffix) matched
and ran without env['api.version'] — the `version` helper returned nil
and entity `version:` embeds were omitted.

When the path is a single segment, record the version on an exact match
against the declared versions, trimming a trailing .format suffix
(right-to-left, so dotted versions like v1.2 still match). An unmatched
segment is left for the router to resolve — never a 404 from here.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* Clarify dotted version and prefix contexts with nested RSpec contexts

Split the dotted version and prefix test cases into separate nested contexts
to make it explicit that they test different scenarios (different options
overrides). This answers the question: why do we need to override options[:versions]
or options[:prefix] mid-context? Answer: we don't — each scenario gets its own
context with its own let(:options).

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

---------

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

1196 of 1285 branches covered (93.07%)

Branch coverage included in aggregate %.

3807 of 3874 relevant lines covered (98.27%)

26783.57 hits per line