diff options
Diffstat (limited to 'test/rubygems/test_gem_package.rb')
| -rw-r--r-- | test/rubygems/test_gem_package.rb | 26 |
1 files changed, 16 insertions, 10 deletions
diff --git a/test/rubygems/test_gem_package.rb b/test/rubygems/test_gem_package.rb index 7848bc2428..2f8747bc97 100644 --- a/test/rubygems/test_gem_package.rb +++ b/test/rubygems/test_gem_package.rb @@ -1,5 +1,5 @@ # coding: UTF-8 -# frozen_string_literal: false +# frozen_string_literal: true require 'rubygems/package/tar_test_case' require 'rubygems/simple_gem' @@ -428,19 +428,25 @@ class TestGemPackage < Gem::Package::TarTestCase "#{@destination} is not allowed", e.message) end - def test_extract_tar_gz_symlink_absolute + def test_extract_tar_gz_symlink_relative_path + skip 'symlink not supported' if Gem.win_platform? + package = Gem::Package.new @gem tgz_io = util_tar_gz do |tar| - tar.add_symlink 'code.rb', '/absolute.rb', 0644 + tar.add_file 'relative.rb', 0644 do |io| io.write 'hi' end + tar.mkdir 'lib', 0755 + tar.add_symlink 'lib/foo.rb', '../relative.rb', 0644 end - e = assert_raises Gem::Package::PathError do - package.extract_tar_gz tgz_io, @destination - end + package.extract_tar_gz tgz_io, @destination - assert_equal("installing into parent path /absolute.rb of " + - "#{@destination} is not allowed", e.message) + extracted = File.join @destination, 'lib/foo.rb' + assert_path_exists extracted + assert_equal '../relative.rb', + File.readlink(extracted) + assert_equal 'hi', + File.read(extracted) end def test_extract_tar_gz_directory @@ -490,7 +496,7 @@ class TestGemPackage < Gem::Package::TarTestCase def test_install_location package = Gem::Package.new @gem - file = 'file.rb' + file = 'file.rb'.dup file.taint destination = package.install_location file, @destination @@ -530,7 +536,7 @@ class TestGemPackage < Gem::Package::TarTestCase skip 'no File.realpath on 1.8' if RUBY_VERSION < '1.9' package = Gem::Package.new @gem - file = 'foo//file.rb' + file = 'foo//file.rb'.dup file.taint destination = @destination.sub '/', '//' |