Matthew Palmer
2008-Jan-29 04:10 UTC
Patch review req: make assert_routing handle HTTP methods
At present, fully testing your resourceful routes is a pest because assert_routing doesn''t like anything other than a string as it''s first argument. Well, no longer! The patch I just attached to http://dev.rubyonrails.org/ticket/8039 fixes up assert_routing to properly handle hashes in it''s first argument, so it works more like assert_recognizes. It has tests *and* documentation. It''s, like, the perfect patch. +1s appreciated. - Matt --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To post to this group, send email to rubyonrails-core@googlegroups.com To unsubscribe from this group, send email to rubyonrails-core-unsubscribe@googlegroups.com For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en -~----------~----~----~----~------~----~------~--~---
Chad Woolley
2008-Jan-29 10:53 UTC
Re: Patch review req: make assert_routing handle HTTP methods
On Jan 28, 2008 9:10 PM, Matthew Palmer <mpalmer@hezmatt.org> wrote:> Well, no longer! The patch I just attached to > http://dev.rubyonrails.org/ticket/8039 fixes up assert_routing to properly > handle hashes in it''s first argument, so it works more like > assert_recognizes. It has tests *and* documentation. It''s, like, the > perfect patch.Looks clean. +1 --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To post to this group, send email to rubyonrails-core@googlegroups.com To unsubscribe from this group, send email to rubyonrails-core-unsubscribe@googlegroups.com For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en -~----------~----~----~----~------~----~------~--~---