hubot 3.5.0 → 4.0.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hubot",
-  "version": "3.5.0",
+  "version": "4.0.0",
   "author": "hubot",
   "keywords": [
     "github",
@@ -21,7 +21,8 @@
     "connect-multiparty": "^2.2.0",
     "express": "^4.18.2",
     "express-basic-auth": "^1.2.1",
-    "log": "1.4.0",
+    "log": "^6.3.1",
+    "log-node": "^8.0.3",
     "optparse": "^1.0.5"
   },
   "devDependencies": {

package/src/robot.js

@@ -1,11 +1,12 @@
 'use strict'
+require('log-node')()
 
 const EventEmitter = require('events').EventEmitter
 const fs = require('fs')
 const path = require('path')
 
 const async = require('async')
-const Log = require('log')
+const log = require('log')
 const HttpClient = require('./httpclient')
 
 const Brain = require('./brain')
@@ -49,7 +50,9 @@ class Robot {
       response: new Middleware(this),
       receive: new Middleware(this)
     }
-    this.logger = new Log(process.env.HUBOT_LOG_LEVEL || 'info')
+    process.env.LOG_LEVEL = process.env.LOG_LEVEL || process.env.HUBOT_LOG_LEVEL || 'info'
+    this.logger = log.get('robot')
+
     this.pingIntervalId = null
     this.globalHttpOptions = {}
 

package/test/robot_test.js

@@ -6,6 +6,7 @@
 // Assertions and Stubbing
 const chai = require('chai')
 const sinon = require('sinon')
+const emitter = require('log/lib/emitter')
 chai.use(require('sinon-chai'))
 
 const expect = chai.expect
@@ -422,15 +423,24 @@ describe('Robot', function () {
         })
 
         it('logs a warning for a .js file', function () {
-          sinon.stub(this.robot.logger, 'warning')
+          let wasCalled = false
+          const listener = e => {
+            wasCalled = e.messageTokens.some(t => t.indexOf('Expected scripts/test-script') > -1)
+          }
+          emitter.on('log', listener)
           this.robot.loadFile('./scripts', 'test-script.js')
-          expect(this.robot.logger.warning).to.have.been.called
+          expect(wasCalled).to.be.true
+          emitter.off('log', listener)
         })
 
         it('logs a warning for a .mjs file', function () {
-          sinon.stub(this.robot.logger, 'warning')
+          let wasCalled = false
+          const listener = e => {
+            wasCalled = e.messageTokens.some(t => t.indexOf('Expected scripts/test-script') > -1)
+          }
+          emitter.on('log', listener)
           this.robot.loadFile('./scripts', 'test-script.mjs')
-          expect(this.robot.logger.warning).to.have.been.called
+          expect(wasCalled).to.be.true
         })
       })
 

package/.node-version

@@ -1 +0,0 @@
-18.16.0