Fix xunit reporter

Requires that the Test objects have a 'state'
wking · Apr 25, 2015 · 06efe96 · 06efe96
1 parent 5d8892e
commit 06efe96
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/index.js b/index.js
@@ -37,7 +37,7 @@ function Formatter (type, options) {
   }
 
   var runner = this.runner = new Runner(options)
-  this.reporter = new reporters[type](this.runner)
+  this.reporter = new reporters[type](this.runner, {})
   Writable.call(this, options)
 
   runner.on('end', function () {

diff --git a/lib/reporters/xunit.js b/lib/reporters/xunit.js
@@ -57,7 +57,7 @@ function XUnit(runner, options) {
 
   runner.on('end', function(){
     self.write(tag('testsuite', {
-        name: 'Mocha Tests'
+        name: 'TAP Tests'
       , tests: stats.tests
       , failures: stats.failures
       , errors: stats.failures

diff --git a/lib/test.js b/lib/test.js
@@ -7,6 +7,9 @@ function Test (result, parent) {
   this._slow = 75
   this.duration = result.time
   this.title = result.name
+  this.state = result.ok ? 'pass' : 'failed'
+  this.pending = result.todo || result.skip || false
+
   Object.defineProperty(this, 'parent', {
     value: parent,
     writable: true,