fix failing builds (#50)

* dependency tidy

* bump node

* no npm-prepublish
 🐿 v2.6.0

* use ignore instead of only in babel-register
 🐿 v2.6.0

.gitignore

@@ -4,3 +4,4 @@
 /.idea/
 /dist/
 /node_modules/
+*.env*

Makefile

@@ -11,6 +11,6 @@ build: $(shell find src -type f)
 
 unit-test: build
 	@echo "Unit Testing…"
-	@mocha --require test/setup --recursive --reporter spec test
+	@mocha --require test/setup --recursive --exit --reporter spec test
 
 test: verify unit-test

circle.yml

@@ -1,10 +1,10 @@
 machine:
   node:
-    version: 4.7.3
-
+    version: 8.9.3
 deployment:
   release:
-    tag: /v.*/
+    tag: /^v?\d+\.\d+\.\d+(?:-beta\.\d+)?$/
     owner: Financial-Times
     commands:
-      - make npm-publish
+      - npm version --no-git-tag-version ${CIRCLE_TAG}
+      - npm publish --access public

package.json

@@ -12,25 +12,22 @@
     "prepush": "make verify -j3"
   },
   "dependencies": {
-    "request": "^2.67.0",
-    "winston": "^2.3.1"
+    "request": "^2.83.0",
+    "winston": "^2.4.0"
   },
   "devDependencies": {
-    "@financial-times/n-gage": "^1.2.0",
-    "babel-cli": "^6.5.1",
-    "babel-preset-env": "^1.1.8",
-    "babel-register": "^6.5.1",
-    "chai": "^3.4.0",
-    "chai-string": "^1.1.6",
-    "eslint": "^3.0.1",
-    "lintspaces-cli": "^0.6.0",
-    "mocha": "^3.2.0",
-    "npm-prepublish": "^1.2.1",
-    "sinon": "^1.17.0",
-    "sinon-chai": "^2.8.0"
+    "@financial-times/n-gage": "^1.19.1",
+    "babel-cli": "^6.26.0",
+    "babel-preset-env": "^1.6.1",
+    "babel-register": "^6.26.0",
+    "chai": "^4.1.2",
+    "chai-string": "^1.4.0",
+    "mocha": "^4.1.0",
+    "sinon": "^4.1.3",
+    "sinon-chai": "^2.14.0"
   },
   "config": {},
   "engines": {
-    "node": ">=4.7.3"
+    "node": "^8.9.3"
   }
 }

src/main.js

@@ -11,7 +11,7 @@ const getLogger = () => {
 
 		logger.addConsole(process.env.CONSOLE_LOG_LEVEL || 'silly');
 
-		// only log to splunk in production
+		// log to splunk only in production
 		if (process.env.NODE_ENV === 'production' && process.env.SPLUNK_URL) {
 			logger.addSplunk(process.env.SPLUNK_URL, process.env.SPLUNK_LOG_LEVEL || 'warn');
 		}

test/lib/app-logger.test.js

@@ -35,7 +35,7 @@ describe('Logger', () => {
 
 	it('should be able to instantiate', () => {
 		const logger = new Logger();
-		logger.should.be.defined;
+		logger.should.not.equal(undefined);
 	});
 
 	describe('#log', () => {
@@ -64,7 +64,7 @@ describe('Logger', () => {
 			logSpy.should.always.have.been.calledWithExactly('info', 'a message', { foo: 'foo' });
 		});
 
-		it('should pass meta only through', () => {
+		it('should pass meta solely through', () => {
 			const logSpy = sinon.spy();
 			const winston = winstonStub({ log: logSpy });
 			const logger = new Logger({ winston });

test/lib/function-logger.test.js

@@ -19,7 +19,7 @@ describe('Logger', () => {
 
 	it('should be able to instantiate', () => {
 		const logger = new Logger();
-		logger.should.be.defined;
+		logger.should.not.equal(undefined);
 	});
 
 	describe('#log', () => {
@@ -50,7 +50,7 @@ describe('Logger', () => {
 			console.log.should.always.have.been.calledWithExactly('a message foo=foo level=info');
 		});
 
-		it('should pass meta only through', () => {
+		it('should pass meta solely through', () => {
 			const logger = new Logger();
 			logger.log('info', { foo: 'bar' });
 			console.log.should.always.have.been.calledWithExactly('foo=bar level=info');

test/setup.js

@@ -1,4 +1,4 @@
 require('babel-register')({
 	// just transpile the tests
-	only: filename => /n-logger\/test/.test(filename)
+	ignore: filename => !(/n-logger\/test/.test(filename))
 });