use predicate methods instead of hard coding verb strings

also change the feeler to subclass AD::Request so that it has all the
methods that Request has

actionpack/lib/action_dispatch/journey/router.rb

@@ -101,7 +101,7 @@ module ActionDispatch
           }
 
           routes =
-            if req.request_method == "HEAD"
+            if req.head?
               match_head_routes(routes, req)
             else
               match_routes(routes, req)

actionpack/test/dispatch/routing_test.rb

@@ -3618,7 +3618,7 @@ private
 end
 
 class TestAltApp < ActionDispatch::IntegrationTest
-  class AltRequest
+  class AltRequest < ActionDispatch::Request
     attr_accessor :path_parameters, :path_info, :script_name
     attr_reader :env
 
@@ -3627,6 +3627,7 @@ class TestAltApp < ActionDispatch::IntegrationTest
       @env = env
       @path_info = "/"
       @script_name = ""
+      super
     end
 
     def request_method

actionpack/test/journey/router_test.rb

@@ -13,7 +13,10 @@ module ActionDispatch
         @formatter = Formatter.new(@routes)
       end
 
-      class FakeRequestFeeler < Struct.new(:env, :called)
+      class FakeRequestFeeler < ActionDispatch::Request
+        attr_writer :env
+        attr_accessor :called
+
         def new env
           self.env = env
           self
@@ -23,10 +26,6 @@ module ActionDispatch
           self.called = true
           'world'
         end
-
-        def path_info; env['PATH_INFO']; end
-        def request_method; env['REQUEST_METHOD']; end
-        def ip; env['REMOTE_ADDR']; end
       end
 
       def test_dashes