refactoring and adding more nodejs versions to travis

Author: goenning

.eslintrc.json

@@ -6,6 +6,7 @@
     "mocha": true
   },
   "rules": {
-    "semi": [2, "always"]
+    "semi": [2, "always"],
+    "no-unused-vars": ["error", { "vars": "all", "args": "none" }]
   }
 }

.travis.yml

@@ -1,4 +1,7 @@
 language: node_js
 node_js:
-  - "4.2.4"
+  - "0.10"
+  - "0.12"
+  - "4"
+  - "5"
   - "stable"

examples/express/server.js

@@ -17,8 +17,8 @@ app.get('/', function (req, res) {
 
 app.get('/multi-query', function (req, res) {
 	pg.connect(connString, function (err, client, done) {
-		client.query('SELECT pg_sleep(1)', [], function (err) {
-			client.query('SELECT $1::int AS number', ['2'], function (err) {
+		client.query('SELECT pg_sleep(1)', [], function (err, result) {
+			client.query('SELECT $1::int AS number', ['2'], function (err, result) {
 				done();
 				res.render('multi-query');
 			});

lib/miniprofiler.js

@@ -16,6 +16,7 @@ var qs = require('querystring');
 var url = require('url');
 var utils = require('./utils.js');
 var uuid = require('node-uuid');
+var debug = require('debug')('miniprofiler');
 
 _.templateSettings = {
 	interpolate: /\{(.+?)\}/g
@@ -418,7 +419,7 @@ function describePerformance(root, request) {
 function diff(start, stop){
 	if (!stop) {
 		stop = process.hrtime();
-		console.error('missing stop, using', stop);
+		debug('missing stop, using', stop);
 	}
 	var deltaSecs = stop[0] - start[0];
 	var deltaNanoSecs = stop[1] - start[1];

package.json

@@ -5,8 +5,7 @@
   "main": "lib/miniprofiler.js",
   "scripts": {
     "lint": "eslint .",
-    "test": "mocha tests/",
-    "coverage": "istanbul cover node_modules/mocha/bin/_mocha tests/"
+    "test": "istanbul cover node_modules/mocha/bin/_mocha tests/"
   },
   "repository": {
     "type": "git",
@@ -16,6 +15,7 @@
   "license": "Apache-2.0",
   "readmeFilename": "README.md",
   "dependencies": {
+    "debug": "^2.2.0",
     "node-uuid": "^1.4.7",
     "underscore": "1.5.2"
   },

tests/utils-test.js

@@ -2,24 +2,6 @@ var expect = require('chai').expect;
 var utils = require('../lib/utils.js');
 
 describe('Utils Tests', function() {
-  [
-    [ 'apple', 'ap' ],
-    [ 'Apple', 'Ap' ]
-  ].forEach((testCase) => {
-    it(`${testCase[0]} should start with ${testCase[1]}`, function() {
-      expect(testCase[0].startsWith(testCase[1])).to.be.true;
-    });
-  });
-
-  [
-    [ 'Apple', 'ap' ],
-    [ 'apple', 'at' ]
-  ].forEach((testCase) => {
-    it(`${testCase[0]} should not start with ${testCase[1]}`, function() {
-      expect(testCase[0].startsWith(testCase[1])).to.be.false;
-    });
-  });
-
   [
     [ 'a=1&b=2', 'a=1&b=2' ],
     [ '"Hello"', '"Hello"' ],