fix: more consistent cache (#385)

Author: evilebottnawi

package-lock.json

@@ -228,14 +228,12 @@ class UglifyJsPlugin {
             };
 
             if (this.options.cache) {
-              const { outputPath } = compiler;
               const defaultCacheKeys = {
                 // eslint-disable-next-line global-require
                 'uglify-js': require('uglify-js/package.json').version,
                 // eslint-disable-next-line global-require
                 'uglifyjs-webpack-plugin': require('../package.json').version,
                 'uglifyjs-webpack-plugin-options': this.options,
-                path: `${outputPath ? `${outputPath}/` : ''}${file}`,
                 hash: crypto
                   .createHash('md4')
                   .update(input)

src/index.js

@@ -1,5 +1,3 @@
-import path from 'path';
-
 import cacache from 'cacache';
 import findCacheDir from 'find-cache-dir';
 
@@ -16,10 +14,11 @@ describe('when applied with `cache` option', () => {
   beforeEach(() => {
     compiler = createCompiler({
       entry: {
-        one: `${__dirname}/fixtures/entry.js`,
-        two: `${__dirname}/fixtures/entry.js`,
-        three: `${__dirname}/fixtures/entry.js`,
-        four: `${__dirname}/fixtures/entry.js`,
+        one: `${__dirname}/fixtures/cache.js`,
+        two: `${__dirname}/fixtures/cache-1.js`,
+        three: `${__dirname}/fixtures/cache-2.js`,
+        four: `${__dirname}/fixtures/cache-3.js`,
+        five: `${__dirname}/fixtures/cache-4.js`,
       },
     });
 
@@ -105,18 +104,6 @@ describe('when applied with `cache` option', () => {
             // Make sure that we cached files
             expect(cacheKeys.length).toBe(countAssets);
 
-            cacheKeys.forEach((cacheEntry) => {
-              // eslint-disable-next-line no-new-func
-              const cacheEntryOptions = new Function(
-                `'use strict'\nreturn ${cacheEntry}`
-              )();
-              const basename = path.basename(cacheEntryOptions.path);
-
-              expect([basename, cacheEntryOptions.hash]).toMatchSnapshot(
-                basename
-              );
-            });
-
             cacache.get.mockClear();
             cacache.put.mockClear();
           })
@@ -192,18 +179,6 @@ describe('when applied with `cache` option', () => {
             // Make sure that we cached files
             expect(cacheKeys.length).toBe(countAssets);
 
-            cacheKeys.forEach((cacheEntry) => {
-              // eslint-disable-next-line no-new-func
-              const cacheEntryOptions = new Function(
-                `'use strict'\nreturn ${cacheEntry}`
-              )();
-              const basename = path.basename(cacheEntryOptions.path);
-
-              expect([basename, cacheEntryOptions.hash]).toMatchSnapshot(
-                basename
-              );
-            });
-
             cacache.get.mockClear();
             cacache.put.mockClear();
           })
@@ -294,15 +269,11 @@ describe('when applied with `cache` option', () => {
               const cacheEntryOptions = new Function(
                 `'use strict'\nreturn ${cacheEntry}`
               )();
-              const basename = path.basename(cacheEntryOptions.path);
 
               expect(cacheEntryOptions.myCacheKey).toBe(1);
               expect(cacheEntryOptions.myCacheKeyBasedOnFile).toMatch(
                 /file-(.+)?\.js/
               );
-              expect([basename, cacheEntryOptions.hash]).toMatchSnapshot(
-                basename
-              );
             });
 
             cacache.get.mockClear();

test/__snapshots__/cache-option.test.js.snap

@@ -0,0 +1 @@
+module.exports = 'string';

test/__snapshots__/test-option.test.js.snap

@@ -0,0 +1 @@
+module.exports = function getRegExp() { return /test/ };

test/cache-option.test.js

@@ -0,0 +1 @@
+module.exports = function foo() {};

test/fixtures/cache-1.js

@@ -0,0 +1,9 @@
+// foo
+/* @preserve*/
+// bar
+var a = 2 + 2;
+
+module.exports = function Foo() {
+  var b = 2 + 2;
+  console.log(b + 1 + 2);
+};

test/fixtures/cache-2.js

@@ -0,0 +1,9 @@
+// foo
+/* @preserve*/
+// bar
+var a = 2 + 2;
+
+module.exports = function Foo() {
+  var b = 2 + 2;
+  console.log(b + 1 + 2);
+};