fix: add a fallback for root file/fullFile

This addresses an issue where the ouput from a test run with Cypress is not the
same as the test run with mocha directly. Specifically mocha does not set the
`file` property on the root suite and instead sets it on the sub suites or
tests whereas Cypress does the opposite.

CHANGELOG.md

@@ -1,6 +1,8 @@
 # mochawesome changelog
 
 ## [Unreleased]
+### Fixed
+- Add fallback when attempting to set `file` and `fullFile` on root suite. [#371](https://github.com/adamgruber/mochawesome/issues/371)
 
 ## [7.1.0] - 2022-02-24
 ### Added

src/mochawesome.js

@@ -163,12 +163,12 @@ function Mochawesome(runner, options) {
         // support `reportFilename` [name] replacement token
         if (rootSuite.suites.length === 1) {
           const firstSuite = rootSuite.suites[0];
-          rootSuite.file = firstSuite.file;
-          rootSuite.fullFile = firstSuite.fullFile;
+          rootSuite.file = firstSuite.file || rootSuite.file;
+          rootSuite.fullFile = firstSuite.fullFile || rootSuite.fullFile;
         } else if (!rootSuite.suites.length && rootSuite.tests.length) {
           const firstTest = this.runner.suite.tests[0];
-          rootSuite.file = firstTest.file;
-          rootSuite.fullFile = firstTest.fullFile;
+          rootSuite.file = firstTest.file || rootSuite.file;
+          rootSuite.fullFile = firstTest.fullFile || rootSuite.fullFile;
         }
 
         const obj = {

test/reporter.test.js

@@ -157,9 +157,34 @@ describe('Mochawesome Reporter', () => {
       subSuite.addTest(test);
       subSuite.file = 'testfile.js';
       runner.run(() => {
-        mochaReporter.output.results[0].suites[0].fullFile.should.equal(
-          'testfile.js'
-        );
+        mochaReporter.output.results[0].fullFile.should.equal('testfile.js');
+        done();
+      });
+    });
+
+    it('should handle root suite with file and test without', done => {
+      const test = makeTest('test', () => {});
+      suite.file = 'testfile.js';
+      suite.fullFile = 'testfile.js';
+      test.file = '';
+      test.fullFile = '';
+      suite.addTest(test);
+      suite.suites = [];
+      runner.run(() => {
+        mochaReporter.output.results[0].fullFile.should.equal('testfile.js');
+        done();
+      });
+    });
+
+    it('should handle root suite with file and subsuite without', done => {
+      const test = makeTest('test', () => {});
+      subSuite.addTest(test);
+      suite.file = 'testfile.js';
+      suite.fullFile = 'testfile.js';
+      subSuite.file = '';
+      subSuite.fullFile = '';
+      runner.run(() => {
+        mochaReporter.output.results[0].fullFile.should.equal('testfile.js');
         done();
       });
     });