file-temp 1.7.1 → 1.7.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2f7a532c6a985ef7afd93f064e38ff3fb81228d642707fa06156f23b1a2d33c6
-  data.tar.gz: 5de639ab3cc6644fb385d35b364f8aa4c6c8afbd989706875a8e6f90193eaa18
+  metadata.gz: 04ece0eed9e9cc0e3ee18ef8637623714efe6bc817abdfc8583f933042e05029
+  data.tar.gz: 67155b0989bf977750f52abde8b57411f49c0dce07cae2a51d616298d4bf85e5
 SHA512:
-  metadata.gz: 7e77feff9b86f7e33c233037e5c5342fac28a7bd3e5b18296cfae915e8a9e39b0d73a4a9dc87ac3ce096869b9bad401c98041bd2cb847a04d4c1b74f98a5b884
-  data.tar.gz: 285fe4b48491c6468e7f0a3646d3a9da2b9d55e9386b5f6ee232882bb611311c3122ffa34ac98162e3f847f618c20fae57254c8135d368cc58aeb7d76761dd8e
+  metadata.gz: 249c699561ea25f3b427f7fc117c69d9ea848055e78063f87b99718786e4ad483f32c5e356dd836879a747b0d724c6dc2239f3c334d0ab0c71be7243bc7b823a
+  data.tar.gz: bf5db324f1069cfeed9a2930cf585edb3d2263c7634c9999c53c026f0ae09961055b03d51561cb5e9d045d4ed017dc2a528374c859f198492de8a1e604a9a0ca

data/CHANGES.md

@@ -1,3 +1,12 @@
+## 1.7.3 - 23-Dec-2025
+* Internal refactoring, replace mktemp with mkstemp.
+* Internal refactoring, replace tmpnam with Ruby's SecureRandom.
+
+## 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.

data/Gemfile

@@ -1,7 +1,2 @@
-source 'https://rubygems.org' do
-  gem 'rake'
-  gem 'ffi', '~> 1.1'
-  group 'test' do
-    gem 'rspec', '~> 3.9'
-  end
-end
+source 'https://rubygems.org'
+gemspec

data/README.md

