Changed where long dash unicode test was writing. Should fix #246.

Author: Ændrew Rininsland

test/test.repo.js

@@ -97,7 +97,7 @@ describe('Github.Repository', function() {
   });
 });
 
-var repoTest = Date.now();
+var repoTest = Math.floor(Math.random() * (100000 - 0)) + 0;
 
 describe('Creating new Github.Repository', function() {
   var github = new Github({
@@ -228,10 +228,10 @@ describe('Creating new Github.Repository', function() {
   });
 
   it('should be able to write unicode to repo', function(done) {
-    repo.write('master', 'TEST.md', '\u2014', 'Long dash unicode', function(err) {
+    repo.write('master', 'TEST_unicode.md', '\u2014', 'Long dash unicode', function(err) {
       should.not.exist(err);
 
-      repo.read('master', 'TEST.md', function(err, obj) {
+      repo.read('master', 'TEST_unicode.md', function(err, obj) {
         should.not.exist(err);
         obj.should.equal('\u2014');
 

test/test.search.js

@@ -1,61 +1,59 @@
 '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("User API", function(t) {
-    var timeout = setTimeout(function () { t.fail(); }, 100000);
-    var github = new Github({
-        username: test_user.USERNAME,
-        password: test_user.PASSWORD,
-        auth: "basic"
-    });
-    // var user = github.getUser();
+// Use should flavour for Mocha
+var should = chai.should();
+chai.use(sinonChai);
 
+describe('Github.Search', function() {
+  var github = new Github({
+    username: test_user.USERNAME,
+    password: test_user.PASSWORD,
+    auth: "basic"
+  });
 
-    t.test('Search.repositories', function(q) {
-        var search = github.getSearch("tetris+language:assembly&sort=stars&order=desc");
-        var options = null;
+  it('should search.repositories', function(done) {
+    var search = github.getSearch('tetris+language:assembly&sort=stars&order=desc');
+    var options = null;
 
-        search.repositories(options, function (err) {
-            q.error(err, 'search repositories');
-            q.end();
-        });
+    search.repositories(options, function (err) {
+      should.not.exist(err);
+      done();
     });
+  });
 
-    t.test('Search.code', function(q) {
-        var search = github.getSearch("addClass+in:file+language:js+repo:jquery/jquery");
-        var options = null;
+  it('should search.code', function(done) {
+    var search = github.getSearch('addClass+in:file+language:js+repo:jquery/jquery');
+    var options = null;
 
-        search.code(options, function (err) {
-            q.error(err, 'search code');
-            q.end();
-        });
+    search.code(options, function (err) {
+      should.not.exist(err);
+      done();
     });
+  });
 
-    t.test('Search.issues', function(q) {
-        var search = github.getSearch("windows+label:bug+language:python+state:open&sort=created&order=asc");
-        var options = null;
+  it('should search.issues', function(done) {
+    var search = github.getSearch('windows+label:bug+language:python+state:open&sort=created&order=asc');
+    var options = null;
 
-        search.issues(options, function (err) {
-            q.error(err, 'search issues');
-            q.end();
-        });
+    search.issues(options, function (err) {
+      should.not.exist(err);
+      done();
     });
+  });
 
-    t.test('Search.users', function(q) {
-        var search = github.getSearch("tom+repos:%3E42+followers:%3E1000");
-        var options = null;
+  it('should search.users', function(done) {
+    var search = github.getSearch('tom+repos:%3E42+followers:%3E1000');
+    var options = null;
 
-        search.users(options, function (err) {
-            q.error(err, 'search users');
-            q.end();
-        });
+    search.users(options, function (err) {
+      should.not.exist(err);
+      done();
     });
-
-
-
-    clearTimeout(timeout);
-    t.end();
- });
+  });
+});

test/test.user.js

@@ -11,6 +11,8 @@ var should = chai.should();
 chai.use(sinonChai);
 
 describe('Github.User', function() {
+  this.timeout(8000); // Bit of a longer timeout
+
   var github = new Github({
     username: test_user.USERNAME,
     password: test_user.PASSWORD,
@@ -82,8 +84,6 @@ describe('Github.User', function() {
   });
 
   it('should show user\'s repos', function(done) {
-    this.timeout(8000); // Bit of a longer timeout
-
     user.userRepos(test_user.USERNAME, function(err) {
       should.not.exist(err);
       done();
@@ -126,8 +126,7 @@ describe('Github.User', function() {
   });
 
   it('should create a repo', function(done) {
-    this.timeout(8000); // Bit of a longer timeout
-    var repoTest = Date.now();
+    var repoTest = Math.floor(Math.random() * (100000 - 0)) + 0;
     var github = new Github({
       username: test_user.USERNAME,
       password: test_user.PASSWORD,