Make hmpo-logger optional by loading at runtime and defaulting to con…

lib/remote-model.js

@@ -4,7 +4,6 @@ const LocalModel = require('./local-model');
 const requestLib = require('request');
 const _ = require('underscore');
 const kebabCase = require('lodash.kebabcase');
-const hmpoLogger = require('hmpo-logger');
 
 class RemoteModel extends LocalModel {
     constructor(attributes, options) {
@@ -15,7 +14,13 @@ class RemoteModel extends LocalModel {
     }
 
     setLogger() {
-        this.logger = hmpoLogger.get(':' + this.options.label);
+        try {
+            const hmpoLogger = require('hmpo-logger');
+            this.logger = hmpoLogger.get(':' + this.options.label);
+        } catch (e) {
+            console.error('Error setting logger, using console instead!', e);
+            this.logger = { outbound: console.log, trimHtml: _.identity };
+        }
     }
 
     fetch(callback) {

package.json

@@ -39,9 +39,6 @@
     "sinon": "^10.0.0",
     "sinon-chai": "^3.6.0"
   },
-  "peerDependencies": {
-    "hmpo-logger": ">= 4"
-  },
   "nyc": {
     "all": true,
     "exclude": [

test/lib/spec.remote-model.js

@@ -54,16 +54,36 @@ describe('Remote Model', () => {
     });
 
     describe('setLogger', () => {
-        let getStub = sinon.stub();
-        let Model = proxyquire('../../lib/remote-model', {
-            'hmpo-logger': {
-                get: getStub
-            }
+        let getStub, Model;
+
+        beforeEach(() => {
+            getStub = sinon.stub();
+            getStub.returns('logger');
+
+            Model = proxyquire('../../lib/remote-model', {
+                'hmpo-logger': {
+                    get: getStub
+                }
+            });
         });
 
-        model = new Model();
+        it('should set up a new hmpo-logger', () => {
+            model = new Model();
 
-        getStub.should.have.been.calledWithExactly(':remote-model');
+            getStub.should.have.been.calledWithExactly(':remote-model');
+            model.logger.should.equal('logger');
+        });
+
+        it('should use console if hmpo-logger is not available', () => {
+            getStub.throws(new Error());
+
+            model = new Model();
+
+            model.logger.should.eql({
+                outbound: console.log,
+                trimHtml: _.identity
+            });
+        });
     });
 
     describe('fetch', () => {