Skip to content

Commit 14f0a2a

Browse files
committed
Use a temp file, not a directory
1 parent 4ac6992 commit 14f0a2a

1 file changed

Lines changed: 9 additions & 4 deletions

File tree

spec/octocatalog-diff/tests/spec_helper.rb

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
require 'logger'
33
require 'rspec'
44
require 'rspec/retry'
5+
require 'tempfile'
56

67
# Enable SimpleCov coverage testing?
78
if ENV['COVERAGE']
@@ -37,16 +38,20 @@ class CustomLogger
3738
attr_accessor :logger
3839

3940
def initialize
40-
@log_tempdir = Dir.mktmpdir
41-
at_exit { FileUtils.remove_entry_secure @log_tempdir if File.directory?(@log_tempdir) }
41+
@tf = Tempfile.new('customlogger.log')
42+
at_exit do
43+
@tf.close
44+
@tf.unlink
45+
end
4246

43-
@logger = Logger.new File.join(@log_tempdir, 'customlogger.out')
47+
@logger = Logger.new @tf
4448
@logger.level = Logger::DEBUG
4549
end
4650

4751
def string
4852
@content ||= begin
49-
content = File.read(File.join(@log_tempdir, 'customlogger.out'))
53+
@tf.close
54+
content = File.read(@tf.path)
5055
content.sub(/\A# Logfile created .+\n/, '')
5156
end
5257
end

0 commit comments

Comments
 (0)