@@ -1,3 +1,5 @@
+[![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.
 
@@ -7,8 +9,11 @@ The file-temp library is an alternate way to handle tempfile generation.
 ## 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
-```
+```ruby
 require 'file/temp'
 
 fh = File::Temp.new
@@ -42,18 +47,22 @@ 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.
 
+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
+of MS Windows that may cause a failure. However, if you're patched up you
 should not see it. 
 
 ## License
 Apache-2.0
 
 ## Copyright
-(C) 2007-2020 Daniel J. Berger
+(C) 2007-2025 Daniel J. Berger
 All Rights Reserved
 
 ## Warranty

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,7 +22,17 @@ namespace 'gem' do
   end
 end
 
+RuboCop::RakeTask.new
+
 desc 'Run the test suite for the file-temp library'
-RSpec::Core::RakeTask.new(:spec)
+RSpec::Core::RakeTask.new(:spec) do |t|
+  t.verbose = false
+  t.rspec_opts = '-f documentation -w'
+end
+
+# Clean up afterwards
+Rake::Task[:spec].enhance do
+  Rake::Task[:clean].invoke
+end
 
 task :default => :spec

data/file-temp.gemspec

@@ -2,7 +2,7 @@ require 'rubygems'
 
 Gem::Specification.new do |spec|
   spec.name       = 'file-temp'
-  spec.version    = '1.7.1'
+  spec.version    = '1.7.3'
   spec.author     = 'Daniel J. Berger'
   spec.license    = 'Apache-2.0'
   spec.email      = 'djberg96@gmail.com'
@@ -14,15 +14,20 @@ Gem::Specification.new do |spec|
 
   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.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'
+    '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',
+    'github_repo'           => 'https://github.com/djberg96/file-temp',
+    'funding_uri'           => 'https://github.com/sponsors/djberg96'
   }
 
   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
@@ -42,14 +44,14 @@ 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
 
     @file.deleteOnExit if delete
     options[:mode] ||= 'wb+'
 
-    path = @file.getName
+    path = @file.getAbsolutePath
     super(path, **options)
 
     @path = path unless delete
@@ -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.1'.freeze
+  VERSION = '1.7.3'
 end
 
 if RUBY_PLATFORM == 'java'

data/lib/file/unix/temp.rb

@@ -1,6 +1,10 @@
+# frozen_string_literal: true
+
 require 'ffi'
 require 'tmpdir'
+require 'securerandom'
 
+# 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
@@ -13,11 +17,10 @@ class File::Temp < File
   attach_function :_fileno, :fileno, [:pointer], :int
   attach_function :strerror, [:int], :string
   attach_function :tmpfile, [], :pointer
-  attach_function :tmpnam, [:pointer], :string
-  attach_function :mktemp, [:pointer], :string
+  attach_function :mkstemp, [:pointer], :int
 
-  private_class_method :mktemp, :strerror, :tmpfile
-  private_class_method :tmpnam, :fclose, :_fileno
+  private_class_method :mkstemp, :strerror, :tmpfile
+  private_class_method :fclose, :_fileno
 
   public
 
@@ -39,7 +42,7 @@ class File::Temp < File
   #
   # If the +delete+ option is set to false, then the file is not deleted. In
   # addition, you can supply a string +template+ that the system replaces with
-  # a unique filename. This template should end with 3 to 6 'X' characters.
+  # a unique filename. This template should end with 6 'X' characters.
   # The default template is 'rb_file_temp_XXXXXX'. In this case the temporary
   # file lives in the directory where it was created.
   #
@@ -53,22 +56,32 @@ class File::Temp < File
   #
   def initialize(delete: true, template: 'rb_file_temp_XXXXXX', directory: TMPDIR, **options)
     @fptr = nil
+    temp_path = nil
 
     if delete
       @fptr = tmpfile()
       raise SystemCallError.new('tmpfile', FFI.errno) if @fptr.null?
       fd = _fileno(@fptr)
     else
+      unless template.is_a?(String)
+        raise TypeError, "template must be a String"
+      end
+
+      unless template.end_with?('XXXXXX')
+        raise ArgumentError, "template must end with 6 'X' characters"
+      end
+
       begin
         omask = File.umask(077)
-        ptr = FFI::MemoryPointer.from_string(template)
-        str = mktemp(ptr)
+        full_template = File.join(directory, template)
+        ptr = FFI::MemoryPointer.from_string(full_template)
+        fd = mkstemp(ptr)
 
-        if str.nil? || str.empty?
-          raise SystemCallError.new('mktemp', FFI.errno)
+        if fd < 0
+          raise SystemCallError.new('mkstemp', FFI.errno)
         end
 
-        @path = File.join(directory, ptr.read_string)
+        temp_path = ptr.read_string
       ensure
         File.umask(omask)
       end
@@ -76,11 +89,9 @@ class File::Temp < File
 
     options[:mode] ||= 'wb+'
 
-    if delete
-      super(fd, **options)
-    else
-      super(@path, **options)
-    end
+    super(fd, **options)
+
+    @path = temp_path
   end
 
   # The close method was overridden to ensure the internal file pointer that we
@@ -101,6 +112,6 @@ class File::Temp < File
   # Note that a file is not actually generated on the filesystem.
   #
   def self.temp_name
-    tmpnam(nil) << '.tmp'
+    File.join(TMPDIR, "tmp.#{SecureRandom.hex(8)}.tmp")
   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
@@ -88,6 +92,14 @@ class File::Temp < File
       @fptr = tmpfile()
       fd = _fileno(@fptr)
     else
+      unless template.is_a?(String)
+        raise TypeError, "template must be a String"
+      end
+
+      unless template.end_with?('XXXXXX')
+        raise ArgumentError, "template must end with 6 'X' characters"
+      end
+
       begin
         omask = File.umask(077)
         ptr = FFI::MemoryPointer.from_string(template)
@@ -134,7 +146,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 +165,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 +182,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
 #
@@ -10,6 +12,7 @@ require 'file/temp'
 RSpec.describe File::Temp do
   let(:windows) { File::ALT_SEPARATOR }
   let(:osx) { RbConfig::CONFIG['host_os'] =~ /darwin/i }
+  let(:bsd) { RbConfig::CONFIG['host_os'] =~ /freebsd|dragonfly/i }
 
   before do
     @dir = File::Temp::TMPDIR
@@ -17,143 +20,148 @@ 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.1')
+  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.3')
       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 "other arguments are treated as file option arguments" do
+    example 'other arguments are treated as file option arguments' do
       expect{
-        @fh = File::Temp.new(
+        @fh = described_class.new(
           :delete    => true,
           :template  => 'temp_bar_XXXXX',
           :directory => Dir.pwd,
           :mode      => 'xb'
         )
-      }.to raise_error(ArgumentError, /invalid access mode/)
+      }.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
+      expect{ described_class.new(:delete => false, :template => 'xx') }.to raise_error(ArgumentError, /template must end with 6/)
     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,10 @@
 --- !ruby/object:Gem::Specification
 name: file-temp
 version: !ruby/object:Gem::Version
-  version: 1.7.1
+  version: 1.7.3
 platform: ruby
 authors:
 - Daniel J. Berger
-autorequire:
 bindir: bin
 cert_chain:
 - |
@@ -35,7 +34,7 @@ cert_chain:
   ORVCZpRuCPpmC8qmqxUnARDArzucjaclkxjLWvCVHeFa9UP7K3Nl9oTjJNv+7/jM
   WZs4eecIcUc4tKdHxcAJ0MO/Dkqq7hGaiHpwKY76wQ1+8xAh
   -----END CERTIFICATE-----
-date:
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ffi
@@ -65,6 +64,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
@@ -88,38 +115,33 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- CHANGES.md
+- Gemfile
 - LICENSE
-- file-temp.gemspec
-- spec
-- spec/file_temp_spec.rb
+- MANIFEST.md
 - README.md
 - Rakefile
-- MANIFEST.md
-- 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
-- Gemfile
-- CHANGES.md
+- 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.md
+  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'
+  github_repo: https://github.com/djberg96/file-temp
+  funding_uri: https://github.com/sponsors/djberg96
 rdoc_options: []
 require_paths:
 - lib
@@ -134,8 +156,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key:
+rubygems_version: 3.7.2
 specification_version: 4
 summary: An alternative way to generate temp files
 test_files: