소스 검색

Clean up acceptance tests

Christopher Hoskin 6 년 전
부모
커밋
e32411eaaa
2개의 변경된 파일과 8개의 추가작업 그리고 20개의 파일을 삭제
  1. 0 4
      libraries/logger/logging-manager.js
  2. 8 16
      libraries/logger/test/unit/loggingManagerTests.js

+ 0 - 4
libraries/logger/logging-manager.js

@@ -22,7 +22,6 @@ const errSerializer = function (err) {
 
 const Logger = (module.exports = {
   initialize(name) {
-    console.log('IN INIT')
     this.logLevelSource = (process.env.LOG_LEVEL_SOURCE || 'file').toLowerCase()
     this.isProduction =
       (process.env.NODE_ENV || '').toLowerCase() === 'production'
@@ -40,7 +39,6 @@ const Logger = (module.exports = {
     })
     this._setupRingBuffer()
     this._setupStackdriver()
-    console.log(this.logger)
     this._setupLogLevelChecker()
     return this
   },
@@ -256,8 +254,6 @@ const Logger = (module.exports = {
   },
 
   _setupLogLevelChecker() {
-    console.log('In _setupLogLevelChecker')
-    console.log(this.logger)
     if (this.isProduction) {
       // clear interval if already set
       if (this.checkInterval) {

+ 8 - 16
libraries/logger/test/unit/loggingManagerTests.js

@@ -111,21 +111,13 @@ describe('LoggingManager', function () {
 
       afterEach(() => delete process.env.NODE_ENV)
 
-      describe('blah', function () {
-        beforeEach(function () {
-          this.logger = this.LoggingManager.initialize(this.loggerName)
-        })
-        it('should default to log level warn', function () {
-
-          const level = this.Bunyan.createLogger.firstCall.args[0].streams[0].level
-          console.log(level)
-          level.should.equal(
-            'warn'
-          )
-        })       
-      })
-
-
+      it('should default to log level warn', function () {
+        this.logger = this.LoggingManager.initialize(this.loggerName)
+        this.Bunyan.createLogger.firstCall.args[0].streams[0].level.should.equal(
+          'warn'
+        )
+      })       
+      
       describe('logLevelSource file', function () {
         beforeEach(function() {
           this.logger = this.LoggingManager.initialize(this.loggerName)
@@ -324,7 +316,7 @@ describe('LoggingManager', function () {
 
     describe('when read errors', function () {
       beforeEach(async function () {
-        this.Fs.promises.readFile.yields(new Error('error'))
+        this.Fs.promises.readFile.throws(new Error('error'))
         this.logger.getTracingEndTime = this.logger.getTracingEndTimeFile
         await this.logger.checkLogLevel()
       })