Added increased code coverage of wlogger

bin/server.js

@@ -19,7 +19,7 @@ const adminLib = new AdminLib()
 // const rpc = new JSONRPC()
 
 const errorMiddleware = require('../src/controllers/rest-api/middleware/error')
-const wlogger = require('../src/adapters/wlogger')
+const { wlogger } = require('../src/adapters/wlogger')
 
 async function startServer () {
   // Create a Koa instance.

src/adapters/contact.js

@@ -9,7 +9,7 @@ const config = require('../../config')
 
 const NodeMailer = require('../adapters/nodemailer')
 const nodemailer = new NodeMailer()
-const wlogger = require('../adapters/wlogger')
+const { wlogger } = require('../adapters/wlogger')
 
 let _this
 

src/adapters/index.js

@@ -10,7 +10,7 @@ const LocalDB = require('./localdb')
 const LogsAPI = require('./logapi')
 const Passport = require('./passport')
 const Nodemailer = require('./nodemailer')
-const wlogger = require('./wlogger')
+const { wlogger } = require('./wlogger')
 const JSONFiles = require('./json-files')
 
 // Instantiate adapter libraries.

src/adapters/nodemailer.js

@@ -7,7 +7,7 @@ const nodemailer = require('nodemailer')
 
 const config = require('../../config')
 
-const wlogger = require('./wlogger')
+const { wlogger } = require('./wlogger')
 
 let _this
 

src/adapters/wlogger.js

@@ -24,9 +24,11 @@ const transport = new winston.transports.DailyRotateFile({
   )
 })
 
-transport.on('rotate', function (oldFilename, newFilename) {
+transport.on('rotate', notifyRotation)
+
+function notifyRotation (oldFilename, newFilename) {
   wlogger.info('Rotating log files')
-})
+}
 
 // This controls what goes into the log FILES
 const wlogger = winston.createLogger({
@@ -43,8 +45,7 @@ const wlogger = winston.createLogger({
   ]
 })
 
-// This controls the logs to CONSOLE
-if (config.env !== 'test') {
+function outputToConsole () {
   wlogger.add(
     new winston.transports.Console({
       format: winston.format.simple(),
@@ -53,4 +54,9 @@ if (config.env !== 'test') {
   )
 }
 
-module.exports = wlogger
+// This controls the logs to CONSOLE
+if (config.env !== 'test') {
+  outputToConsole()
+}
+
+module.exports = { wlogger, notifyRotation, outputToConsole }

src/controllers/json-rpc/auth/index.js

@@ -8,7 +8,7 @@ const jsonrpc = require('jsonrpc-lite')
 // Local libraries
 // const AuthLib = require('../../lib/auth')
 // const UserLib = require('../../../use-cases/user')
-const wlogger = require('../../../adapters/wlogger')
+const { wlogger } = require('../../../adapters/wlogger')
 const RateLimit = require('../rate-limit')
 
 class AuthRPC {

src/controllers/json-rpc/index.js

@@ -6,7 +6,7 @@
 const jsonrpc = require('jsonrpc-lite')
 
 // Local support libraries
-const wlogger = require('../../adapters/wlogger')
+const { wlogger } = require('../../adapters/wlogger')
 const UserController = require('./users')
 const AuthController = require('./auth')
 const AboutController = require('./about')

src/controllers/rest-api/middleware/validators.js

@@ -5,7 +5,7 @@
 const User = require('../../../adapters/localdb/models/users')
 const config = require('../../../../config')
 const jwt = require('jsonwebtoken')
-const wlogger = require('../../../adapters/wlogger')
+const { wlogger } = require('../../../adapters/wlogger')
 
 let _this
 

src/controllers/rest-api/users/controller.js

@@ -8,7 +8,7 @@
 // User library for business logic.
 // const UserLib = require('../../../use-cases/user')
 
-const wlogger = require('../../../adapters/wlogger')
+const { wlogger } = require('../../../adapters/wlogger')
 
 let _this
 

src/use-cases/user.js

@@ -4,7 +4,7 @@
 */
 
 // const UserModel = require('../adapters/localdb/models/users')
-const wlogger = require('../adapters/wlogger')
+const { wlogger } = require('../adapters/wlogger')
 
 class UserLib {
   constructor (localConfig = {}) {

test/unit/adapters/wlogger.adapter.unit.js

@@ -0,0 +1,30 @@
+// const assert = require('chai').assert
+const {
+  notifyRotation,
+  outputToConsole
+} = require('../../../src/adapters/wlogger')
+
+const sinon = require('sinon')
+
+// let uut
+let sandbox
+
+describe('#wlogger.js', () => {
+  beforeEach(() => {
+    sandbox = sinon.createSandbox()
+  })
+
+  afterEach(() => sandbox.restore())
+
+  describe('#notifyRotation', () => {
+    it('should notify of a log rotation', () => {
+      notifyRotation()
+    })
+  })
+
+  describe('#envronment', () => {
+    it('should write to console in non-test environment', () => {
+      outputToConsole()
+    })
+  })
+})