summaryrefslogtreecommitdiff
path: root/test
diff options
context:
space:
mode:
authorDavid Rodríguez <deivid.rodriguez@riseup.net>2023-10-26 22:11:34 +0200
committerHiroshi SHIBATA <hsbt@ruby-lang.org>2023-11-13 11:06:10 +0900
commit54511303a46901a82e9b2d4ee460b2048e59a2e5 (patch)
tree3a465cce48ee0228b455a6810ac8cc026b1bded2 /test
parent435eb56f6175b7c9a16121ec8441f7492fa9aec5 (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.rb4
-rw-r--r--test/rubygems/helper.rb6
-rw-r--r--test/rubygems/test_gem.rb4
-rw-r--r--test/rubygems/test_gem_command_manager.rb2
-rw-r--r--test/rubygems/test_gem_package.rb4
-rw-r--r--test/rubygems/test_gem_safe_marshal.rb2
-rw-r--r--test/rubygems/test_gem_specification.rb22
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{'