Skip to content

Commit

Permalink
Fix broken test case
Browse files Browse the repository at this point in the history
  • Loading branch information
clayreimann committed Jul 12, 2023
1 parent 571a45e commit 4b954f0
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions index.js
Expand Up @@ -402,7 +402,7 @@ MochaJUnitReporter.prototype.removeInvalidCharacters = function(input){
*/
MochaJUnitReporter.prototype.flush = function(testsuites){
this._xml = this.getXml(testsuites);

var reportFilename = this.formatReportFilename(this._xml, testsuites);

this.writeXmlToDisk(this._xml, reportFilename);
Expand Down Expand Up @@ -432,10 +432,10 @@ MochaJUnitReporter.prototype.formatReportFilename = function(xml, testsuites) {
reportFilename = reportFilename.replace('[rootSuiteTitle]', this._options.rootSuiteTitle);
}
if (reportFilename.indexOf('[suiteFilename]') !== -1) {
reportFilename = reportFilename.replace('[suiteFilename]', testsuites[0]?.testsuite[0]?._attr.file ?? 'suiteFilename');
reportFilename = reportFilename.replace('[suiteFilename]', testsuites[0]?.testsuite[0]?._attr?.file ?? 'suiteFilename');
}
if (reportFilename.indexOf('[suiteName]') !== -1) {
reportFilename = reportFilename.replace('[suiteName]', testsuites[1]?.testsuite[0]?._attr.name ?? 'suiteName');
reportFilename = reportFilename.replace('[suiteName]', testsuites[1]?.testsuite[0]?._attr?.name ?? 'suiteName');
}

return reportFilename;
Expand Down
2 changes: 1 addition & 1 deletion test/mocha-junit-reporter-spec.js
Expand Up @@ -321,7 +321,7 @@ describe('mocha-junit-reporter', function() {
reporter.runner,
dir +
"results." +
(reporter._testsuites[0]?.testsuite[0]?._attr?.file ?? 'suiteFileName') +
(reporter._testsuites[0]?.testsuite[0]?._attr?.file ?? 'suiteFilename') +
".xml"
);
done();
Expand Down

0 comments on commit 4b954f0

Please sign in to comment.