Merge pull request #54 from dblock/test-coverage

Added test coverage.

.gitignore

@@ -2,3 +2,4 @@ node_modules
 project.production.json
 .env.json
 .DS_Store
+coverage

.travis.yml

@@ -1,9 +1,15 @@
 language: node_js
 
 node_js:
-  - "4.3"
+  - 4.3
 
 before_script:
   - npm run-script danger
 
+script:
+  - make coverage
+
+after_script:
+  - cat ./coverage/lcov.info | ./node_modules/coveralls/bin/coveralls.js
+
 email: false

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ### 0.3.1 (Next)
 
+* [#54](https://github.com/artsy/elderfield/pull/54): Added test code coverage - [@dblock](https://github.com/dblock).
 * Your contribution here.
 
 ### 0.3.0 (2017/1/9)

Makefile

@@ -22,5 +22,9 @@ deploy: build
 test: build
 	ENV=test $(KEYS) ./node_modules/mocha/bin/mocha
 
+# Test
+coverage: build
+	ENV=test $(KEYS) npm run coverage
+
 production-deploy: build
 	apex deploy --env=production

README.md

@@ -1,6 +1,7 @@
 # Elderfield
 
 [![Build Status](https://travis-ci.org/artsy/elderfield.svg?branch=master)](https://travis-ci.org/artsy/elderfield)
+[![Coverage Status](https://coveralls.io/repos/github/artsy/elderfield/badge.svg?branch=master)](https://coveralls.io/github/artsy/elderfield?branch=master)
 
 [![](images/echo-dot-2nd-gen.jpg)](https://developer.amazon.com/alexa)
 

package.json

@@ -15,7 +15,8 @@
   "scripts": {
     "start": "make s",
     "test": "make test",
-    "danger": "./node_modules/.bin/danger"
+    "danger": "./node_modules/.bin/danger",
+    "coverage": "./node_modules/.bin/istanbul cover ./node_modules/.bin/_mocha -- -R spec"
   },
   "dependencies": {
     "alexa-app-server": "2.2.4"
@@ -26,6 +27,8 @@
     "chai-http": "3.0.0",
     "chai-string": "1.3.0",
     "danger": "0.7.5",
-    "jest-environment-node": "18.0.0"
+    "jest-environment-node": "18.0.0",
+    "istanbul": "0.4.5",
+    "coveralls": "^2.11.15"
   }
 }

test/setup.js

@@ -3,4 +3,7 @@ expect = chai.expect;
 chai.use(require('chai-string'));
 chai.use(require('chai-http'));
 
+// for code coverage instrumentation
+require('../functions/artsy/index.js');
+
 server = require('../server');