Fix test failures from v5 upgrade

Author: jall

test/helpers/mock-runner.js

@@ -1,39 +1,52 @@
-'use strict';
+"use strict"
 
-var EventEmitter = require('events').EventEmitter;
-var util = require('util');
+var EventEmitter = require("events").EventEmitter
+var util = require("util")
 
 // mock test runner
 function Runner() {
-  Runner.super_.call(this);
+  Runner.super_.call(this)
 }
 
-util.inherits(Runner, EventEmitter);
+util.inherits(Runner, EventEmitter)
 
 Runner.prototype.start = function() {
-  this.emit('start');
-};
+  this.emit("start")
+}
 
 Runner.prototype.end = function() {
-  this.emit('end');
-};
+  this.emit("end")
+}
 
 Runner.prototype.startSuite = function(suite) {
-  this.emit('suite', suite);
-};
+  suite.suites = suite.suites || []
+  suite.tests = suite.tests || []
+
+  if (this._currentSuite) {
+    suite.parent = this._currentSuite
+  }
+
+  this._currentSuite = suite
+  this.emit("suite", suite)
+}
 
 Runner.prototype.pass = function(test) {
-  this.emit('pass', test);
-  this.endTest();
-};
+  this.emit("pass", test)
+  this.endTest()
+}
 
 Runner.prototype.fail = function(test, reason) {
-  this.emit('fail', test, reason);
-  this.endTest();
-};
+  this.emit("fail", test, reason)
+  this.endTest()
+}
+
+Runner.prototype.pending = function(test) {
+  this.emit("pending", test)
+  this.endTest()
+}
 
 Runner.prototype.endTest = function() {
-  this.emit('end test');
-};
+  this.emit("end test")
+}
 
-module.exports = Runner;
+module.exports = Runner

test/helpers/mock-test.js

@@ -1,15 +1,17 @@
-'use strict';
+"use strict"
 
 function Test(fullTitle, title, duration) {
-  
-  this.title = title;
-  this.duration = duration;
-  
-  this.fullTitle = function() { 
-    return fullTitle; 
-  };
-  
-  this.slow = function() { };
+  return {
+    title: title,
+    duration: duration,
+    fullTitle: function() {
+      return fullTitle
+    },
+    titlePath: function() {
+      return [fullTitle]
+    },
+    slow: function() {},
+  }
 }
 
-module.exports = Test
+module.exports = Test

test/index.js

@@ -10,74 +10,76 @@ var fs = require('fs');
 
 
 describe('mocha-circleci-reporter', function() {
-    
+
     var runner;
-    var file = './test/output/console.xml';
-    
+    var file = __dirname + '/output/console.xml';
+
     beforeEach(function() {
         runner = new Runner();
     });
-    
+
     afterEach(function(){
-        fs.unlinkSync(file);
+        if (fs.existsSync(file)) {
+            fs.unlinkSync(file);
+        }
     })
-    
+
     it('should output spec to stdout', function() {
-        
+
         new Reporter(runner, {
-          reporterOptions: { mochaFile: 'test/output/console.xml' }
+          reporterOptions: { mochaFile: file }
         });
-        
+
         var stdout = testConsole.stdout.inspect();
         try{
           executeTestRunner();
         }
-        finally{            
+        finally{
           stdout.restore();
         }
-        
+
         assert(stdout.output[1], '\u001b[0mFoo Bar module\u001b[0m\n');
     });
-    
+
     it('should output junit file', function() {
-        
+
         new Reporter(runner, {
-          reporterOptions: { mochaFile: 'test/output/console.xml' }
+          reporterOptions: { mochaFile: file }
         });
-        
+
         var stdout = testConsole.stdout.inspect();
         try{
           executeTestRunner();
-          
+
           assert(fs.existsSync(file));
         }
-        finally{            
+        finally{
           stdout.restore();
         }
     });
-    
+
     function executeTestRunner(char){
-        
+
         char = char || '';
         runner.start();
 
         runner.startSuite({
           title: 'Foo Bar module',
           tests: [1, 2]
         });
-        
+
         runner.pass(new Test('Foo can weez the juice', 'can weez the juice', 1));
         runner.fail(new Test('Bar can narfle the garthog', 'can narfle the garthog', 1), {
           message: char + 'expected garthog to be dead' + char
         });
-    
+
         runner.startSuite({
           title: 'Another suite!',
           tests: [1]
         });
         runner.pass(new Test('Another suite', 'works', 4));
-    
+
         runner.end();
     }
-        
-});
+
+});