Rename automation_log_wrapper

Author: agrare

lib/vmdb/loggers/request_logger.rb

@@ -1,19 +1,12 @@
 module Vmdb::Loggers
   class RequestLogger < ManageIQ::Loggers::Base
-    attr_reader :automation_log_wrapper
+    attr_reader :log_wrapper
 
-    def initialize(*args, automation_log_wrapper:, **kwargs)
-      @automation_log_wrapper = automation_log_wrapper
+    def initialize(*args, log_wrapper:, **kwargs)
+      @log_wrapper = log_wrapper
       super(*args, **kwargs)
     end
 
-    def self.create_log_wrapper(io = File::NULL)
-      # We modify the interface of logger methods such as info/warn/etc. to allow the keyword argument
-      # resource_id. Therefore, we need to wrap all client logger calls to these methods to process the resource_id,
-      # cut the request_log entry and forward the remaining arguments to the logger.
-      new(io, :progname => "automation", :automation_log_wrapper => Vmdb::Loggers.create_logger("automation.log"))
-    end
-
     private def add_to_db(severity, message = nil, progname = nil, resource_id: nil)
       return [severity, message, progname] unless resource_id
 
@@ -41,37 +34,37 @@ def self.create_log_wrapper(io = File::NULL)
 
     def info(progname = nil, resource_id: nil, &block)
       severity, message, progname = add_to_db(INFO, nil, progname, resource_id: resource_id, &block)
-      automation_log_wrapper.add(severity, message, progname, &block)
+      log_wrapper.add(severity, message, progname, &block)
       add(severity, message, progname, &block)
     end
 
     def debug(progname = nil, resource_id: nil, &block)
       severity, message, progname = add_to_db(DEBUG, nil, progname, resource_id: resource_id, &block)
-      automation_log_wrapper.add(severity, message, progname, &block)
+      log_wrapper.add(severity, message, progname, &block)
       add(severity, message, progname, &block)
     end
 
     def warn(progname = nil, resource_id: nil, &block)
       severity, message, progname = add_to_db(WARN, nil, progname, resource_id: resource_id, &block)
-      automation_log_wrapper.add(severity, message, progname, &block)
+      log_wrapper.add(severity, message, progname, &block)
       add(severity, message, progname, &block)
     end
 
     def error(progname = nil, resource_id: nil, &block)
       severity, message, progname = add_to_db(ERROR, nil, progname, resource_id: resource_id, &block)
-      automation_log_wrapper.add(severity, message, progname, &block)
+      log_wrapper.add(severity, message, progname, &block)
       add(severity, message, progname, &block)
     end
 
     def fatal(progname = nil, resource_id: nil, &block)
       severity, message, progname = add_to_db(FATAL, nil, progname, resource_id: resource_id, &block)
-      automation_log_wrapper.add(severity, message, progname, &block)
+      log_wrapper.add(severity, message, progname, &block)
       add(severity, message, progname, &block)
     end
 
     def unknown(progname = nil, resource_id: nil, &block)
       severity, message, progname = add_to_db(UNKNOWN, nil, progname, resource_id: resource_id, &block)
-      automation_log_wrapper.add(severity, message, progname, &block)
+      log_wrapper.add(severity, message, progname, &block)
       add(severity, message, progname, &block)
     end
   end