specs for declared(params) inside route_param bug.

[Arkanain]

I check some specs on my project and I find out that this functionality work little bit wrong with declared when we have several route_param in one namespace.
For example we have:

class Artists
  namespace :artists do
    route_param :id, type: Integer do
      get do
        declared(params)
      end

      params do
        requires :filter, type: String
      end
      get :some_route do
        declared(params)
      end
    end
    route_param :artist_id, type: Integer do
      mount Compositions
    end
  end
end

declared(params) should return us next hash {id: 1}, but it return hash {id: 1, artist_id: nil}.
Also in first route_param in second endpoint(get :some_route) when we call declared(params) we have only filter params in returned hash {filter: 'some_filter'}

P.S.: I find out where we have problem! Problem is in options for route_param. where I use route_param :id without type: Integer everything is ok.

[Arkanain]

@dblock @namusyaka please review this.

[dblock]

Well these specs pass... weren't they supposed to fail?

[Arkanain]

@dblock Hmm sorry I missed up. I wrote regression specs. I will rewrite them.

[Arkanain]

@dblock I changed specs. not it failed and show problem correct.

[Arkanain]

Hi guys. Any news about this issue?

[Arkanain]

@dblock @namusyaka Please review my code. I find out where was a problem and fixed it. If everything ok can you please merge it, because I need this commit in my project.

[namusyaka]

Looks good to me.
@dblock Any thoughts on this?

[dblock]

Looks good, needs a CHANGELOG entry please.

[Arkanain]

@dblock Done.

[dblock]

Sorry to be a pain. There shouldn't be a period after route_param, also maybe quote the code as below?

[Arkanain]

@dblock update.

[dblock]

Excellent, merging.