Merge pull request rails#25693 from volmer/master

Deprecate usage of nil as route path

Author: rafaelfranca

actionpack/lib/action_dispatch/routing/mapper.rb

@@ -1571,6 +1571,12 @@ def match(path, *rest)
             options  = path
             path, to = options.find { |name, _value| name.is_a?(String) }
 
+            if path.nil?
+              ActiveSupport::Deprecation.warn 'Omitting the route path is deprecated. '\
+                'Specify the path with a String or a Symbol instead.'
+              path = ''
+            end
+
             case to
             when Symbol
               options[:action] = to

actionpack/test/dispatch/routing_test.rb

@@ -402,6 +402,9 @@ def test_pagemarks
         post "create", :as => ""
         put  "update"
         get  "remove", :action => :destroy, :as => :remove
+        tc.assert_deprecated do
+          get action: :show, as: :show
+        end
       end
     end
 
@@ -420,6 +423,10 @@ def test_pagemarks
     get '/pagemark/remove'
     assert_equal 'pagemarks#destroy', @response.body
     assert_equal '/pagemark/remove', pagemark_remove_path
+
+    get '/pagemark'
+    assert_equal 'pagemarks#show', @response.body
+    assert_equal '/pagemark', pagemark_show_path
   end
 
   def test_admin