Skip to content
This repository has been archived by the owner on Jun 19, 2020. It is now read-only.

Commit

Permalink
(FACT-2511) Remove file logger. (#425)
Browse files Browse the repository at this point in the history
  • Loading branch information
Bogdan Irimie authored Apr 1, 2020
1 parent 2912b01 commit 5dec73b
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 17 deletions.
11 changes: 1 addition & 10 deletions lib/framework/logging/logger.rb
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,8 @@ module Facter
RED = 31

class Log
@@file_logger = Logger.new(File.new("#{ROOT_DIR}/example.log", 'a'))
@@legacy_logger = nil
@@logger = MultiLogger.new([@@file_logger])
@@logger = MultiLogger.new([])
@@logger.level = :warn
@@message_callback = nil
@@has_errors = false
Expand All @@ -33,7 +32,6 @@ def errors?

def initialize(logged_class)
determine_callers_name(logged_class)
set_format_for_file_logger
end

def self.add_legacy_logger(output)
Expand All @@ -57,13 +55,6 @@ def determine_callers_name(sender_self)
end
end

def set_format_for_file_logger
@@file_logger.formatter = proc do |severity, datetime, _progname, msg|
datetime = datetime.strftime(@datetime_format || '%Y-%m-%d %H:%M:%S.%6N ')
"[#{datetime}] #{severity} #{msg} \n"
end
end

def self.set_format_for_legacy_logger
@@legacy_logger.formatter = proc do |severity, datetime, _progname, msg|
datetime = datetime.strftime(@datetime_format || '%Y-%m-%d %H:%M:%S.%6N ')
Expand Down
7 changes: 0 additions & 7 deletions spec/framework/logging/logger_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -13,13 +13,6 @@
allow(file_logger_double).to receive(:formatter=)
end

describe '#initialize' do
it 'sets file_logger formatters' do
log
expect(file_logger_double).to have_received(:formatter=)
end
end

describe '#debug' do
before do
allow(Facter).to receive(:debugging?).and_return(true)
Expand Down

0 comments on commit 5dec73b

Please sign in to comment.