Merge pull request rails#54647 from Edouard-chin/ec-test-root

Modify the Test Runner to allow running test at root:

Author: byroot

railties/lib/rails/test_unit/runner.rb

@@ -91,12 +91,22 @@ def compose_filter(runnable, filter)
 
         private
           def extract_filters(argv)
+            previous_arg_was_a_flag = false
             # Extract absolute and relative paths but skip -n /.*/ regexp filters.
             argv.filter_map do |path|
-              next unless path_argument?(path)
+              current_arg_is_a_flag = /^-{1,2}[a-zA-Z0-9\-_.]+=?.*\Z/.match?(path)
+
+              if previous_arg_was_a_flag && !current_arg_is_a_flag
+                # Handle the case where a flag is followed by another flag (e.g. --fail-fast --seed ...)
+                previous_arg_was_a_flag = false
+                next
+              end
 
               path = path.tr("\\", "/")
               case
+              when current_arg_is_a_flag
+                previous_arg_was_a_flag = true unless path.include?("=") # Handle the case when "--foo=bar" is used.
+                next
               when /(:\d+(-\d+)?)+$/.match?(path)
                 file, *lines = path.split(":")
                 filters << [ file, lines ]
@@ -122,10 +132,6 @@ def regexp_filter?(arg)
             arg.start_with?("/") && arg.end_with?("/")
           end
 
-          def path_argument?(arg)
-            PATH_ARGUMENT_PATTERN.match?(arg)
-          end
-
           def list_tests(patterns)
             tests = Rake::FileList[patterns.any? ? patterns : default_test_glob]
             tests.exclude(default_test_exclude_glob) if patterns.empty?

railties/test/application/test_runner_test.rb

@@ -222,6 +222,54 @@ def test_sanae
       end
     end
 
+    def test_run_test_at_root
+      app_file "my_test.rb", <<-RUBY
+        require "test_helper"
+
+        class MyTest < ActiveSupport::TestCase
+          def test_rikka
+            puts 'Rikka'
+          end
+        end
+      RUBY
+
+      run_test_command("my_test.rb").tap do |output|
+        assert_match "Rikka", output
+      end
+    end
+
+    def test_run_test_having_a_slash_in_its_name
+      app_file "my_test.rb", <<-RUBY
+        require "test_helper"
+
+        class MyTest < ActiveSupport::TestCase
+          test "foo/foo" do
+            puts 'Rikka'
+          end
+        end
+      RUBY
+
+      run_test_command("my_test.rb -n foo\/foo").tap do |output|
+        assert_match "Rikka", output
+      end
+    end
+
+    def test_run_test_with_flags_unordered
+      app_file "my_test.rb", <<-RUBY
+        require "test_helper"
+
+        class MyTest < ActiveSupport::TestCase
+          test "foo/foo" do
+            puts 'Rikka'
+          end
+        end
+      RUBY
+
+      run_test_command("--seed 344 my_test.rb --fail-fast -n foo\/foo").tap do |output|
+        assert_match "Rikka", output
+      end
+    end
+
     def test_run_matched_test
       app_file "test/unit/chu_2_koi_test.rb", <<-RUBY
         require "test_helper"