1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[ruby/tempfile] Make Tempfile#open return the underlying File

Browse Source 
Add test for this behavior.

https://github.com/ruby/tempfile/commit/0ca31a6b8d
This commit is contained in:
Jeremy Evans 2023-11-08 08:35:16 -08:00 committed by git
parent eb2abc3f16
commit ddd99a5290
2 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
lib/tempfile.rb
View File

@ -191,6 +191,7 @@ class Tempfile < DelegateClass(File)
mode = @mode & ~(File::CREAT|File::EXCL) mode = @mode & ~(File::CREAT|File::EXCL)
__setobj__(File.open(__getobj__.path, mode, **@opts)) __setobj__(File.open(__getobj__.path, mode, **@opts))
ObjectSpace.define_finalizer(self, Closer.new(__getobj__)) ObjectSpace.define_finalizer(self, Closer.new(__getobj__))
__getobj__
end end
def _close # :nodoc: def _close # :nodoc:

8
test/test_tempfile.rb
View File

@ -378,6 +378,14 @@ puts Tempfile.new('foo').path
assert_file.not_exist?(path) assert_file.not_exist?(path)
end end
def test_open
Tempfile.open {|f|
file = f.open
assert_kind_of File, file
assert_equal f.to_i, file.to_i
}
end
def test_open_traversal_dir def test_open_traversal_dir
assert_mktmpdir_traversal do |traversal_path| assert_mktmpdir_traversal do |traversal_path|
t = Tempfile.open([traversal_path, 'foo']) t = Tempfile.open([traversal_path, 'foo'])