Permalink
Browse files

Merge pull request #30 from JeffCohen/master

Fix test that would never fail; and make the test more intention-revealing
  • Loading branch information...
2 parents b0ef9f1 + 7247798 commit 622f53556f32285f80093a0c064e4a024c70c834 @tenderlove tenderlove committed May 19, 2012
Showing with 2 additions and 2 deletions.
  1. +2 −2 test/test_router.rb
View
@@ -257,11 +257,11 @@ def test_path_not_found
"/messages/:id/edit(.:format)",
"/messages/:id(.:format)"
]
- env = rails_env 'PATH_INFO' => '/messages/1.1.1'
+ env = rails_env 'PATH_INFO' => '/messages/unknown/path'
yielded = false
@router.recognize(env) do |*whatever|
- yielded = false
+ yielded = true
end
refute yielded
end

0 comments on commit 622f535

Please sign in to comment.