[pinpoint-apm#117] Logger integration test

feelform · Oct 19, 2022 · 1a3969a · 1a3969a
1 parent 710d830
commit 1a3969a
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 2 deletions.
diff --git a/lib/config.js b/lib/config.js
@@ -247,10 +247,15 @@ function hasDockerCGroup() {
   }
 }
 
+function needsLoadConfig() {
+  return typeof agentConfig !== 'undefined'
+}
+
 module.exports = {
   getConfig,
   clear,
   readConfigJson,
   readRootConfigFile,
-  getMainModulePath
+  getMainModulePath,
+  needsLoadConfig
 }
diff --git a/lib/supports.js b/lib/supports.js
@@ -7,6 +7,7 @@
 'use strict'
 
 const Logger = require('./utils/log/logger2')
+const { needsLoadConfig } = require('./config')
 
 const log = new Logger.NoneBuilder({
    logs: [],
@@ -26,6 +27,8 @@ const log = new Logger.NoneBuilder({
 
 module.exports = {
    getLog: function () {
-      return log
+      if (needsLoadConfig()) {
+         return log
+      }
    }
 }
diff --git a/test/utils/log/logger.integration.test.js b/test/utils/log/logger.integration.test.js
@@ -8,10 +8,13 @@
 const test = require('tape')
 const { getLog } = require('../../../lib/supports')
 const Agent = require('../../../lib/agent')
+const { clear } = require('../../../lib/config')
 
 test('Logger Adaptor', (t) => {
+    clear()
     const actual = getLog()
     new Agent()
     t.equal(actual.adaptor.constructor.name, 'LoggerOutputAdaptor', 'log adaptor')
+
     t.end()
 })