Merge pull request #2725 from ruby/fix-ruby-skip-tests

soutaro · web-flow · commit 79a7d462e7ab · 2025-12-03T11:57:12.000+09:00
Fix tests for rbs_skip_tests in ruby repo
diff --git a/core/ractor.rbs b/core/ractor.rbs
@@ -350,68 +350,6 @@ class Ractor
   #
   def self.receive: () -> untyped
 
-  # <!--
-  #   rdoc-file=ractor.rb
-  #   - Ractor.receive_if {|msg| block } -> msg
-  # -->
-  # Receive only a specific message.
-  #
-  # Instead of Ractor.receive, Ractor.receive_if can be given a pattern (or any
-  # filter) in a block and you can choose the messages to accept that are
-  # available in your ractor's incoming queue.
-  #
-  #     r = Ractor.new do
-  #       p Ractor.receive_if{|msg| msg.match?(/foo/)} #=> "foo3"
-  #       p Ractor.receive_if{|msg| msg.match?(/bar/)} #=> "bar1"
-  #       p Ractor.receive_if{|msg| msg.match?(/baz/)} #=> "baz2"
-  #     end
-  #     r << "bar1"
-  #     r << "baz2"
-  #     r << "foo3"
-  #     r.take
-  #
-  # This will output:
-  #
-  #     foo3
-  #     bar1
-  #     baz2
-  #
-  # If the block returns a truthy value, the message is removed from the incoming
-  # queue and returned. Otherwise, the message remains in the incoming queue and
-  # the next messages are checked by the given block.
-  #
-  # If there are no messages left in the incoming queue, the method will block
-  # until new messages arrive.
-  #
-  # If the block is escaped by break/return/exception/throw, the message is
-  # removed from the incoming queue as if a truthy value had been returned.
-  #
-  #     r = Ractor.new do
-  #       val = Ractor.receive_if{|msg| msg.is_a?(Array)}
-  #       puts "Received successfully: #{val}"
-  #     end
-  #
-  #     r.send(1)
-  #     r.send('test')
-  #     wait
-  #     puts "2 non-matching sent, nothing received"
-  #     r.send([1, 2, 3])
-  #     wait
-  #
-  # Prints:
-  #
-  #     2 non-matching sent, nothing received
-  #     Received successfully: [1, 2, 3]
-  #
-  # Note that you can not call receive/receive_if in the given block recursively.
-  # You should not do any tasks in the block other than message filtration.
-  #
-  #     Ractor.current << true
-  #     Ractor.receive_if{|msg| Ractor.receive}
-  #     #=> `receive': can not call receive/receive_if recursively (Ractor::Error)
-  #
-  def self.receive_if: () { (untyped) -> boolish } -> untyped
-
   # <!--
   #   rdoc-file=ractor.rb
   #   - recv()
@@ -425,7 +363,7 @@ class Ractor
   # -->
   # TBD
   #
-  def self.select: (*Ractor ractors, ?move: boolish, ?yield_value: untyped) -> [ Ractor | Symbol, untyped ]
+  def self.select: (?) -> untyped
 
   # <!--
   #   rdoc-file=ractor.rb
@@ -459,46 +397,6 @@ class Ractor
   #
   def self.store_if_absent: [A] (Symbol) { (nil) -> A } -> A
 
-  # <!--
-  #   rdoc-file=ractor.rb
-  #   - Ractor.yield(msg, move: false) -> nil
-  # -->
-  # Send a message to the current ractor's outgoing port to be accepted by #take.
-  #
-  #     r = Ractor.new {Ractor.yield 'Hello from ractor'}
-  #     puts r.take
-  #     # Prints: "Hello from ractor"
-  #
-  # This method is blocking, and will return only when somebody consumes the sent
-  # message.
-  #
-  #     r = Ractor.new do
-  #       Ractor.yield 'Hello from ractor'
-  #       puts "Ractor: after yield"
-  #     end
-  #     wait
-  #     puts "Still not taken"
-  #     puts r.take
-  #
-  # This will print:
-  #
-  #     Still not taken
-  #     Hello from ractor
-  #     Ractor: after yield
-  #
-  # If the outgoing port was closed with #close_outgoing, the method will raise:
-  #
-  #     r = Ractor.new do
-  #       close_outgoing
-  #       Ractor.yield 'Hello from ractor'
-  #     end
-  #     wait
-  #     # `yield': The outgoing-port is already closed (Ractor::ClosedError)
-  #
-  # The meaning of the `move` argument is the same as for #send.
-  #
-  def self.yield: (untyped obj, ?move: boolish) -> untyped
-
   # <!--
   #   rdoc-file=ractor.rb
   #   - <<(...)
