feature: Strip ANSI sequences from all titles (#60)

* Stripped ANSI sequences from all titles
* dependencies: Added strip-ansi v4.X

index.js

@@ -7,6 +7,7 @@ var path = require('path');
 var debug = require('debug')('mocha-junit-reporter');
 var mkdirp = require('mkdirp');
 var md5 = require('md5');
+var stripAnsi = require('strip-ansi');
 
 module.exports = MochaJUnitReporter;
 
@@ -31,9 +32,9 @@ function configureDefaults(options) {
 
 function defaultSuiteTitle(suite) {
   if (suite.root && suite.title === '') {
-      return this._options.rootSuiteTitle;
+      return stripAnsi(this._options.rootSuiteTitle);
   }
-  return suite.title;
+  return stripAnsi(suite.title);
 }
 
 function fullSuiteTitle(suite) {
@@ -49,7 +50,7 @@ function fullSuiteTitle(suite) {
     parent = parent.parent;
   }
 
-  return title.join(this._options.suiteTitleSeparatedBy);
+  return stripAnsi(title.join(this._options.suiteTitleSeparatedBy));
 }
 
 function isInvalidSuite(suite) {
@@ -186,9 +187,9 @@ MochaJUnitReporter.prototype.getTestcaseData = function(test, err) {
   var config = {
     testcase: [{
       _attr: {
-        name: this._options.testCaseSwitchClassnameAndName ? test.title : test.fullTitle(),
+        name: this._options.testCaseSwitchClassnameAndName ? stripAnsi(test.title) : stripAnsi(test.fullTitle()),
         time: (typeof test.duration === 'undefined') ? 0 : test.duration / 1000,
-        classname: this._options.testCaseSwitchClassnameAndName ? test.fullTitle() : test.title
+        classname: this._options.testCaseSwitchClassnameAndName ? stripAnsi(test.fullTitle()) : stripAnsi(test.title)
       }
     }]
   };

package.json

@@ -33,6 +33,7 @@
     "debug": "^2.2.0",
     "md5": "^2.1.0",
     "mkdirp": "~0.5.1",
+    "strip-ansi": "^4.0.0",
     "xml": "^1.0.0"
   },
   "peerDependencies": {

test/mocha-junit-reporter-spec.js

@@ -189,6 +189,13 @@ describe('mocha-junit-reporter', function() {
     verifyMochaFile(filePath);
   });
 
+  it('creates valid XML report even if title contain ANSI character sequences', function() {
+    createReporter({mochaFile: 'test/mocha.xml'});
+    executeTestRunner({title: 'Foo Bar module'});
+
+    verifyMochaFile(filePath);
+  });
+
   it('outputs pending tests if "includePending" is specified', function() {
     createReporter({mochaFile: 'test/mocha.xml', includePending: true});
     executeTestRunner({includePending: true});