From 261eea422a276dbc1fbfeae5b72cf32efd64b076 Mon Sep 17 00:00:00 2001 From: Sampo Kuokkanen Date: Wed, 27 May 2026 13:00:24 +0900 Subject: [PATCH] Fix tautological assertions in test_each_value in test_hash.rb Both assertions tested `[].length == 0` rather than `res.length`, so the test passed even when `each_value` yielded nothing. --- test/ruby/test_hash.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/ruby/test_hash.rb b/test/ruby/test_hash.rb index 32384f5a5c..2d1b513c70 100644 --- a/test/ruby/test_hash.rb +++ b/test/ruby/test_hash.rb @@ -465,10 +465,10 @@ class TestHash < Test::Unit::TestCase def test_each_value res = [] @cls[].each_value { |v| res << v } - assert_equal(0, [].length) + assert_equal(0, res.length) @h.each_value { |v| res << v } - assert_equal(0, [].length) + assert_equal(@h.size, res.length) expected = [] @h.each { |k, v| expected << v }