@@ -513,6 +411,7 @@ class Ractor
   # get a value from ractor-local storage for current Ractor Obsolete and use
   # Ractor.[] instead.
   #
+  %a{deprecated: Use Ractor.[] instead}
   def []: (interned sym) -> untyped
 
   # <!--
@@ -522,40 +421,9 @@ class Ractor
   # set a value in ractor-local storage for current Ractor Obsolete and use
   # Ractor.[]= instead.
   #
+  %a{deprecated: Use Ractor.[]= instead}
   def []=: [T] (interned sym, T val) -> T
 
-  # <!--
-  #   rdoc-file=ractor.rb
-  #   - ractor.close_incoming -> true | false
-  # -->
-  # Closes the incoming port and returns whether it was already closed. All
-  # further attempts to Ractor.receive in the ractor, and #send to the ractor will
-  # fail with Ractor::ClosedError.
-  #
-  #     r = Ractor.new {sleep(500)}
-  #     r.close_incoming  #=> false
-  #     r.close_incoming  #=> true
-  #     r.send('test')
-  #     # Ractor::ClosedError (The incoming-port is already closed)
-  #
-  def close_incoming: () -> bool
-
-  # <!--
-  #   rdoc-file=ractor.rb
-  #   - ractor.close_outgoing -> true | false
-  # -->
-  # Closes the outgoing port and returns whether it was already closed. All
-  # further attempts to Ractor.yield in the ractor, and #take from the ractor will
-  # fail with Ractor::ClosedError.
-  #
-  #     r = Ractor.new {sleep(500)}
-  #     r.close_outgoing  #=> false
-  #     r.close_outgoing  #=> true
-  #     r.take
-  #     # Ractor::ClosedError (The outgoing-port is already closed)
-  #
-  def close_outgoing: () -> bool
-
   # <!--
   #   rdoc-file=ractor.rb
   #   - inspect()
@@ -579,73 +447,6 @@ class Ractor
   #
   def send: (untyped obj, ?move: boolish) -> Ractor
 
-  # <!--
-  #   rdoc-file=ractor.rb
-  #   - ractor.take -> msg
-  # -->
-  # Get a message from the ractor's outgoing port, which was put there by
-  # Ractor.yield or at ractor's termination.
-  #
-  #     r = Ractor.new do
-  #       Ractor.yield 'explicit yield'
-  #       'last value'
-  #     end
-  #     puts r.take #=> 'explicit yield'
-  #     puts r.take #=> 'last value'
-  #     puts r.take # Ractor::ClosedError (The outgoing-port is already closed)
-  #
-  # The fact that the last value is also sent to the outgoing port means that
-  # `take` can be used as an analog of Thread#join ("just wait until ractor
-  # finishes"). However, it will raise if somebody has already consumed that
-  # message.
-  #
-  # If the outgoing port was closed with #close_outgoing, the method will raise
-  # Ractor::ClosedError.
-  #
-  #     r = Ractor.new do
-  #       sleep(500)
-  #       Ractor.yield 'Hello from ractor'
-  #     end
-  #     r.close_outgoing
-  #     r.take
-  #     # Ractor::ClosedError (The outgoing-port is already closed)
-  #     # The error would be raised immediately, not when ractor will try to receive
-  #
-  # If an uncaught exception is raised in the Ractor, it is propagated by take as
-  # a Ractor::RemoteError.
-  #
-  #     r = Ractor.new {raise "Something weird happened"}
-  #
-  #     begin
-  #       r.take
-  #     rescue => e
-  #       p e              #  => #<Ractor::RemoteError: thrown by remote Ractor.>
-  #       p e.ractor == r  # => true
-  #       p e.cause        # => #<RuntimeError: Something weird happened>
-  #     end
-  #
-  # Ractor::ClosedError is a descendant of StopIteration, so the termination of
-  # the ractor will break out of any loops that receive this message without
-  # propagating the error:
-  #
-  #     r = Ractor.new do
-  #       3.times {|i| Ractor.yield "message #{i}"}
-  #       "finishing"
-  #     end
-  #
-  #     loop {puts "Received: " + r.take}
-  #     puts "Continue successfully"
-  #
-  # This will print:
-  #
-  #     Received: message 0
-  #     Received: message 1
-  #     Received: message 2
-  #     Received: finishing
-  #     Continue successfully
-  #
-  def take: () -> untyped
-
   # <!--
   #   rdoc-file=ractor.rb
   #   - to_s()
