Merge pull request #12 from caseyWebb/fix-coverage-reporting

Fix CodeClimate coverage reporting

.travis.yml

@@ -11,4 +11,4 @@ before_script:
   - sh -e /etc/init.d/xvfb start
 
 after_script:
-  - node node_modules/codeclimate-test-reporter/bin/codeclimate.js < coverage/lcov.txt
+  - codeclimate-test-reporter < coverage/lcov.txt

karma.travis.js

@@ -21,9 +21,11 @@ module.exports = function(config) {
     reporters: ['dots', 'coverage'],
 
     coverageReporter: {
-      type: 'lcovonly',
-      dir: 'coverage/',
-      file: 'lcov.txt'
+      dir : 'coverage/',
+      reporters: [
+        { type: 'html', subdir: 'html' },
+        { type: 'lcovonly', subdir: '.', file: 'lcov.txt' }
+      ]
     },
 
     webpack: {

package.json

@@ -43,6 +43,7 @@
     "babel-loader": "^6.2.0",
     "babel-preset-es2015": "^6.1.18",
     "bootstrap": "^3.3.6",
+    "codeclimate-test-reporter": "^0.1.1",
     "css-loader": "^0.23.0",
     "es6-promise": "^3.0.2",
     "escape-html": "^1.0.3",

test.js

@@ -98,7 +98,7 @@ function runTests(t, config) {
     .step((done) => {
       const foo = router.query.get('foo', 'foo')
       t.equal(foo(), 'bar', 'querystring param is writeable')
-      t.ok(window.location.href.indexOf('%5Bfoo%5D=bar') > -1, 'non-default params are written to the querystring')
+      t.ok(decodeURIComponent(window.location.href).indexOf('[foo]=bar') > -1, 'non-default params are written to the querystring')
 
       const killMe = foo.subscribe(() => {
         t.equal(router.query.get('foo')(), 'foo', 'ctx.query.clear sets the params to defaults')
@@ -168,7 +168,7 @@ function runTests(t, config) {
     .step(() => {
       const activeLink = $('#should-be-active', dom)
       t.ok(activeLink.hasClass('active-path'), 'path binding sets `active` class')
-      
+
       resolve()
     })
   })