Ensure that test time attribute is capped at 4 digits after the decimal

Author: clayreimann

index.js

@@ -388,17 +388,20 @@ MochaJUnitReporter.prototype.getXml = function(testsuites) {
     var _cases = suite.testsuite.slice(_casesIndex);
     var missingProps;
 
-    _suiteAttr.failures = 0;
     _suiteAttr.time = 0;
+    _suiteAttr.failures = 0;
     _suiteAttr.skipped = 0;
 
+    var suiteTime = 0;
     _cases.forEach(function(testcase) {
       var lastNode = testcase.testcase[testcase.testcase.length - 1];
 
       _suiteAttr.skipped += Number('skipped' in lastNode);
       _suiteAttr.failures += Number('failure' in lastNode);
-      _suiteAttr.time += testcase.testcase[0]._attr.time;
+      suiteTime += testcase.testcase[0]._attr.time;
+      testcase.testcase[0]._attr.time = testcase.testcase[0]._attr.time.toFixed(4);
     });
+    _suiteAttr.time = suiteTime.toFixed(4);
 
     if (antMode) {
       missingProps = ['system-out', 'system-err'];
@@ -417,8 +420,8 @@ MochaJUnitReporter.prototype.getXml = function(testsuites) {
     if (!_suiteAttr.skipped) {
       delete _suiteAttr.skipped;
     }
-
-    totalSuitesTime += _suiteAttr.time;
+    
+    totalSuitesTime += suiteTime;
     totalTests += _suiteAttr.tests;
   });
 
@@ -427,7 +430,7 @@ MochaJUnitReporter.prototype.getXml = function(testsuites) {
     var rootSuite = {
       _attr: {
         name: this._options.testsuitesTitle,
-        time: totalSuitesTime,
+        time: totalSuitesTime.toFixed(4),
         tests: totalTests,
         failures: stats.failures
       }

test/mocha-junit-reporter-spec.js

@@ -38,14 +38,14 @@ describe('mocha-junit-reporter', function() {
     });
 
     if (!options.skipPassedTests) {
-      runner.pass(new Test('Foo can weez the juice', 'can weez the juice', 1));
+      runner.pass(new Test('Foo can weez the juice', 'can weez the juice', 101));
     }
 
-    runner.fail(new Test('Bar can narfle the garthog', 'can narfle the garthog', 1), {
+    runner.fail(new Test('Bar can narfle the garthog', 'can narfle the garthog', 2002), {
       stack: options.invalidChar + 'expected garthog to be dead' + options.invalidChar
     });
 
-    runner.fail(new Test('Baz can behave like a flandip', 'can behave like a flandip', 1), {
+    runner.fail(new Test('Baz can behave like a flandip', 'can behave like a flandip', 30003), {
       name: 'BazError',
       message: 'expected baz to be masher, a hustler, an uninvited grasper of cone'
     });
@@ -54,7 +54,7 @@ describe('mocha-junit-reporter', function() {
       title: 'Another suite!',
       tests: [1]
     });
-    runner.pass(new Test('Another suite', 'works', 4));
+    runner.pass(new Test('Another suite', 'works', 400004));
 
     if (options && options.includePending) {
       runner.startSuite({

test/mock-results.js

@@ -8,7 +8,7 @@ module.exports = function(stats, options) {
           name: "Mocha Tests",
           tests: 3,
           failures: "2",
-          time: "0.007"
+          time: "432.1100"
         }
       },
       {
@@ -19,15 +19,15 @@ module.exports = function(stats, options) {
               timestamp: stats.start.toISOString().substr(0,stats.start.toISOString().indexOf('.')),
               tests: "2",
               failures: "2",
-              time: "0.003"
+              time: "32.1060"
             }
           },
           {
             testcase: {
               _attr: {
                 name: "Foo can weez the juice",
                 classname: "can weez the juice",
-                time: "0.001"
+                time: "0.1010"
               }
             }
           },
@@ -37,7 +37,7 @@ module.exports = function(stats, options) {
                 _attr: {
                   name: "Bar can narfle the garthog",
                   classname: "can narfle the garthog",
-                  time: "0.001"
+                  time: "2.0020"
                 }
               },
               {
@@ -57,7 +57,7 @@ module.exports = function(stats, options) {
                 _attr: {
                   name: "Baz can behave like a flandip",
                   classname: "can behave like a flandip",
-                  time: "0.001"
+                  time: "30.0030"
                 }
               },
               {
@@ -81,15 +81,15 @@ module.exports = function(stats, options) {
               timestamp: stats.start.toISOString().substr(0,stats.start.toISOString().indexOf('.')),
               tests: "1",
               failures: "0",
-              time: "0.004"
+              time: "400.0040"
             }
           },
           {
             testcase: {
               _attr: {
                 name: "Another suite",
                 classname: "works",
-                time: "0.004"
+                time: "400.0040"
               }
             }
           }
@@ -99,8 +99,8 @@ module.exports = function(stats, options) {
   };
 
   if (options && options.skipPassedTests) {
-    data.testsuites[0]._attr.time = "0.006";
-    data.testsuites[1].testsuite[0]._attr.time = "0.002";
+    data.testsuites[0]._attr.time = "432.0090";
+    data.testsuites[1].testsuite[0]._attr.time = "32.0050";
     data.testsuites[1].testsuite.splice(1, 1);
   }
 
@@ -137,7 +137,7 @@ module.exports = function(stats, options) {
             tests: "1",
             failures: "0",
             skipped: "1",
-            time: "0"
+            time: "0.0000"
           }
         },
         {
@@ -146,7 +146,7 @@ module.exports = function(stats, options) {
               _attr: {
                 name: "Pending suite",
                 classname: "pending",
-                time: "0"
+                time: "0.0000"
               }
             },
             {