diff --git a/test/stdlib/Pathname_test.rb b/test/stdlib/Pathname_test.rb
@@ -27,13 +27,11 @@ def test_pwd
                      Pathname, :pwd
   end
 
-  def test_initialize
-    omit "Pathname is developing in head" if RUBY_VERSION >= '3.5'
-
+  def test_new
     assert_send_type '(String) -> Pathname',
                      Pathname, :new, 'foo'
     assert_send_type '(ToStr) -> Pathname',
-                     Pathname, :new, ToStr.new('foo')
+                     Pathname, :new, ToStr.new('foo').__with_object_methods(:respond_to?)
     assert_send_type '(Pathname) -> Pathname',
                      Pathname, :new, Pathname('foo')
   end
@@ -45,25 +43,21 @@ class PathnameInstanceTest < Test::Unit::TestCase
   testing '::Pathname'
 
   def test_plus
-    omit "Pathname is developing in head" if RUBY_VERSION >= '3.5'
-
     assert_send_type '(Pathname) -> Pathname',
                      Pathname('foo'), :+, Pathname('bar')
     assert_send_type '(String) -> Pathname',
                      Pathname('foo'), :+, 'bar'
     assert_send_type '(ToStr) -> Pathname',
-                     Pathname('foo'), :+, ToStr.new('bar')
+                     Pathname('foo'), :+, ToStr.new('bar').__with_object_methods(:respond_to?)
   end
 
   def test_slash
-    omit "Pathname is developing in head" if RUBY_VERSION >= '3.5'
-
     assert_send_type '(Pathname) -> Pathname',
                      Pathname('foo'), :/, Pathname('bar')
     assert_send_type '(String) -> Pathname',
                      Pathname('foo'), :/, 'bar'
     assert_send_type '(ToStr) -> Pathname',
-                     Pathname('foo'), :/, ToStr.new('bar')
+                     Pathname('foo'), :/, ToStr.new('bar').__with_object_methods(:respond_to?)
   end
 
   def test_spaceship
@@ -386,16 +380,14 @@ def test_inspect
   end
 
   def test_join
-    omit "Pathname is developing in head" if RUBY_VERSION >= '3.5'
-
     assert_send_type '() -> Pathname',
                      Pathname('.'), :join
     assert_send_type '(String) -> Pathname',
                      Pathname('.'), :join, 'foo'
     assert_send_type '(String, String) -> Pathname',
                      Pathname('.'), :join, 'foo', 'bar'
     assert_send_type '(ToStr) -> Pathname',
-                     Pathname('.'), :join, ToStr.new('foo')
+                     Pathname('.'), :join, ToStr.new('foo').__with_object_methods(:respond_to?)
     assert_send_type '(Pathname) -> Pathname',
                      Pathname('.'), :join, Pathname('foo')
   end
@@ -610,15 +602,13 @@ def test_relative?
   end
 
   def test_relative_path_from
-    omit "Pathname is developing in head" if RUBY_VERSION >= '3.5'
-
     assert_send_type '(Pathname) -> Pathname',
                      Pathname('.'), :relative_path_from, Pathname('.')
     assert_send_type '(String) -> Pathname',
                      Pathname('.'), :relative_path_from, '.'
 
     assert_send_type '(_ToStr) -> Pathname',
-                     Pathname('.'), :relative_path_from, ToStr.new('.').__with_object_methods(:is_a?)
+                     Pathname('.'), :relative_path_from, ToStr.new('.').__with_object_methods(:is_a?, :respond_to?)
   end
 
   def test_rename
@@ -834,14 +824,15 @@ class PathnameKernelTest < Test::Unit::TestCase
   testing '::Kernel'
 
   def test_Pathname
-    omit "Pathname is developing in head" if RUBY_VERSION >= '3.5'
+    assert_send_type(
+      "(::String) -> ::Pathname",
+      self, :Pathname, "Gemfile"
+    )
 
-    with_string("Gemfile") do
-      assert_send_type(
-        "(::string) -> ::Pathname",
-        self, :Pathname, _1
-      )
-    end
+    assert_send_type(
+      "(::_ToStr) -> ::Pathname",
+      self, :Pathname, ToStr.new("Gemfile").__with_object_methods(:respond_to?)
+    )
 
     assert_send_type(
       "(::Pathname) -> ::Pathname",
diff --git a/test/stdlib/Ractor_test.rb b/test/stdlib/Ractor_test.rb
