Update short link integration tests for hash links

openstreetmap · Aug 4, 2013 · b8f126b · b8f126b
1 parent 58eb17c
commit b8f126b
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/test/integration/short_links_test.rb b/test/integration/short_links_test.rb
@@ -13,23 +13,24 @@ def test_short_link_params
   # utility method to test short links
   def assert_short_link_redirect(short_link)
     lon, lat, zoom = ShortLink::decode(short_link)
+    anchor = "#{zoom}/#{lat}/#{lon}"
 
     # test without marker
     get '/go/' + short_link
-    assert_redirected_to :controller => 'site', :action => 'index', :lat => lat.to_s, :lon => lon.to_s, :zoom => zoom.to_s
+    assert_redirected_to :controller => 'site', :action => 'index', :anchor => anchor
 
     # test with marker
     get '/go/' + short_link + "?m"
-    assert_redirected_to :controller => 'site', :action => 'index', :mlat => lat.to_s, :mlon => lon.to_s, :zoom => zoom.to_s
+    assert_redirected_to :controller => 'site', :action => 'index', :mlat => lat.to_s, :mlon => lon.to_s, :anchor => anchor
 
     # test with layers and a marker
     get '/go/' + short_link + "?m&layers=B000FTF"
-    assert_redirected_to :controller => 'site', :action => 'index', :mlat => lat.to_s, :mlon => lon.to_s, :zoom => zoom.to_s, :layers => "B000FTF"
+    assert_redirected_to :controller => 'site', :action => 'index', :mlat => lat.to_s, :mlon => lon.to_s, :layers => "B000FTF", :anchor => anchor
     get '/go/' + short_link + "?layers=B000FTF&m"
-    assert_redirected_to :controller => 'site', :action => 'index', :mlat => lat.to_s, :mlon => lon.to_s, :zoom => zoom.to_s, :layers => "B000FTF"
+    assert_redirected_to :controller => 'site', :action => 'index', :mlat => lat.to_s, :mlon => lon.to_s, :layers => "B000FTF", :anchor => anchor
 
     # test with some random query parameters we haven't even implemented yet
     get '/go/' + short_link + "?foobar=yes"
-    assert_redirected_to :controller => 'site', :action => 'index', :lat => lat.to_s, :lon => lon.to_s, :zoom => zoom.to_s, :foobar => "yes"
+    assert_redirected_to :controller => 'site', :action => 'index', :foobar => "yes", :anchor => anchor
   end
 end