Rails Ruby on Rails → 2.x

Comments and changes to this ticket

• You flagged this item as spam.
  

  mudboy September 1st, 2008 @ 09:57 AM

  It's very easy to repeat this bug, and it only seems to effect the integration test.

  step to reproduce.

  $ rails appname

  then

  $ script/generate scaffold Book title:string description:text

  run the migration

  $ rake db:migrate

  make a integration test

  $ script/generate integration_test Book

  Then add the following to the test.

  
    def test_truth
      id = books(:one).id
      post "/books/#{id}", :book => { :title => 'hello'}
      assert_response :success
  @@@@
  

• You flagged this item as spam.
  

  Mike Weber September 3rd, 2008 @ 05:30 AM

  I don't believe this is a bug in Rails. The routing code is resolving to the only route that makes sense for a POST. But in a RESTful application, POSTs are only used for creating resources, which don't have an ID in the route. You'll need to use PUT in your test, rather than a POST.

• You flagged this item as spam.
  

  mudboy September 3rd, 2008 @ 10:50 AM

  Thanks for your insight Mike, I have been tearing my hair out trying to work out why this wasn't working. It just stems from my not understanding how the RESTful routes work.

  For future reference where is the best place to ask questions like this?

• 

  Michael Koziarski September 3rd, 2008 @ 12:56 PM

  
  elsif ActionController::CgiRequest::DEFAULT_SESSION_OPTIONS[:database_manager] == CGI::Session::CookieStore && session.respond_to?(:dbman) && session.dbman.respond_to?(:generate_digest)
  

  Couldn't you just do:

  
    if ActionController::Base.session_store == CGI::Session::CookieStore
  

  Or is there a case where we're using the cookie store and don't have dbman respond_to

• You flagged this item as spam.
  

  Mike Weber September 3rd, 2008 @ 04:43 PM

  The best resources for help with understanding Rails would either be the mailing list http://lists.rubyonrails.org/mai... or the #rubyonrails IRC channel on irc.freenode.net.

• 

  Tarmo Tänav September 18th, 2008 @ 04:30 AM

  • State changed from “new” to “invalid”
  • Tag changed from “2.1, actionpack, bug, routing” to “2.1, actionpack, bug, routing”

  Looks like Michael actually intended to reply to #957 :), closing this due to not being a bug.