On Thu, Feb 14, 2008 at 9:48 AM, Rick DeNatale <[EMAIL PROTECTED]> wrote:
> On 2/14/08, David Chelimsky <[EMAIL PROTECTED]> wrote:
> > On Thu, Feb 14, 2008 at 8:22 AM, Rick DeNatale <[EMAIL PROTECTED]> wrote:
> > > On 2/14/08, Max Williams <[EMAIL PROTECTED]> 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@rubyforge.org
> http://rubyforge.org/mailman/listinfo/rspec-users
>
_______________________________________________
rspec-users mailing list
rspec-users@rubyforge.org
http://rubyforge.org/mailman/listinfo/rspec-users
