Drop support for misspelled option suiteTitleSeparatedBy

michaelleeallen · Jun 7, 2020 · 3d1909d · 3d1909d
1 parent fababe6
commit 3d1909d
Show file tree

Hide file tree

Showing 5 changed files with 3 additions and 14 deletions.
diff --git a/README.md b/README.md
@@ -187,7 +187,6 @@ output line 2
 | toConsole                      | `false`                | if set to a truthy value the produced XML will be logged to the console                                                 |
 | useFullSuiteTitle              | `false`                | if set to a truthy value nested suites' titles will show the suite lineage                                              |
 | suiteTitleSeparatedBy          | ` ` (space)            | the character to use to separate nested suite titles. (defaults to ' ', '.' if in jenkins mode)                         |
-| suiteTitleSeparedBy            | ` ` (space)            | deprecated. use `suiteTitleSeparatedBy`                                                                                 |
 | testCaseSwitchClassnameAndName | `false`                | set to a truthy value to switch name and classname values                                                               |
 | rootSuiteTitle                 | `Root Suite`           | the name for the root suite. (defaults to 'Root Suite')                                                                 |
 | testsuitesTitle                | `Mocha Tests`          | the name for the `testsuites` tag (defaults to 'Mocha Tests')                                                           |

diff --git a/index.js b/index.js
@@ -76,7 +76,7 @@ function configureDefaults(options) {
     updateOptionsForJenkinsMode(config);
   }
 
-  config.suiteTitleSeparatedBy = config.suiteTitleSeparatedBy || config.suiteTitleSeparedBy || ' ';
+  config.suiteTitleSeparatedBy = config.suiteTitleSeparatedBy || ' ';
 
   return config;
 }

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "mocha-junit-reporter",
-  "version": "1.23.3",
+  "version": "2.0.0",
   "description": "A JUnit reporter for mocha.",
   "main": "index.js",
   "scripts": {

diff --git a/test/mocha-junit-reporter-spec.js b/test/mocha-junit-reporter-spec.js
@@ -343,16 +343,6 @@ describe('mocha-junit-reporter', function() {
       });
     });
 
-    it('respects the misspelled "suiteTitleSeparatedBy" option', function(done) {
-      var reporter = createReporter({useFullSuiteTitle: true, suiteTitleSeparedBy: '.'});
-      executeTestRunner(reporter.runner, function() {
-        expect(suiteName(reporter._testsuites[0])).to.equal('');
-        expect(suiteName(reporter._testsuites[1])).to.equal('Root Suite.Foo Bar');
-        expect(suiteName(reporter._testsuites[2])).to.equal('Root Suite.Another suite!');
-        done();
-      });
-    });
-
     function suiteName(suite) {
       return suite.testsuite[0]._attr.name;
     }