From 5dec73b3921b911178e80047bd7cf9a97fcf9d74 Mon Sep 17 00:00:00 2001 From: Bogdan Irimie Date: Wed, 1 Apr 2020 10:08:56 +0300 Subject: [PATCH] (FACT-2511) Remove file logger. (#425) --- lib/framework/logging/logger.rb | 11 +---------- spec/framework/logging/logger_spec.rb | 7 ------- 2 files changed, 1 insertion(+), 17 deletions(-) diff --git a/lib/framework/logging/logger.rb b/lib/framework/logging/logger.rb index e46f8445b..3755a5cc5 100644 --- a/lib/framework/logging/logger.rb +++ b/lib/framework/logging/logger.rb @@ -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 @@ -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) @@ -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 ') diff --git a/spec/framework/logging/logger_spec.rb b/spec/framework/logging/logger_spec.rb index f365cb1a2..4ca146127 100644 --- a/spec/framework/logging/logger_spec.rb +++ b/spec/framework/logging/logger_spec.rb @@ -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)