file-temp 1.7.0 → 1.7.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 720ea1ce81f51967d708b7cff1756ec572337149a9312f38639654db1cb053ed
-  data.tar.gz: 5e2076ee4cc854a014789b26c1a21a520a1aa83af301de9e0e0677953da2c423
+  metadata.gz: eff786722d8fdd2d775c3bf89a93256c3e6cb52a73966d13aa5f5520c37c96ee
+  data.tar.gz: 57637cb99fbb42afa24779c95527faeedf0a0ff304fc0eba3de517912377459e
 SHA512:
-  metadata.gz: fe66533b78b824b96cb75e03632ef5dc94ee4df4bedbb1f717d1fdb0c3a00bcd8b768bdc645d6453fd4b8ba5ff5ea739a38d128c472d9da95fc14f1cd5b104ec
-  data.tar.gz: 249034641f45d4523ea15ee6c64195fdae53ec0727305bd11fed655463c2a89dbb92510f1c281bdadd3357039777b15255f35da7d08f756ed6d5241d5689acba
+  metadata.gz: 35e334372e93fec4e19687f353262910bc7b77e7ccde89d48f01c92f86c85e329efcc00363bc7689447b67230dce4b50047dc8390ab06586e99bdd5c9fef55a2
+  data.tar.gz: 7443857009f786d4c8d460a1e2d3924f5deb1d6074c5f5813c39a4138e40e8e88132a5ec36152718fba2484d234cdc9c40f73fb98a15694aafddda07e89480c0

data/{CHANGES.rdoc → CHANGES.md}

@@ -1,13 +1,23 @@
-= 1.7.0 - 1-Jul-2020
+## 1.7.2 - 11-Jul-2023
+* Fixed a bug in the Windows version where it would sometimes fail.
+* Dropped JRuby from the test matrix for now because of a bug.
+  See https://github.com/jruby/jruby/issues/7847 for more details.
+
+## 1.7.1 - 28-Dec-2020
+* Updated file option kwarg handling so that it's compatible with Ruby 3.x.
+* Switched from rdoc to markdown format since github isn't rendering rdoc properly.
+* Fixed up one of the specs.
+
+## 1.7.0 - 1-Jul-2020
 * Replaced test-unit with rspec, and updated the tests.
 * Updated ffi dependency to 1.1.x.
 * Removed some archiving options from the Rakefile that I wasn't using.
 
-= 1.6.0 - 9-May-2020
+## 1.6.0 - 9-May-2020
 * Added an +options+ argument that is passed along as options to the the
   underlying File.new constructor.
 
-= 1.5.0 - 8-May-2020
+## 1.5.0 - 8-May-2020
 * Switched to keyword arguments.
 * Added the ability to specify a tmp directory.
 * Updates to the JRuby version, removed some code that no longer worked, and
@@ -15,13 +25,13 @@
 * Added a LICENSE file to the distro as part of the Apache-2.0 requirement.
 * Added explicit .rdoc extensions to the README, CHANGES and MANIFEST files.
 
-= 1.4.0 - 12-Oct-2019
+## 1.4.0 - 12-Oct-2019
 * Changed license to Apache-2.0.
 * VERSION constant now lives in a single place, and is frozen.
 * Added metadata to the gemspec.
 * Updated cert.
 
-= 1.3.0 - 9-Apr-2016
+## 1.3.0 - 9-Apr-2016
 * This gem is now signed.
 * Added a file-temp.rb file for convenience.
 * The gem tasks in the Rakefile now assume Rubygems 2.x.
@@ -31,7 +41,7 @@
   are each in their own directory and have their own versions for ease
   of maintenance.
 
-= 1.2.1 - 17-Feb-2014
+## 1.2.1 - 17-Feb-2014
 * There is now a completely separate implementation for JRuby that uses the
   Java API instead of C. It uses a GUID to create the temporary file name
   instead of the 'XXXXXX' template, but is otherwise identical in function.
@@ -44,53 +54,53 @@
 * Use tmpnam_s on Windows instead of tmpnam.
 * Use wide character functions on Windows.
 
