explicit "namespace" necessary at top of files mounted to a resource location

[clifton]

class Users < Grape::API
  get '/' do
    User.all
  end
end

class API < Grape::API
  mount Users => "/users"
end

Calling /users results in a 404. This is circumvented by adding another explicit namespace at the top of Users. Should this be necessary?

[jch]

I reproduced this locally, but I don't think the extra namespace should be necessary. api.rb's mount method does keep track of the path, but I didn't follow further to see how the route matching is done when a path is set. If you come up with a fix for this, I can look it over and merge it.

[clifton]

Thanks for the quick reply. Yeah, it's probably worth making a pull request for it. I guess the pull request would modify API#mount to set the namespace inside of the mounted class, but that's just from glancing at the source.

[dblock]

This is a dup of #60. Setting the namespace could be a solution. There's a small spec that demonstrates this failure in that issue, too.