Merge pull request #36741 from Edouard-chin/ec-system-test-url-options

Define the `url_options` needed for SytemTest inside the route proxy:

1 files changed, 7 insertions, 5 deletions

diff --git a/actionpack/lib/action_dispatch/system_test_case.rb b/actionpack/lib/action_dispatch/system_test_case.rb
index 9772beb8aa..4fda2cf44f 100644
--- a/actionpack/lib/action_dispatch/system_test_case.rb
+++ b/actionpack/lib/action_dispatch/system_test_case.rb
@@ -120,7 +120,13 @@ module ActionDispatch
       super
       self.class.driver.use
       @proxy_route = if ActionDispatch.test_app
-        Class.new { include ActionDispatch.test_app.routes.url_helpers }.new
+        Class.new do
+          include ActionDispatch.test_app.routes.url_helpers
+
+          def url_options
+            default_url_options.merge(host: Capybara.app_host)
+          end
+        end.new
       else
         nil
       end
@@ -164,10 +170,6 @@ module ActionDispatch
 
     driven_by :selenium
 
-    def url_options # :nodoc:
-      default_url_options.merge(host: Capybara.app_host)
-    end
-
     def method_missing(method, *args, &block)
       if @proxy_route.respond_to?(method)
         @proxy_route.send(method, *args, &block)