hubot 4.0.0 → 4.1.0

package/README.md

@@ -18,4 +18,3 @@ are building your own bot. But if you do, check out [CONTRIBUTING.md](CONTRIBUTI
 ## License
 
 See the [LICENSE](LICENSE.md) file for license rights and limitations (MIT).
-`

package/package.json

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

package/src/robot.js

@@ -1,12 +1,10 @@
 '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 pino = require('pino')
 const HttpClient = require('./httpclient')
 
 const Brain = require('./brain')
@@ -50,8 +48,15 @@ class Robot {
       response: new Middleware(this),
       receive: new Middleware(this)
     }
-    process.env.LOG_LEVEL = process.env.LOG_LEVEL || process.env.HUBOT_LOG_LEVEL || 'info'
-    this.logger = log.get('robot')
+    this.logger = pino({
+      name,
+      level: process.env.HUBOT_LOG_LEVEL || 'info'
+    })
+    Reflect.defineProperty(this.logger, 'warning', {
+      value: this.logger.warn,
+      enumerable: true,
+      configurable: true
+    })
 
     this.pingIntervalId = null
     this.globalHttpOptions = {}

package/test/robot_test.js

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