Replace some legacy routes with redirects
authorTom Hughes <tom@compton.nu>
Tue, 3 Dec 2013 19:08:21 +0000 (19:08 +0000)
committerTom Hughes <tom@compton.nu>
Tue, 3 Dec 2013 19:08:21 +0000 (19:08 +0000)
config/routes.rb
test/functional/site_controller_test.rb
test/functional/user_controller_test.rb
test/integration/redirect_test.rb

index d110472bfc3ce10416ad40d73d6fcd22e50e06ef..500a9b13c279c04a74a7b4ecb0023bef3ba11fda 100644 (file)
@@ -162,9 +162,9 @@ OpenStreetMap::Application.routes.draw do
   match '/user/forgot-password' => 'user#lost_password', :via => [:get, :post]
   match '/user/suspended' => 'user#suspended', :via => :get
 
-  match '/index.html' => 'site#index', :via => :get
-  match '/create-account.html' => 'user#new', :via => :get
-  match '/forgot-password.html' => 'user#lost_password', :via => :get
+  get '/index.html', :to => redirect(:path => "/")
+  get '/create-account.html', :to => redirect(:path => "/user/new")
+  get '/forgot-password.html', :to => redirect(:path => "/user/forgot-password")
 
   # permalink
   match '/go/:code' => 'site#permalink', :via => :get, :code => /[a-zA-Z0-9_@~]+[=-]*/
index 86a8fe609f1a66c2d43aef97823b482e8bf74041..d55761f8203315570579705578197cfd3f742572 100644 (file)
@@ -14,10 +14,6 @@ class SiteControllerTest < ActionController::TestCase
       { :path => "/", :method => :post },
       { :controller => "site", :action => "index" }
     )
-    assert_recognizes(
-      { :controller => "site", :action => "index" },
-      { :path => "/index.html", :method => :get }
-    )
     assert_routing(
       { :path => "/edit", :method => :get },
       { :controller => "site", :action => "edit" }
index 873beccadd0b1e88041818407c2e0895abdc73a7..d4121c1bd495765a478d7cdee90c5258eb115f31 100644 (file)
@@ -49,10 +49,6 @@ class UserControllerTest < ActionController::TestCase
       { :path => "/user/new", :method => :get },
       { :controller => "user", :action => "new" }
     )
-    assert_recognizes(
-      { :controller => "user", :action => "new" },
-      { :path => "/create-account.html", :method => :get }
-    )
 
     assert_routing(
       { :path => "/user/new", :method => :post },
@@ -112,10 +108,6 @@ class UserControllerTest < ActionController::TestCase
       { :path => "/user/forgot-password", :method => :post },
       { :controller => "user", :action => "lost_password" }
     )
-    assert_recognizes(
-      { :controller => "user", :action => "lost_password" },
-      { :path => "/forgot-password.html", :method => :get }
-    )
     assert_routing(
       { :path => "/user/reset-password", :method => :get },
       { :controller => "user", :action => "reset_password" }
index 1c4af7d4fe81752888294b9218519c1227744d63..357c20212fe542e852282ca5441dcfac737355ac 100644 (file)
@@ -1,6 +1,20 @@
 require File.dirname(__FILE__) + '/../test_helper'
 
 class RedirectTest  < ActionDispatch::IntegrationTest
+  def test_legacy_redirects
+    get "/index.html"
+    assert_response :redirect
+    assert_redirected_to "/"
+
+    get "/create-account.html"
+    assert_response :redirect
+    assert_redirected_to "/user/new"
+
+    get "/forgot-password.html"
+    assert_response :redirect
+    assert_redirected_to "/user/forgot-password"
+  end
+
   def test_search_redirects
     get "/?query=test"
     assert_response :redirect