Skip to content

Commit 27f870b

Browse files
authored
Merge pull request #2724 from joshcooper/logtofile
Allow facter test log to be written to a file
2 parents 066e4f5 + ed2039a commit 27f870b

File tree

1 file changed

+9
-2
lines changed

1 file changed

+9
-2
lines changed

spec/spec_helper.rb

+9-2
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,10 @@ def colorize(str, color)
4848
require 'webmock/rspec'
4949
WebMock.disable_net_connect!
5050

51+
# Configure test logger
52+
logdest = ENV['FACTER_TEST_LOG'] ? File.new(ENV['FACTER_TEST_LOG'], 'w') : StringIO.new
53+
logger = Logger.new(logdest)
54+
5155
# Configure RSpec
5256
RSpec.configure do |config|
5357
# Enable flags like --only-failures and --next-failure
@@ -77,10 +81,13 @@ def colorize(str, color)
7781
end
7882

7983
config.before(:all) do
80-
Facter::Log.class_variable_set(:@@logger, Logger.new(StringIO.new)) # rubocop:disable Style/ClassVars
84+
Facter::Log.class_variable_set(:@@logger, logger) # rubocop:disable Style/ClassVars
8185
end
8286

83-
config.before do
87+
config.before do |test|
88+
m = test.metadata
89+
logger.info("*** BEGIN TEST #{m[:file_path]}:#{m[:line_number]}")
90+
8491
LegacyFacter.clear
8592
Facter.clear_messages
8693
end

0 commit comments

Comments
 (0)