Skip asset copying and report render when html=false (#93)

adamgruber · Apr 18, 2018 · b97fc41 · b97fc41
1 parent 033fbf6
commit b97fc41
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,8 @@
 # mochawesome-report-generator changelog
 
 ## Unreleased
+### Fixed
+- Skip copying assets and rendering HTML when `html` option is `false` [mochawesome #237](https://github.com/adamgruber/mochawesome/issues/237)
 
 ## [3.1.1] / 2018-01-28
 ### Added

diff --git a/lib/src/main.js b/lib/src/main.js
@@ -231,6 +231,11 @@ function prepare(reportData, opts) {
   // Get the options
   const reportOptions = getOptions(opts);
 
+  // Stop here if we're not generating an HTML report
+  if (!reportOptions.saveHtml) {
+    return { reportOptions };
+  }
+
   // Get the assets
   const assets = getAssets(reportOptions);
 

diff --git a/test/spec/lib/main.test.js b/test/spec/lib/main.test.js
@@ -71,6 +71,18 @@ describe('lib/main', () => {
       return expect(promise).to.become([ null, expectedJsonFile ]);
     });
 
+    it('does not copy assets when saving only json', () => {
+      opts.saveHtml = false;
+      opts.saveJson = true;
+      const expectedJsonFile = path.resolve(process.cwd(), 'test', 'test.json');
+      outputFileStub
+        .onCall(0).resolves('')
+        .onCall(1).resolves(expectedJsonFile);
+      return mareport.create(testData, opts).then(() => {
+        expect(copySyncStub.called).to.equal(false);
+      });
+    });
+
     it('saves html and json', () => {
       opts.saveJson = true;
       const expectedHtmlFile = path.resolve(process.cwd(), 'test', 'test.html');