Refactor rummager client to inherit from GdsApi::Base

This will mean that it uses `JsonClient`, and therefore gets all the timeout goodness etc.

I'm not sure that the behaviour is still correct when not given a query param.  I'm tempted to say that this is not the place to handle empty query params, and therfore there should be no special case handling in here.