Skip to content
This repository
Browse code

Merge pull request #13985 from zzak/remove_old_todo_in_url_for_test

Remove outdated TODO from url_for test, this test was fixed in 900a2d3
  • Loading branch information...
commit e08514e822b691fcb76d8c6f3d583a309c81f865 2 parents a503444 + dfe9cf1
Carlos Antonio da Silva carlosantoniodasilva authored

Showing 1 changed file with 0 additions and 3 deletions. Show diff stats Hide diff stats

  1. +0 3  actionpack/test/controller/url_for_test.rb
3  actionpack/test/controller/url_for_test.rb
@@ -204,9 +204,6 @@ def test_trailing_slash_with_params
204 204 end
205 205
206 206 def test_relative_url_root_is_respected
207   - # ROUTES TODO: Tests should not have to pass :relative_url_root directly. This
208   - # should probably come from routes.
209   -
210 207 add_host!
211 208 assert_equal('https://www.basecamphq.com/subdir/c/a/i',
212 209 W.new.url_for(:controller => 'c', :action => 'a', :id => 'i', :protocol => 'https', :script_name => '/subdir')

0 comments on commit e08514e

Please sign in to comment.
Something went wrong with that request. Please try again.