Route_for

Testing RSpec
1

Get restful-authentication and rspec trunk from github. And a couple
tests
fail. One of them is this assertion:

route_for(:controller => ‘users’, :action => ‘destroy’, :id =>
‘1’).should
== “/users/1”

error msg is:

should route users’s ‘destroy’ action correctly

The recognized options <{“action”=>“show”, “id”=>“1”,
“controller”=>“users”}> did not match <{“action”=>“destroy”, “id”=>“1”,
“controller”=>“users”}>, difference: <{“action”=>“destroy”}>

But assert_generates from rails works fine.

I myself never used route_for helper before. So I am not sure is this is
the
problem of the method, or restful_authentication author misused it, or
rails
2.3RC2 breaks it.

Thanks

Yi

2

On Sun, Mar 8, 2009 at 9:49 PM, Yi [email protected] wrote:

The recognized options <{“action”=>“show”, “id”=>“1”,
“controller”=>“users”}> did not match <{“action”=>“destroy”, “id”=>“1”,
“controller”=>“users”}>, difference: <{“action”=>“destroy”}>

But assert_generates from rails works fine.

I myself never used route_for helper before. So I am not sure is this is the
problem of the method, or restful_authentication author misused it, or rails
2.3RC2 breaks it.

Actually, it’s rspec’s edge that breaks it, documented here:

http://github.com/dchelimsky/rspec-rails/blob/master/Upgrade.markdown

Somebody should probably offer up a patch to restful_authentication.
I’ll do it unless anybody else cares to volunteer.

Cheers,
David

3

On Sun, Mar 8, 2009 at 10:15 PM, David C. [email protected]
wrote:

Actually, it’s rspec’s edge that breaks it, documented here:

http://github.com/dchelimsky/rspec-rails/blob/master/Upgrade.markdown

Somebody should probably offer up a patch to restful_authentication.
I’ll do it unless anybody else cares to volunteer.

Actually, I’m already on it.

4

David, thanks very much

Yi

5

On Sun, Mar 8, 2009 at 10:21 PM, David C. [email protected]
wrote:

should route users’s ‘destroy’ action correctly

Actually, it’s rspec’s edge that breaks it, documented here:

http://github.com/dchelimsky/rspec-rails/blob/master/Upgrade.markdown

Somebody should probably offer up a patch to restful_authentication.
I’ll do it unless anybody else cares to volunteer.

Actually, I’m already on it.

Here it is:

I’ve sent technoweenie a pull request.

Thanks for bringing this to my attention.

Cheers,
David