Header versioning does not play nicely with Crepe::API.mount

[davidcelis]

Here's a failing spec:

RSpec.describe Crepe::API, 'versioning' do
  context 'with header' do
    app do
      version with: :header, vendor: :pancake, default: :v2

      version :v1 do
        mount Proc.new { [200, {'Content-Type' => 'text/plain'}, ['v1']] }
      end

      version :v2 do
        mount Proc.new { [200, {'Content-Type' => 'text/plain'}, ['v2']] }
      end

      mount Proc.new { [200, {'Content-Type' => 'text/plain'}, ['v3']] }, at: :v3
    end

    it 'routes to several versions at the same path' do
      get '/', {}, 'HTTP_ACCEPT' => 'application/vnd.pancake-v2+json'
      expect(last_response.body).to include 'v2'

      get '/', {}, 'HTTP_ACCEPT' => 'application/vnd.pancake-v1+json'
      expect(last_response.body).to include 'v1'

      expect(get('/v3').body).to include 'v3'
    end

    it 'defaults to the specified version' do
      expect(get('/').body).to include 'v2'
    end
  end
end

By using mount with a proc or Rack application from within a version declaration, the requested endpoints become 404s instead of routing to an app.

[davidcelis]

Adding method: 'GET' to those first two mounts fix the 404s, but then requesting /v3 goes to the v2 Proc.