Rewrote all unit tests to use Mocha Chai instead of Tape.

Author: Ændrew Rininsland

package.json

@@ -8,6 +8,7 @@
     "xmlhttprequest": "~1.7.0"
   },
   "devDependencies": {
+    "chai": "^3.4.0",
     "codecov": "^1.0.1",
     "gulp": "^3.9.0",
     "gulp-jscs": "^3.0.1",
@@ -18,12 +19,17 @@
     "jshint": "^2.5.8",
     "jshint-stylish": "^2.0.1",
     "plato": "^1.4.0",
-    "tape": "^3.0.3"
+    "karma": "^0.13.14",
+    "karma-coverage": "^0.5.3",
+    "karma-mocha": "^0.2.0",
+    "mocha": "^2.3.3",
+    "sinon": "^1.17.2",
+    "sinon-chai": "^2.8.0"
   },
   "scripts": {
-    "test": "tape test/*.js && npm run-script jshint",
+    "test": "mocha test/*.js && npm run-script jshint",
     "jshint": "jshint github.js test/",
-    "codecov": "istanbul cover tape test/*.js && cat ./coverage/coverage.json | ./node_modules/.bin/codecov"
+    "codecov": "istanbul cover ./node_modules/mocha/bin/_mocha -- -R spec && cat ./coverage/lcov.info | ./node_modules/.bin/codecov"
   },
   "repository": {
     "type": "git",

test/test.auth.js

@@ -1,41 +1,45 @@
 'use strict';
 
-var test = require('tape'); //jshint ignore:line
-var Github = require("../");
+// module dependencies
+var chai = require('chai'),
+    sinonChai = require('sinon-chai');
+
+var Github = require('../');
 var test_user = require('./user.json');
 
-test("Basic Auth - Pass", function(t) {
-  var timeout = setTimeout(function () { t.fail(); }, 10000);
+// Use should flavour for Mocha
+var should = chai.should();
+chai.use(sinonChai);
+
+describe('Github constructor', function() {
   var github = new Github({
     username: test_user.USERNAME,
     password: test_user.PASSWORD,
-    auth: "basic"
+    auth: 'basic'
   });
   var user = github.getUser();
-  
-  user.notifications(function(err) {
-    t.error(err, 'user is authd');
+
+  it('should authenticate and return no errors', function(done){
+    user.notifications(function(err){
+      should.not.exist(err);
+      done();
+    });
   });
-  
-  clearTimeout(timeout);
-  t.end(); 
 });
 
-test("Basic Auth - Fail", function(t) {
-  var timeout = setTimeout(function () { t.fail(); }, 10000);
+describe('Github constructor (failing case)', function() {
   var github = new Github({
     username: test_user.USERNAME,
     password: 'fake124',
-    auth: "basic"
+    auth: 'basic'
   });
   var user = github.getUser();
 
-  user.notifications(function(err) {
-      t.ok(err, 'user is not authd');
-      t.equals(JSON.parse(err.request.responseText).message, 'Bad credentials', 'confirm error');
+  it('should fail authentication and return err', function(done){
+    user.notifications(function(err){
+      err.request.status.should.equal(401, 'Return 401 status for bad auth');
+      JSON.parse(err.request.responseText).message.should.equal('Bad credentials');
+      done();
+    });
   });
-
-  clearTimeout(timeout);
-  t.end();
-
 });

test/test.gist.js

@@ -1,36 +1,40 @@
 'use strict';
 
-var test = require('tape'); //jshint ignore:line
-var Github = require("../");
+// module dependencies
+var chai = require('chai'), sinonChai = require('sinon-chai');
+
+// GitHub data
+var Github = require('../');
 var test_user = require('./user.json');
 
-test("Issues API", function(t) {
-	var github = new Github({
-      username: test_user.USERNAME,
-      password: test_user.PASSWORD,
-      auth: "basic"
-    });
+// Use should flavour for Mocha
+var should = chai.should();
+chai.use(sinonChai);
 
-    var issues = github.getIssues('mikedeboertest', 'TestRepo');
+describe('Github.Issue', function() {
+  var github = new Github({
+    username : test_user.USERNAME,
+    password : test_user.PASSWORD,
+    auth : 'basic'
+  });
 
-    t.test('issues.list', function(q) {
-        issues.list({},function(err, issues) {
-			q.error(err);
-			t.equals(issues.length > 0, true, 'Issues!');
-			q.end();
-        });
-        t.end();
-    });
+  var issues = github.getIssues('mikedeboertest', 'TestRepo');
 
-    t.test('issues.comment', function(q) {
-        issues.list({},function(err, issuesList) {
-			issues.comment(issuesList[0], 'Comment test', function(err, res){
-				q.error(err);
-				t.equals(res.body, 'Comment test', 'Comments!');
-				q.end();
-	        });  
-        });
+  it('should list issues', function(done) {
+    issues.list({}, function(err, issues) {
+      should.not.exist(err);
+			issues.should.have.length.above(0);
+			done();
     });
+  });
 
-
-});
+  it('should post issue comment', function(done) {
+    issues.list({}, function(err, issuesList) {
+      issues.comment(issuesList[0], 'Comment test', function(err, res) {
+        should.not.exist(err);
+        res.body.should.equal('Comment test');
+        done();
+      });
+    });
+  });
+});