Add tests for Kernel#respond_to? without #respond_to_missing?

trinistr · trinistr · commit 7778e3e4f310 · 2026-03-03T22:11:39.000+03:00
`#respond_to?` should work with unknown names even if `#respond_to_missing?` is not defined.
diff --git a/core/io/buffer/clear_spec.rb b/core/io/buffer/clear_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#clear" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/comparison_spec.rb b/core/io/buffer/comparison_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#<=>" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/constants_spec.rb b/core/io/buffer/constants_spec.rb
@@ -0,0 +1,4 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer::BIG_ENDIAN" do
+end
diff --git a/core/io/buffer/copy_spec.rb b/core/io/buffer/copy_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#copy" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/each_byte_spec.rb b/core/io/buffer/each_byte_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#each_byte" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/each_spec.rb b/core/io/buffer/each_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#each" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/get_string_spec.rb b/core/io/buffer/get_string_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#get_string" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/get_value_spec.rb b/core/io/buffer/get_value_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#get_value" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/get_values_spec.rb b/core/io/buffer/get_values_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#get_values" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/hexdump_spec.rb b/core/io/buffer/hexdump_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#hexdump" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/initialize_copy_spec.rb b/core/io/buffer/initialize_copy_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#initialize_copy" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/inspect_spec.rb b/core/io/buffer/inspect_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#inspect" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/match_spec.rb b/core/io/buffer/match_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#~" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/pread_spec.rb b/core/io/buffer/pread_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#pread" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/pwrite_spec.rb b/core/io/buffer/pwrite_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#pwrite" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/read_spec.rb b/core/io/buffer/read_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#read" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/set_string_spec.rb b/core/io/buffer/set_string_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#set_string" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/set_value_spec.rb b/core/io/buffer/set_value_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#set_value" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/set_values_spec.rb b/core/io/buffer/set_values_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#set_values" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/size_of_spec.rb b/core/io/buffer/size_of_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer.size_of" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/size_spec.rb b/core/io/buffer/size_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#size" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/slice_spec.rb b/core/io/buffer/slice_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#slice" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/to_s_spec.rb b/core/io/buffer/to_s_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#to_s" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/values_spec.rb b/core/io/buffer/values_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#values" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/io/buffer/write_spec.rb b/core/io/buffer/write_spec.rb
@@ -0,0 +1,5 @@
+require_relative '../../../spec_helper'
+
+describe "IO::Buffer#write" do
+  it "needs to be reviewed for spec completeness"
+end
diff --git a/core/kernel/fixtures/classes.rb b/core/kernel/fixtures/classes.rb
@@ -180,6 +180,25 @@ class B < A
     alias aliased_pub_method pub_method
   end
 
+  class BasicA < BasicObject
+    define_method(:respond_to?, ::Kernel.instance_method(:respond_to?))
+
+    def pub_method; :public_method; end
+
+    def undefed_method; :undefed_method; end
+    undef_method :undefed_method
+
+    protected
+    def protected_method; :protected_method; end
+
+    private
+    def private_method; :private_method; end
+  end
+
+  class MissingA < A
+    undef :respond_to_missing?
+  end
+
   class VisibilityChange
     class << self
       private :new
diff --git a/core/kernel/respond_to_spec.rb b/core/kernel/respond_to_spec.rb
@@ -69,4 +69,31 @@ class KernelSpecs::Foo; def bar; 'done'; end; end
     KernelSpecs::Foo.new.respond_to?(:bar).should == true
     KernelSpecs::Foo.new.respond_to?(:invalid_and_silly_method_name).should == false
   end
+
+  context "if object does not have #respond_to_missing?" do
+    it "returns true if object responds to the given public method" do
+      KernelSpecs::BasicA.new.respond_to?(:pub_method).should == true
+      KernelSpecs::MissingA.new.respond_to?(:pub_method).should == true
+    end
+
+    it "returns false if object responds to the given protected method" do
+      KernelSpecs::BasicA.new.respond_to?(:protected_method).should == false
+      KernelSpecs::MissingA.new.respond_to?(:protected_method).should == false
+    end
+
+    it "returns false if object responds to the given private method" do
+      KernelSpecs::BasicA.new.respond_to?(:private_method).should == false
+      KernelSpecs::MissingA.new.respond_to?(:private_method).should == false
+    end
+
+    it "returns false if the given method was undefined" do
+      KernelSpecs::BasicA.new.respond_to?(:undefed_method).should == false
+      KernelSpecs::MissingA.new.respond_to?(:undefed_method).should == false
+    end
+
+    it "returns false if the given method never existed" do
+      KernelSpecs::BasicA.new.respond_to?(:invalid_and_silly_method_name).should == false
+      KernelSpecs::MissingA.new.respond_to?(:invalid_and_silly_method_name).should == false
+    end
+  end
 end

-Original file line number
+Diff line change
@@ @@ -0,0 +1,4 @@ @@
 +require_relative '../../../spec_helper'
++
 +describe "IO::Buffer::BIG_ENDIAN" do
 +end