diff options
author | David RodrÃguez <deivid.rodriguez@riseup.net> | 2023-10-26 22:11:34 +0200 |
---|---|---|
committer | Hiroshi SHIBATA <hsbt@ruby-lang.org> | 2023-11-13 11:06:10 +0900 |
commit | 54511303a46901a82e9b2d4ee460b2048e59a2e5 (patch) | |
tree | 3a465cce48ee0228b455a6810ac8cc026b1bded2 /test | |
parent | 435eb56f6175b7c9a16121ec8441f7492fa9aec5 (diff) |
[rubygems/rubygems] Drop support for Ruby 2.6 and Ruby 2.7 in RubyGems
https://github.com/rubygems/rubygems/commit/10c26a483d
Diffstat (limited to 'test')
-rw-r--r-- | test/rubygems/bundler_test_gem.rb | 4 | ||||
-rw-r--r-- | test/rubygems/helper.rb | 6 | ||||
-rw-r--r-- | test/rubygems/test_gem.rb | 4 | ||||
-rw-r--r-- | test/rubygems/test_gem_command_manager.rb | 2 | ||||
-rw-r--r-- | test/rubygems/test_gem_package.rb | 4 | ||||
-rw-r--r-- | test/rubygems/test_gem_safe_marshal.rb | 2 | ||||
-rw-r--r-- | test/rubygems/test_gem_specification.rb | 22 |
7 files changed, 8 insertions, 36 deletions
diff --git a/test/rubygems/bundler_test_gem.rb b/test/rubygems/bundler_test_gem.rb index 78adfc94c5..319112bac8 100644 --- a/test/rubygems/bundler_test_gem.rb +++ b/test/rubygems/bundler_test_gem.rb @@ -3,7 +3,7 @@ require_relative "helper" class TestBundlerGem < Gem::TestCase - PROJECT_DIR = File.expand_path("../..", __dir__).tap(&Gem::UNTAINT) + PROJECT_DIR = File.expand_path("../..", __dir__) def test_self_use_gemdeps with_local_bundler_at(Gem.dir) do @@ -221,7 +221,7 @@ class TestBundlerGem < Gem::TestCase def test_use_gemdeps with_local_bundler_at(Gem.dir) do - gem_deps_file = "gem.deps.rb".tap(&Gem::UNTAINT) + gem_deps_file = "gem.deps.rb" spec = util_spec "a", 1 install_specs spec diff --git a/test/rubygems/helper.rb b/test/rubygems/helper.rb index b55e526d77..d04ed3dc8e 100644 --- a/test/rubygems/helper.rb +++ b/test/rubygems/helper.rb @@ -295,7 +295,6 @@ class Gem::TestCase < Test::Unit::TestCase FileUtils.mkdir_p @tmp @tempdir = Dir.mktmpdir("test_rubygems_", @tmp) - @tempdir.tap(&Gem::UNTAINT) ENV["GEM_VENDOR"] = nil ENV["GEMRC"] = nil @@ -345,7 +344,6 @@ class Gem::TestCase < Test::Unit::TestCase File.expand_path(s) end if expand_path != s - expand_path.tap(&Gem::UNTAINT) if s.instance_variable_defined?(:@gem_prelude_index) expand_path.instance_variable_set(:@gem_prelude_index, expand_path) end @@ -606,7 +604,7 @@ class Gem::TestCase < Test::Unit::TestCase end end - gem = File.join(@tempdir, File.basename(gem)).tap(&Gem::UNTAINT) + gem = File.join(@tempdir, File.basename(gem)) end Gem::Installer.at(gem, options.merge({ :wrappers => true })).install @@ -645,7 +643,7 @@ class Gem::TestCase < Test::Unit::TestCase # Reads a Marshal file at +path+ def read_cache(path) - File.open path.dup.tap(&Gem::UNTAINT), "rb" do |io| + File.open path.dup, "rb" do |io| Marshal.load io.read end end diff --git a/test/rubygems/test_gem.rb b/test/rubygems/test_gem.rb index a7174cf689..6537afad2a 100644 --- a/test/rubygems/test_gem.rb +++ b/test/rubygems/test_gem.rb @@ -11,7 +11,7 @@ require "rbconfig" class TestGem < Gem::TestCase PLUGINS_LOADED = [] # rubocop:disable Style/MutableConstant - PROJECT_DIR = File.expand_path("../..", __dir__).tap(&Gem::UNTAINT) + PROJECT_DIR = File.expand_path("../..", __dir__) def setup super @@ -201,7 +201,7 @@ class TestGem < Gem::TestCase end assert_equal(expected, result) ensure - File.chmod(0o755, *Dir.glob(@gemhome + "/gems/**/").map {|path| path.tap(&Gem::UNTAINT) }) + File.chmod(0o755, *Dir.glob(@gemhome + "/gems/**/")) end def test_require_missing diff --git a/test/rubygems/test_gem_command_manager.rb b/test/rubygems/test_gem_command_manager.rb index fe7f3da137..f04ec0cafa 100644 --- a/test/rubygems/test_gem_command_manager.rb +++ b/test/rubygems/test_gem_command_manager.rb @@ -4,7 +4,7 @@ require_relative "helper" require "rubygems/command_manager" class TestGemCommandManager < Gem::TestCase - PROJECT_DIR = File.expand_path("../..", __dir__).tap(&Gem::UNTAINT) + PROJECT_DIR = File.expand_path("../..", __dir__) def setup super diff --git a/test/rubygems/test_gem_package.rb b/test/rubygems/test_gem_package.rb index 68dbf119a4..2065864107 100644 --- a/test/rubygems/test_gem_package.rb +++ b/test/rubygems/test_gem_package.rb @@ -760,12 +760,10 @@ class TestGemPackage < Gem::Package::TarTestCase package = Gem::Package.new @gem file = "file.rb".dup - file.taint if RUBY_VERSION < "2.7" destination = package.install_location file, @destination assert_equal File.join(@destination, "file.rb"), destination - refute destination.tainted? if RUBY_VERSION < "2.7" end def test_install_location_absolute @@ -799,12 +797,10 @@ class TestGemPackage < Gem::Package::TarTestCase package = Gem::Package.new @gem file = "foo//file.rb".dup - file.taint if RUBY_VERSION < "2.7" destination = package.install_location file, @destination assert_equal File.join(@destination, "foo", "file.rb"), destination - refute destination.tainted? if RUBY_VERSION < "2.7" end def test_install_location_relative diff --git a/test/rubygems/test_gem_safe_marshal.rb b/test/rubygems/test_gem_safe_marshal.rb index 362e5e00ad..29685f458c 100644 --- a/test/rubygems/test_gem_safe_marshal.rb +++ b/test/rubygems/test_gem_safe_marshal.rb @@ -212,7 +212,7 @@ class TestGemSafeMarshal < Gem::TestCase Time.at(secs, 1.00001, :nanosecond), Time.at(secs, 1.00001, :nanosecond), ].tap do |times| - unless RUBY_ENGINE == "truffleruby" && RUBY_ENGINE_VERSION < "23" || RUBY_VERSION < "2.7" + unless RUBY_ENGINE == "truffleruby" && RUBY_ENGINE_VERSION < "23" times.concat [ Time.at(secs, in: "UTC"), Time.at(secs, in: "Z"), diff --git a/test/rubygems/test_gem_specification.rb b/test/rubygems/test_gem_specification.rb index 8d86820c71..28697529ef 100644 --- a/test/rubygems/test_gem_specification.rb +++ b/test/rubygems/test_gem_specification.rb @@ -797,28 +797,6 @@ dependencies: [] assert_equal File.join(@tempdir, "a-2.gemspec"), spec.loaded_from end - if RUBY_ENGINE == "ruby" && RUBY_VERSION < "2.7" - def test_self_load_tainted - full_path = @a2.spec_file - write_file full_path do |io| - io.write @a2.to_ruby_for_cache - end - - full_path.taint - loader = Thread.new do - $SAFE = 1 - Gem::Specification.load full_path - end - spec = loader.value - - @a2.files.clear - - assert_equal @a2, spec - ensure - $SAFE = 0 - end - end - def test_self_load_escape_curly @a2.name = 'a};raise "improper escaping";%q{' |