« Return to Thread: params are making my "should redirect_to" test fail - why??

Re: params are making my "should redirect_to" test fail - why??

by David Chelimsky-2 :: Rate this Message:

Reply to Author | View in Thread

On Thu, Feb 14, 2008 at 9:48 AM, Rick DeNatale <rick.denatale@...> wrote:

> On 2/14/08, David Chelimsky <dchelimsky@...> wrote:
>  > On Thu, Feb 14, 2008 at 8:22 AM, Rick DeNatale <rick.denatale@...> wrote:
>  >  > On 2/14/08, Max Williams <toastkid.williams@...> wrote:
>
>
> >  >  >  I've been looking for documentation for methods for the response object, to
>  >  >  > get the url, but i can't find any in the api docs (i'm probably just looking
>  >  >  > in the wrong place).  I can see the url data in the object but it's private.
>  >  >
>  >  >  Well the RedirectTo matcher gets it using
>  >  >
>  >  >     response.redirect_url
>  >  >
>  >  >  I'd probably write the check as
>  >  >
>  >  >    response.should be_redirect
>  >  >    response.redirect_url.should
>  >  >  match(%r{^http://test.host/admin/users/batch_saved(\?|$)})
>  >
>  >
>  > That seems reasonable given what is offered.
>  >
>  >  What do you guys think of a new matcher named redirect_with:
>  >
>  >  response.should redirect_with(:controller => 'admin/users', :action =>
>  >  'batch_saved')
>  >
>  >  This would let you express exactly what you want and only accept a
>  >  Hash and only match those present in the expectation, ignoring
>  >  anything else in the Hash.
>  >
>  >  I don't have the cycles to add this anytime soon, so if you like the
>  >  idea, feel free to submit a patch to the tracker.
>
>  I'd be happy to do that, although I'm not entirely convinced that
>  redirect_with is the right name, not that I've got a better
>  alternative.

redirect_with_options?

How about something added to the existing call?

response.should redirect_to(:controller => 'foo', :action =>
'bar').ignoring_other_options

Or something like that?

>
>  --
>
>
> Rick DeNatale
>
>  My blog on Ruby
>  http://talklikeaduck.denhaven2.com/
>  _______________________________________________
>  rspec-users mailing list
>  rspec-users@...
>  http://rubyforge.org/mailman/listinfo/rspec-users
>
_______________________________________________
rspec-users mailing list
rspec-users@...
http://rubyforge.org/mailman/listinfo/rspec-users

 « Return to Thread: params are making my "should redirect_to" test fail - why??