On Thu, Feb 14, 2008 at 10:16 AM, Max Williams <[EMAIL PROTECTED]> wrote: > I'm new to rspec (and coding in general) but would it break a lot of > people's tests if redirect_to was changed to do this by default? It's what > i expected it to do, personally, that is to pass if the given :action and > :controller match up. If i pass a url string, and it's missing the params, > then it seems fair that it should fail, but if for example i just specify a > controller, and that controller is called (with any action) i'd expect it to > pass as well. > > Like i say i'm a newb with no idea of the impact of this to existing tests > :)
It is conceivable that existing examples could break, but the docs do set out the same expectation that you have. Can you file a bug at http://rspec.lighthouseapp.com so we can get it in the queue? > On 14/02/2008, Rick DeNatale <[EMAIL PROTECTED]> wrote: > > On 2/14/08, David Chelimsky <[EMAIL PROTECTED]> wrote: > > > 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: > > > > > > > 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? > > > > > > I think I like that. > > > > > > > > > How about something added to the existing call? > > > > > > response.should redirect_to(:controller => 'foo', :action => > > > 'bar').ignoring_other_options > > > > > > This seems too wordy. > > > > > > > > -- > > > > 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 > _______________________________________________ rspec-users mailing list rspec-users@rubyforge.org http://rubyforge.org/mailman/listinfo/rspec-users
