Add skipping for stale tests

Author: danielahrnsbrak

ruby/lib/minitest/queue.rb

@@ -435,7 +435,13 @@ def run
         with_timestamps do
           begin
             return build_error_result(@load_error) if @load_error
-            Minitest.run_one_method(runnable, @method_name)
+
+            klass = runnable
+            unless klass.method_defined?(@method_name) || klass.private_method_defined?(@method_name)
+              return build_stale_skip_result
+            end
+
+            Minitest.run_one_method(klass, @method_name)
           rescue StandardError, ScriptError => error
             build_error_result(error)
           end
@@ -506,6 +512,19 @@ def build_error_result(error)
         result
       end
 
+      def build_stale_skip_result
+        $stderr.puts "[ci-queue] Skipping stale preresolved entry: #{@class_name}##{@method_name} " \
+          "(method no longer exists in #{@file_path || 'unknown file'})"
+
+        result_class = defined?(Minitest::Result) ? Minitest::Result : Minitest::Test
+        result = result_class.new(@method_name)
+        result.klass = @class_name if result.respond_to?(:klass=)
+        result.failures << Minitest::Skip.new(
+          "[ci-queue] Stale preresolved entry: #{@class_name}##{@method_name} no longer exists"
+        )
+        result
+      end
+
       def current_timestamp
         CI::Queue.time_now.to_i
       end

ruby/test/minitest/queue/lazy_single_example_test.rb

@@ -54,6 +54,33 @@ def test_run_handles_script_error
       end
     end
 
+    def test_run_skips_stale_preresolved_entry
+      Dir.mktmpdir do |dir|
+        class_name = "StaleEntry#{Process.pid}#{rand(1000)}"
+        file_path = File.join(dir, "stale_entry_test.rb")
+        File.write(
+          file_path,
+          "class #{class_name} < Minitest::Test\n" \
+          "  def test_exists\n" \
+          "    assert true\n" \
+          "  end\n" \
+          "end\n"
+        )
+
+        loader = CI::Queue::FileLoader.new
+        resolver = CI::Queue::ClassResolver
+        example = LazySingleExample.new(class_name, 'test_no_longer_exists', file_path, loader: loader, resolver: resolver)
+
+        result = example.run
+
+        assert result.skipped?
+        assert_match(/Stale preresolved entry/, result.failure.message)
+        assert_match(/test_no_longer_exists/, result.failure.message)
+      ensure
+        Object.send(:remove_const, class_name) if Object.const_defined?(class_name)
+      end
+    end
+
     def test_marshal_round_trip
       Dir.mktmpdir do |dir|
         class_name = "LazyMarshal#{Process.pid}#{rand(1000)}"