test: add test to reproduce the issue #190

webdiscus · webdiscus · commit 05f66f4ce68c · 2026-05-31T23:45:53.000+02:00
diff --git a/test/issue.test.js b/test/issue.test.js
@@ -3,7 +3,7 @@ import os from 'os';
 import path from 'path';
 import webpack from 'webpack';
 import HtmlBundlerPlugin from '../src';
-import { compareFiles } from './utils/helpers';
+import { compareFiles, compareFilesRuns } from './utils/helpers';
 
 beforeAll(() => {
   // important: the environment constant is used in code
@@ -25,6 +25,22 @@ describe('issue tests', () => {
 
   test('issue infinity walk by circular dependency', () => compareFiles('issue-mui-css'));
 
+  test('filesystem cache restores script on second run, issue #190', async () => {
+    const testPath = path.join(__dirname, 'issues/issue-190-cache-filesystem-js');
+    const cleanup = () => {
+      fs.rmSync(path.join(testPath, 'dist'), { recursive: true, force: true });
+      fs.rmSync(path.join(testPath, '.cache'), { recursive: true, force: true });
+    };
+
+    cleanup();
+
+    try {
+      await compareFilesRuns('../issues/issue-190-cache-filesystem-js', false, 2);
+    } finally {
+      cleanup();
+    }
+  });
+
   test('multi compiler invalid hook handles null filename, issue #191', () => {
     const context = fs.mkdtempSync(path.join(os.tmpdir(), 'html-bundler-webpack-plugin-'));
     const sourcePath = path.join(context, 'src');
diff --git a/test/issues/issue-190-cache-filesystem-js/expected/index.html b/test/issues/issue-190-cache-filesystem-js/expected/index.html
@@ -0,0 +1,10 @@
+<!doctype html>
+<html>
+  <head>
+    <title>Test</title>
+  </head>
+  <body>
+    <h1>Hello World!</h1>
+    <script src="main.6ad9992e.js"></script>
+  </body>
+</html>
diff --git a/test/issues/issue-190-cache-filesystem-js/expected/main.6ad9992e.js b/test/issues/issue-190-cache-filesystem-js/expected/main.6ad9992e.js
@@ -0,0 +1 @@
+console.log("Hello World! Test: 123");
diff --git a/test/issues/issue-190-cache-filesystem-js/src/index.html b/test/issues/issue-190-cache-filesystem-js/src/index.html
@@ -0,0 +1,10 @@
+<!doctype html>
+<html>
+  <head>
+    <title>Test</title>
+  </head>
+  <body>
+    <h1>Hello World!</h1>
+    <script src="main.js"></script>
+  </body>
+</html>
diff --git a/test/issues/issue-190-cache-filesystem-js/src/main.js b/test/issues/issue-190-cache-filesystem-js/src/main.js
@@ -0,0 +1 @@
+console.log('Hello World! Test: 123');
diff --git a/test/issues/issue-190-cache-filesystem-js/webpack.config.js b/test/issues/issue-190-cache-filesystem-js/webpack.config.js
@@ -0,0 +1,31 @@
+const path = require('path');
+const HtmlBundlerPlugin = require('html-bundler-webpack-plugin');
+
+module.exports = {
+  mode: 'production',
+
+  output: {
+    path: path.join(__dirname, 'dist/'),
+    clean: false,
+  },
+
+  plugins: [
+    new HtmlBundlerPlugin({
+      entry: {
+        index: {
+          import: './src/index.html',
+          filename: () => '[name].html',
+        },
+      },
+
+      js: {
+        filename: () => '[name].[contenthash:8].js',
+      },
+    }),
+  ],
+
+  cache: {
+    type: 'filesystem',
+    cacheDirectory: path.join(__dirname, '.cache'),
+  },
+};
diff --git a/test/utils/helpers.js b/test/utils/helpers.js
@@ -133,9 +133,11 @@ export const compareFilesRuns = (relTestCasePath, compareContent = true, num = 1
   const results = [];
   const expected = Array(num).fill(true);
   const filter = /.(html|css|css.map|js|js.map|json)$/;
+  let promise = Promise.resolve();
 
   for (let i = 0; i < num; i++) {
-    const res = compile(PATHS, relTestCasePath, {})
+    promise = promise
+      .then(() => compile(PATHS, relTestCasePath, {}))
       .then(() => {
         const { received: receivedFiles, expected: expectedFiles } = getCompareFileList(webRootPath, expectedPath);
         expect(receivedFiles).toEqual(expectedFiles);
@@ -146,15 +148,14 @@ export const compareFilesRuns = (relTestCasePath, compareContent = true, num = 1
           });
         }
 
-        return Promise.resolve(true);
+        results.push(true);
       })
       .catch((error) => {
-        return Promise.reject(new Error(error.stack));
+        return Promise.reject(new Error(error.stack ? error.stack : error));
       });
-    results.push(res);
   }
 
-  return expect(Promise.all(results)).resolves.toEqual(expected);
+  return expect(promise.then(() => results)).resolves.toEqual(expected);
 };
 
 /**