-= 1.2.0 - 10-Apr-2012
+## 1.2.0 - 10-Apr-2012
 * Removed the old FileTemp alias for File::Temp. It was deprecated and
   has now been officially removed.
 * Some refactoring of the custom internal Windows functions.
 * Nicer error handling if certain FFI functions fail.
 * Made the FFI functions private.
 
-= 1.1.5 - 17-Jul-2011
+## 1.1.5 - 17-Jul-2011
 * Now stores file path information if the file is retained on the filesystem.
   Thanks go to joerixaop for the patch.
 * The TMPDIR fallback determination on Windows is no longer hard coded.
 
-= 1.1.4 - 16-Sep-2010
+## 1.1.4 - 16-Sep-2010
 * The File::Temp.temp_name method has been altered on Unix systems. It
   no longer prefixes TMPDIR to the name since it was redundant and could
   generate a bogus path.
 * Set the license to Artistic 2.0.
 * Set the test task as the default Rake task.
 
-= 1.1.3 - 14-Sep-2010
+## 1.1.3 - 14-Sep-2010
 * Fixed potential libc linker failure.
 
-= 1.1.2 - 28-Apr-2010
+## 1.1.2 - 28-Apr-2010
 * Explicitly link against libc for Unix versions.
 * Refactored the Rakefile. An old install task was removed and the gem
   related tasks were placed under the 'gem' namespace.
 
-= 1.1.1 - 24-Oct-2009
+## 1.1.1 - 24-Oct-2009
 * Removed the 'use' library as a dependency.
 
-= 1.1.0 - 21-Oct-2009
+## 1.1.0 - 21-Oct-2009
 * Now pure Ruby, using FFI.
 * Fixed RF Bug #26757 - FILE pointer leak. Thanks go to Eric Wong for the spot.
 * Renamed and refactored the test file slightly.
 * Updated the gemspec.
 
-= 1.0.0 - 12-Apr-2008
+## 1.0.0 - 12-Apr-2008
 * Added security via umask().
 * Version bump to 1.0.0.
 
-= 0.1.2 - 6-Jun-2007
+## 0.1.2 - 6-Jun-2007
 * Gemspec fix (forgot the temp.h file - oops).
 * Added an extra test.
  
-= 0.1.1 - 2-Jun-2007
+## 0.1.1 - 2-Jun-2007
 * Core code and test case now work properly on MS Windows.
 * Now uses MS VC++ 8 functions when available (tmpfile_s, _sopen_s).
 
-= 0.1.0 - 1-Jun-2007
+## 0.1.0 - 1-Jun-2007
 * Initial release.

data/Gemfile

@@ -0,0 +1,2 @@
+source 'https://rubygems.org'
+gemspec

data/{MANIFEST.rdoc → MANIFEST.md}

@@ -1,7 +1,7 @@
-* CHANGES
+* CHANGES.md
 * LICENSE
-* MANIFEST
-* README
+* MANIFEST.md
+* README.md
 * Rakefile
 * file-temp.gemspec
 * certs/djberg96_pub.pem

data/{README.rdoc → README.md}

