Breaking: Drop node <0.10 support, switch to mocha + expect for testing

Author: phated

.eslintrc

@@ -0,0 +1,3 @@
+{
+  "extends": "gulp"
+}

.jscsrc

@@ -0,0 +1,3 @@
+{
+  "preset": "gulp"
+}

.travis.yml

@@ -1,8 +1,10 @@
+sudo: false
 language: node_js
 node_js:
-  - "0.7"
-  - "0.8"
-  - "0.9"
-  - "0.10"
+  - '6'
+  - '5'
+  - '4'
+  - '0.12'
+  - '0.10'
 after_script:
-  - npm run coveralls
+  - npm run coveralls

index.js

@@ -1,9 +1,14 @@
 var path = require('path');
 
 module.exports = function(npath, ext) {
-  if (typeof npath !== 'string') return npath;
-  if (npath.length === 0) return npath;
+  if (typeof npath !== 'string') {
+    return npath;
+  }
 
-  var nFileName = path.basename(npath, path.extname(npath))+ext;
+  if (npath.length === 0) {
+    return npath;
+  }
+
+  var nFileName = path.basename(npath, path.extname(npath)) + ext;
   return path.join(path.dirname(npath), nFileName);
-};
+};

package.json

@@ -8,17 +8,21 @@
   "main": "./index.js",
   "dependencies": {},
   "devDependencies": {
-    "mocha": "~1.17.0",
-    "should": "~3.1.0",
-    "mocha-lcov-reporter": "~0.0.1",
-    "coveralls": "~2.6.1",
-    "istanbul": "~0.2.3",
-    "rimraf": "~2.2.5",
-    "jshint": "~2.4.1"
+    "eslint": "^1.10.3",
+    "eslint-config-gulp": "^2.0.0",
+    "expect": "^1.16.0",
+    "istanbul": "^0.4.3",
+    "istanbul-coveralls": "^1.0.3",
+    "jscs": "^2.3.5",
+    "jscs-preset-gulp": "^1.0.0",
+    "mocha": "^2.4.5"
   },
   "scripts": {
-    "test": "mocha --reporter spec && jshint",
-    "coveralls": "istanbul cover _mocha --report lcovonly -- -R spec && cat ./coverage/lcov.info | coveralls && rm -rf ./coverage"
+    "lint": "eslint . && jscs index.js test/",
+    "pretest": "npm run lint",
+    "test": "mocha --async-only",
+    "cover": "istanbul cover node_modules/mocha/bin/_mocha --report lcovonly",
+    "coveralls": "npm run cover && istanbul-coveralls"
   },
   "engines": {
     "node": ">= 0.4"

test/main.js

@@ -1,51 +1,50 @@
-var replaceExt = require('../');
+'use strict';
+
 var path = require('path');
-var should = require('should');
-require('mocha');
+
+var expect = require('expect');
+
+var replaceExt = require('../');
 
 describe('replace-ext', function() {
-  it('should return a valid replaced extension on nested', function(done) {
+
+  it('returns a valid replaced extension on long path', function(done) {
     var fname = path.join(__dirname, './fixtures/test.coffee');
     var expected = path.join(__dirname, './fixtures/test.js');
-    var nu = replaceExt(fname, '.js');
-    should.exist(nu);
-    nu.should.equal(expected);
+    var result = replaceExt(fname, '.js');
+    expect(result).toEqual(expected);
     done();
   });
 
-  it('should return a valid replaced extension on flat', function(done) {
+  it('returns a valid replaced extension on basename', function(done) {
     var fname = 'test.coffee';
     var expected = 'test.js';
-    var nu = replaceExt(fname, '.js');
-    should.exist(nu);
-    nu.should.equal(expected);
+    var result = replaceExt(fname, '.js');
+    expect(result).toEqual(expected);
     done();
   });
 
   it('should not return a valid replaced extension on empty string', function(done) {
     var fname = '';
     var expected = '';
-    var nu = replaceExt(fname, '.js');
-    should.exist(nu);
-    nu.should.equal(expected);
+    var result = replaceExt(fname, '.js');
+    expect(result).toEqual(expected);
     done();
   });
 
-  it('should return a valid removed extension on nested', function(done) {
+  it('returns a valid removed extension on long path', function(done) {
     var fname = path.join(__dirname, './fixtures/test.coffee');
     var expected = path.join(__dirname, './fixtures/test');
-    var nu = replaceExt(fname, '');
-    should.exist(nu);
-    nu.should.equal(expected);
+    var result = replaceExt(fname, '');
+    expect(result).toEqual(expected);
     done();
   });
 
-  it('should return a valid added extension on nested', function(done) {
+  it('returns a valid added extension on long path', function(done) {
     var fname = path.join(__dirname, './fixtures/test');
     var expected = path.join(__dirname, './fixtures/test.js');
-    var nu = replaceExt(fname, '.js');
-    should.exist(nu);
-    nu.should.equal(expected);
+    var result = replaceExt(fname, '.js');
+    expect(result).toEqual(expected);
     done();
   });
 });