@@ -1,29 +1,36 @@
-== Description
+[![Ruby](https://github.com/djberg96/file-temp/actions/workflows/ruby.yml/badge.svg)](https://github.com/djberg96/file-temp/actions/workflows/ruby.yml)
+
+## Description
 The file-temp library is an alternate way to handle tempfile generation.
 
-== Requirements
+## Requirements
 * ffi 1.1.0 or later
 
-== Installation
-   gem install file-temp
+## Installation
+`gem install file-temp`
+
+## Adding the trusted cert
+`gem cert --add <(curl -Ls https://raw.githubusercontent.com/djberg96/file-temp/main/certs/djberg96_pub.pem)`
 
-== Synopsis
-   require 'file/temp'
+## Synopsis
+```ruby
+require 'file/temp'
 
-   fh = File::Temp.new
-   fh.puts "hello"
-   fh.close # => Tempfile automatically deleted
+fh = File::Temp.new
+fh.puts "hello"
+fh.close # => Tempfile automatically deleted
 
-   fh = File::Temp.new(delete: false)
-   fh.puts "world"
-   fh.close # => Tempfile still on your filesystem
+fh = File::Temp.new(delete: false)
+fh.puts "world"
+fh.close # => Tempfile still on your filesystem
+```
 
-== Motivation
+## Motivation
 Unlike the tempfile library that ships with Ruby's standard library, this
-library uses your system's native +tmpfile+ or +mkstemp+ functions.
+library uses your system's native `tmpfile` or `mkstemp` functions.
    
 This library is also more secure because it restricts file permission via
-+umask+ for files created with +mkstemp+.
+`umask` for files created with `mkstemp`.
 
 Finally, this library subclasses the File class. This means you get almost
 exactly the same interface as the File class. The only difference is the
@@ -34,33 +41,37 @@ the tempfile library, but that was a very long time ago, and my assumption
 at this point is that it's no longer true. But, I still prefer this version
 to the one in the Ruby standard library.
 
-== JRuby
+## JRuby
 The implementation for JRuby uses the Java API, not the C API. The
 temporary file name generated by Java is different than the C version,
 since it uses a GUID instead of the 'XXXXXX' template, but the
 interface is otherwise identical.
 
-== MS Windows
+As of version 1.7.2 there is a known issue with using File::Temp.open
+vs File::Temp.new. See https://github.com/jruby/jruby/issues/7847 for
+more details.
+
+## MS Windows
 You may need to use the mingw build in order to use this library.
 
-Also, there was a bug with the +GetTempFileName+ function on certain versions
-of MS Windows that may cause a failure. However, if you'r patched up you
+Also, there was a bug with the `GetTempFileName` function on certain versions
+of MS Windows that may cause a failure. However, if you're patched up you
 should not see it. 
 
-== License
+## License
 Apache-2.0
 
-== Copyright
-(C) 2007-2020 Daniel J. Berger
+## Copyright
+(C) 2007-2023 Daniel J. Berger
 All Rights Reserved
 
-== Warranty
+## Warranty
 This library is provided "as is" and without any express or
 implied warranties, including, without limitation, the implied
 warranties of merchantability and fitness for a particular purpose.
 
-== Author
+## Author
 Daniel J. Berger
 
-== See also
+## See also
 tmpfile(), mkstemp(), tmpnam()

data/Rakefile

@@ -1,17 +1,18 @@
 require 'rake'
 require 'rake/clean'
 require 'rspec/core/rake_task'
+require 'rubocop/rake_task'
 
 CLEAN.include('**/*.tar', '**/*.zip', '**/*.gz', '**/*.bz2')
-CLEAN.include('**/*.rbc', '**/*.gem', '**/*.tmp')
+CLEAN.include('**/*.rbc', '**/*.gem', '**/*.tmp', '**/*.lock')
 
 namespace 'gem' do
   desc 'Create the file-temp gem'
   task :create => [:clean] do
     require 'rubygems/package'
-    spec = eval(IO.read('file-temp.gemspec'))
+    spec = Gem::Specification.load('file-temp.gemspec')
     spec.signing_key = File.join(Dir.home, '.ssh', 'gem-private_key.pem')
-    Gem::Package.build(spec, true)
+    Gem::Package.build(spec)
   end
 
   desc 'Install the file-temp gem'
@@ -21,6 +22,8 @@ namespace 'gem' do
   end
 end
 
+RuboCop::RakeTask.new
+
 desc 'Run the test suite for the file-temp library'
 RSpec::Core::RakeTask.new(:spec)
 

data/file-temp.gemspec

@@ -2,7 +2,7 @@ require 'rubygems'
 
 Gem::Specification.new do |spec|
   spec.name       = 'file-temp'
-  spec.version    = '1.7.0'
+  spec.version    = '1.7.2'
   spec.author     = 'Daniel J. Berger'
   spec.license    = 'Apache-2.0'
   spec.email      = 'djberg96@gmail.com'
@@ -12,19 +12,20 @@ Gem::Specification.new do |spec|
   spec.files      = Dir['**/*'].delete_if{ |item| item.include?('git') }
   spec.cert_chain = Dir['certs/*']
 
-  spec.extra_rdoc_files = ['CHANGES.rdoc', 'README.rdoc', 'MANIFEST.rdoc']
-
   spec.add_dependency('ffi', '~> 1.1')
   spec.add_development_dependency('rspec', '~> 3.9')
+  spec.add_development_dependency('rubocop')
+  spec.add_development_dependency('rubocop-rspec')
   spec.add_development_dependency('rake')
 
   spec.metadata = {
-    'homepage_uri'      => 'https://github.com/djberg96/file-temp',
-    'bug_tracker_uri'   => 'https://github.com/djberg96/file-temp/issues',
-    'changelog_uri'     => 'https://github.com/djberg96/file-temp/blob/master/CHANGES',
-    'documentation_uri' => 'https://github.com/djberg96/file-temp/wiki',
-    'source_code_uri'   => 'https://github.com/djberg96/file-temp',
-    'wiki_uri'          => 'https://github.com/djberg96/file-temp/wiki'
+    'homepage_uri'          => 'https://github.com/djberg96/file-temp',
+    'bug_tracker_uri'       => 'https://github.com/djberg96/file-temp/issues',
+    'changelog_uri'         => 'https://github.com/djberg96/file-temp/blob/main/CHANGES.md',
+    'documentation_uri'     => 'https://github.com/djberg96/file-temp/wiki',
+    'source_code_uri'       => 'https://github.com/djberg96/file-temp',
+    'wiki_uri'              => 'https://github.com/djberg96/file-temp/wiki',
+    'rubygems_mfa_required' => 'true'
   }
 
   spec.description = <<-EOF

data/lib/file/java/temp.rb

@@ -1,9 +1,11 @@
+# frozen_string_literal: true
+
 require 'java'
-import java.lang.System
 
+# The File::Temp class encapsulates temporary files. It is a subclass of File.
 class File::Temp < File
   # The temporary directory used on MS Windows or Unix.
-  TMPDIR = java.lang.System.getProperties["java.io.tmpdir"]
+  TMPDIR = java.lang.System.getProperties['java.io.tmpdir']
 
   # The name of the temporary file.
   attr_reader :path
@@ -31,7 +33,7 @@ class File::Temp < File
   #    fh.puts 'hello world'
   #    fh.close
   #
-  def initialize(delete: true, template: 'rb_file_temp_XXXXXX', directory: TMPDIR, options: {})
+  def initialize(delete: true, template: 'rb_file_temp_XXXXXX', directory: TMPDIR, **options)
     raise TypeError unless template.is_a?(String)
 
     # Since Java uses a GUID extension to generate a unique file name
@@ -42,7 +44,7 @@ class File::Temp < File
     # to Errno::EINVAL.
     begin
       @file = java.io.File.createTempFile(template, nil, java.io.File.new(directory))
-    rescue NativeException => err
+    rescue NativeException
       raise SystemCallError.new(22), template # 22 is EINVAL
     end
 
@@ -50,7 +52,7 @@ class File::Temp < File
     options[:mode] ||= 'wb+'
 
     path = @file.getName
-    super(path, options)
+    super(path, **options)
 
     @path = path unless delete
   end
@@ -69,6 +71,6 @@ class File::Temp < File
   #
   def close
     super
-    @file.finalize
+    @file.finalize if @file.respond_to?(:finalize)
   end
 end

data/lib/file/temp.rb

@@ -1,6 +1,8 @@
+# frozen_string_literal: true
+
 class File::Temp < File
   # The version of the file-temp library
-  VERSION = '1.7.0'.freeze
+  VERSION = '1.7.2'
 end
 
 if RUBY_PLATFORM == 'java'

data/lib/file/unix/temp.rb

@@ -1,6 +1,9 @@
+# frozen_string_literal: true
+
 require 'ffi'
 require 'tmpdir'
 
+# The File::Temp class encapsulates temporary files. It is a subclass of File.
 class File::Temp < File
   extend FFI::Library
   ffi_lib FFI::Library::LIBC
@@ -51,7 +54,7 @@ class File::Temp < File
   #    fh.puts 'hello world'
   #    fh.close
   #
-  def initialize(delete: true, template: 'rb_file_temp_XXXXXX', directory: TMPDIR, options: {})
+  def initialize(delete: true, template: 'rb_file_temp_XXXXXX', directory: TMPDIR, **options)
     @fptr = nil
 
     if delete
@@ -77,9 +80,9 @@ class File::Temp < File
     options[:mode] ||= 'wb+'
 
     if delete
-      super(fd, options)
+      super(fd, **options)
     else
-      super(@path, options)
+      super(@path, **options)
     end
   end
 

data/lib/file/windows/temp.rb

@@ -1,9 +1,13 @@
+# frozen_string_literal: true
+
 require 'ffi'
 require 'tmpdir'
 
+# The File::Temp class encapsulates temporary files. It is a subclass of File.
 class File::Temp < File
   extend FFI::Library
   ffi_lib FFI::Library::LIBC
+  ffi_convention :stdcall
 
   # :stopdoc:
 
@@ -11,13 +15,13 @@ class File::Temp < File
 
   attach_function :_close, [:int], :int
   attach_function :fclose, [:pointer], :int
-  attach_function :_fdopen, [:int, :string], :pointer
+  attach_function :_fdopen, %i[int string], :pointer
   attach_function :_fileno, [:pointer], :int
   attach_function :_get_errno, [:pointer], :int
-  attach_function :_open, [:string, :int, :int], :int
-  attach_function :_open_osfhandle, [:long, :int], :int
-  attach_function :tmpnam_s, [:pointer, :size_t], :int
-  attach_function :mktemp_s, :_mktemp_s, [:pointer, :size_t], :int
+  attach_function :_open, %i[string int int], :int
+  attach_function :_open_osfhandle, %i[long int], :int
+  attach_function :tmpnam_s, %i[pointer size_t], :int
+  attach_function :mktemp_s, :_mktemp_s, %i[pointer size_t], :int
 
   private_class_method :_close, :fclose, :_fdopen, :_fileno, :_get_errno
   private_class_method :_open, :_open_osfhandle, :mktemp_s, :tmpnam_s
@@ -25,10 +29,10 @@ class File::Temp < File
   ffi_lib :kernel32
 
   attach_function :CloseHandle, [:long], :bool
-  attach_function :CreateFileW, [:buffer_in, :ulong, :ulong, :pointer, :ulong, :ulong, :ulong], :long
+  attach_function :CreateFileW, %i[buffer_in ulong ulong pointer ulong ulong ulong], :long
   attach_function :DeleteFileW, [:string], :bool
-  attach_function :GetTempPathW, [:ulong, :buffer_out], :ulong
-  attach_function :GetTempFileNameW, [:buffer_in, :string, :uint, :buffer_out], :uint
+  attach_function :GetTempPathW, %i[ulong buffer_out], :ulong
+  attach_function :GetTempFileNameW, %i[buffer_in string uint buffer_out], :uint
 
   private_class_method :_close, :_fdopen, :_open, :_open_osfhandle
   private_class_method :CloseHandle, :CreateFileW, :DeleteFileW
@@ -81,7 +85,7 @@ class File::Temp < File
   #    fh.puts 'hello world'
   #    fh.close
   #
-  def initialize(delete: true, template: 'rb_file_temp_XXXXXX', directory: TMPDIR, options: {})
+  def initialize(delete: true, template: 'rb_file_temp_XXXXXX', directory: TMPDIR, **options)
     @fptr = nil
 
     if delete
@@ -105,9 +109,9 @@ class File::Temp < File
     options[:mode] ||= 'wb+'
 
     if delete
-      super(fd, options)
+      super(fd, **options)
     else
-      super(@path, options)
+      super(@path, **options)
     end
   end
 
@@ -134,7 +138,7 @@ class File::Temp < File
 
     raise SystemCallError.new('tmpnam_s', errno) if errno != 0
 
-    directory + ptr.read_string + 'tmp'
+    directory + (ptr.read_string << 'tmp')
   end
 
   private
@@ -153,13 +157,12 @@ class File::Temp < File
   #
   def get_temp_path
     buf = 0.chr * 1024
-    buf.encode!("UTF-16LE")
+    buf.encode!('UTF-16LE')
 
-    if GetTempPathW(buf.size, buf) == 0
-      raise SystemCallError, FFI.errno, 'GetTempPathW'
-    end
+    rv = GetTempPathW(buf.size, buf)
+    raise SystemCallError, FFI.errno, 'GetTempPathW' if rv == 0
 
-    buf.strip.chop # remove trailing slash
+    buf[0, rv+2] # Characters plus null terminator
   end
 
   # The version of tmpfile() implemented by Microsoft is unacceptable.
@@ -171,15 +174,16 @@ class File::Temp < File
   # project.
   #
   def tmpfile
-    file_name = get_temp_path()
+    file_name = get_temp_path
+
     buf = 0.chr * 1024
-    buf.encode!("UTF-16LE")
+    buf.encode!('UTF-16LE')
 
-    if GetTempFileNameW(file_name, 'rb_', 0, buf) == 0
+    if GetTempFileNameW(file_name, String.new('rb_', encoding: 'UTF-16LE'), 0, buf) == 0
       raise SystemCallError, FFI.errno, 'GetTempFileNameW'
     end
 
-    file_name = buf.strip
+    file_name = buf.strip + "\000\000".encode('UTF-16LE')
 
     handle = CreateFileW(
       file_name,

data/lib/file-temp.rb

@@ -1 +1,3 @@
+# frozen_string_literal: true
+
 require_relative 'file/temp'

data/spec/file_temp_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 ######################################################################
 # file_temp_spec.rb
 #
@@ -17,143 +19,149 @@ RSpec.describe File::Temp do
     @fh = nil
 
     # Because Dir[] doesn't work right with backslashes
-    @dir = @dir.tr("\\", "/") if windows
+    @dir = @dir.tr('\\', '/') if windows
   end
 
-  context "constants" do
-    example "library version is set to expected value" do
-      expect( File::Temp::VERSION).to eq('1.7.0')
+  after do
+    @dir = nil
+    @template = nil
+    @fh.close if @fh && !@fh.closed?
+    @fh = nil
+
+    Dir['temp_*'].each{ |f| File.delete(f) }
+    Dir['rb_file_temp_*'].each{ |f| File.delete(f) }
+
+    Dir.chdir(File::Temp::TMPDIR) do
+      Dir['temp_*'].each{ |f| File.delete(f) }
+      Dir['rb_file_temp_*'].each{ |f| File.delete(f) }
+    end
+  end
+
+  context 'constants' do
+    example 'library version is set to expected value' do
+      expect(File::Temp::VERSION).to eq('1.7.2')
       expect(File::Temp::VERSION).to be_frozen
     end
 
-    example "TMPDIR constant is defined" do
-      expect(File::Temp::TMPDIR).to be_kind_of(String)
+    example 'TMPDIR constant is defined' do
+      expect(File::Temp::TMPDIR).to be_a(String)
       expect(File::Temp::TMPDIR.size).to be > 0
     end
   end
 
-  context "threads" do
-    example "library works as expected with multiple threads" do
+  context 'threads' do
+    example 'library works as expected with multiple threads' do
       threads = []
-      expect{ 100.times{ threads << Thread.new{ File::Temp.new }}}.not_to raise_error
-      expect{ threads.each{ |t| t.join }.not_to raise_error }
+      expect{ 100.times{ threads << Thread.new{ described_class.new } } }.not_to raise_error
+      expect{ threads.each(&:join) }.not_to raise_error
     end
   end
 
-  context "constructor" do
-    example "constructor works as expected with default auto delete option" do
+  context 'constructor' do
+    example 'constructor works as expected with default auto delete option' do
       expect{
-        @fh = File::Temp.new
-        @fh.print "hello"
+        @fh = described_class.new
+        @fh.print 'hello'
         @fh.close
       }.not_to raise_error
     end
 
-    example "constructor works as expected with false auto delete option" do
+    example 'constructor works as expected with false auto delete option' do
       expect{
-        @fh = File::Temp.new(:delete => false)
-        @fh.print "hello"
+        @fh = described_class.new(:delete => false)
+        @fh.print 'hello'
         @fh.close
       }.not_to raise_error
     end
 
-    example "constructor accepts and uses an optional template as expected" do
-      expect{ File::Temp.new(:delete => false, :template => 'temp_foo_XXXXXX').close }.not_to raise_error
+    example 'constructor accepts and uses an optional template as expected' do
+      expect{ described_class.new(:delete => false, :template => 'temp_foo_XXXXXX').close }.not_to raise_error
       expect(Dir["#{@dir}/temp_foo*"].length).to be >= 1
     end
 
-    example "constructor with false auto delete and block works as expected" do
+    example 'constructor with false auto delete and block works as expected' do
       expect{
-        File::Temp.open(:delete => false, :template => 'temp_foo_XXXXXX'){ |fh| fh.puts "hello" }
+        described_class.open(:delete => false, :template => 'temp_foo_XXXXXX'){ |fh| fh.puts 'hello' }
       }.not_to raise_error
       expect(Dir["#{@dir}/temp_foo*"].length).to be >= 1
     end
 
-    example "constructor accepts a maximum of three arguments" do
+    example 'other arguments are treated as file option arguments' do
       expect{
-        @fh = File::Temp.new(
-          :delete     => true,
-          :template   => 'temp_bar_XXXXX',
-          :directory  => Dir.pwd,
-          :bogus      => 1
+        @fh = described_class.new(
+          :delete    => true,
+          :template  => 'temp_bar_XXXXX',
+          :directory => Dir.pwd,
+          :mode      => 'xb'
         )
-      }.to raise_error(ArgumentError)
+      }.to raise_error(ArgumentError, /invalid.*?(access)?.*?mode/) # Truffleruby missing the word 'access'
     end
   end
 
-  context "template" do
-    example "template argument must be a string" do
-      expect{ @fh = File::Temp.new(:delete => false, :template => 1) }.to raise_error(TypeError)
+  context 'template' do
+    example 'template argument must be a string' do
+      expect{ @fh = described_class.new(:delete => false, :template => 1) }.to raise_error(TypeError)
     end
 
-    example "an error is raised if a custom template is invalid" do
-      skip "skipped on OSX" if osx
-      expect{ File::Temp.new(:delete => false, :template => 'xx') }.to raise_error(Errno::EINVAL)
+    example 'an error is raised if a custom template is invalid' do
+      skip 'skipped on OSX' if osx
+      expect{ described_class.new(:delete => false, :template => 'xx') }.to raise_error(Errno::EINVAL)
     end
   end
 
-  context "temp_name" do
-    example "temp_name basic functionality" do
-      expect(File::Temp).to respond_to(:temp_name)
-      expect{ File::Temp.temp_name }.not_to raise_error
-      expect(File::Temp.temp_name).to be_kind_of(String)
+  context 'temp_name' do
+    example 'temp_name basic functionality' do
+      expect(described_class).to respond_to(:temp_name)
+      expect{ described_class.temp_name }.not_to raise_error
+      expect(described_class.temp_name).to be_a(String)
     end
 
-    example "temp_name returns expected value" do
+    example 'temp_name returns expected value' do
       if windows
-        expect( File.extname(File::Temp.temp_name)).to match(/^.*?\d*?tmp/)
+        expect(File.extname(described_class.temp_name)).to match(/^.*?\d*?tmp/)
       else
-        expect( File.extname(File::Temp.temp_name)).to eq('.tmp')
+        expect(File.extname(described_class.temp_name)).to eq('.tmp')
       end
     end
   end
 
-  context "path" do
-    example "temp path basic functionality" do
-      @fh = File::Temp.new
+  context 'path' do
+    example 'temp path basic functionality' do
+      @fh = described_class.new
       expect(@fh).to respond_to(:path)
     end
 
-    example "temp path is nil if delete option is true" do
-      @fh = File::Temp.new
+    example 'temp path is nil if delete option is true' do
+      @fh = described_class.new
       expect(@fh.path).to be_nil
     end
 
-    example "temp path is not nil if delete option is false" do
-      @fh = File::Temp.new(delete: false)
+    example 'temp path is not nil if delete option is false' do
+      @fh = described_class.new(delete: false)
       expect(@fh.path).not_to be_nil
     end
   end
 
-  context "ffi" do
-    example "ffi functions are private" do
-      methods = File::Temp.methods(false).map(&:to_s)
-      expect(methods).not_to include('_fileno')
-      expect(methods).not_to include('mkstemp')
-      expect(methods).not_to include('_umask')
-      expect(methods).not_to include('fclose')
-      expect(methods).not_to include('strerror')
-      expect(methods).not_to include('tmpnam')
-      expect(methods).not_to include('CloseHandle')
-      expect(methods).not_to include('CreateFileA')
-      expect(methods).not_to include('DeleteFileA')
-      expect(methods).not_to include('GetTempPathA')
-      expect(methods).not_to include('GetTempFileNameA')
+  context 'ffi' do
+    before do
+      @methods = described_class.methods(false).map(&:to_s)
     end
-  end
-
-  after do
-    @dir = nil
-    @template = nil
-    @fh.close if @fh && !@fh.closed?
-    @fh = nil
 
-    Dir["temp_*"].each{ |f| File.delete(f) }
-    Dir["rb_file_temp_*"].each{ |f| File.delete(f) }
+    example 'ffi unix functions are private', :unix do
+      expect(@methods).not_to include('_fileno')
+      expect(@methods).not_to include('mkstemp')
+      expect(@methods).not_to include('_umask')
+      expect(@methods).not_to include('fclose')
+      expect(@methods).not_to include('strerror')
+      expect(@methods).not_to include('tmpnam')
+    end
 
-    Dir.chdir(File::Temp::TMPDIR) do
-      Dir["temp_*"].each{ |f| File.delete(f) }
-      Dir["rb_file_temp_*"].each{ |f| File.delete(f) }
+    example 'ffi windows functions are private', :windows do
+      expect(@methods).not_to include('CloseHandle')
+      expect(@methods).not_to include('CreateFileA')
+      expect(@methods).not_to include('DeleteFileA')
+      expect(@methods).not_to include('GetTempPathA')
+      expect(@methods).not_to include('GetTempFileNameA')
     end
   end
 end

metadata

@@ -1,11 +1,11 @@
 --- !ruby/object:Gem::Specification
 name: file-temp
 version: !ruby/object:Gem::Version
-  version: 1.7.0
+  version: 1.7.2
 platform: ruby
 authors:
 - Daniel J. Berger
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain:
 - |
@@ -35,7 +35,7 @@ cert_chain:
   ORVCZpRuCPpmC8qmqxUnARDArzucjaclkxjLWvCVHeFa9UP7K3Nl9oTjJNv+7/jM
   WZs4eecIcUc4tKdHxcAJ0MO/Dkqq7hGaiHpwKY76wQ1+8xAh
   -----END CERTIFICATE-----
-date: 
+date: 2023-07-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ffi
@@ -65,6 +65,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.9'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -86,42 +114,34 @@ description: |2
 email: djberg96@gmail.com
 executables: []
 extensions: []
-extra_rdoc_files:
-- CHANGES.rdoc
-- README.rdoc
-- MANIFEST.rdoc
+extra_rdoc_files: []
 files:
+- CHANGES.md
+- Gemfile
 - LICENSE
-- file-temp.gemspec
-- spec
-- spec/file_temp_spec.rb
+- MANIFEST.md
+- README.md
 - Rakefile
-- certs
 - certs/djberg96_pub.pem
-- lib
+- file-temp.gemspec
 - lib/file-temp.rb
-- lib/file
-- lib/file/temp.rb
-- lib/file/java
 - lib/file/java/temp.rb
-- lib/file/unix
+- lib/file/temp.rb
 - lib/file/unix/temp.rb
-- lib/file/windows
 - lib/file/windows/temp.rb
-- CHANGES.rdoc
-- MANIFEST.rdoc
-- README.rdoc
+- spec/file_temp_spec.rb
 homepage: http://github.com/djberg96/file-temp
 licenses:
 - Apache-2.0
 metadata:
   homepage_uri: https://github.com/djberg96/file-temp
   bug_tracker_uri: https://github.com/djberg96/file-temp/issues
-  changelog_uri: https://github.com/djberg96/file-temp/blob/master/CHANGES
+  changelog_uri: https://github.com/djberg96/file-temp/blob/main/CHANGES.md
   documentation_uri: https://github.com/djberg96/file-temp/wiki
   source_code_uri: https://github.com/djberg96/file-temp
   wiki_uri: https://github.com/djberg96/file-temp/wiki
-post_install_message: 
+  rubygems_mfa_required: 'true'
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -136,8 +156,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.3.26
+signing_key:
 specification_version: 4
 summary: An alternative way to generate temp files
 test_files: