Merge pull request #1 from Manas-Dikshit/fix/workererror-integration

test: use existing webpack test infra for worker-error integration test

Author: Manas-Dikshit

src/WorkerError.js

@@ -11,15 +11,15 @@ const buildStackTrace = (err, workerStack = '', workerId = 'unknown') => {
   // Compute the difference between the worker's stack and the error's stack
   const extraWorkerLines = workerStackLines.slice(
     0,
-    Math.max(0, workerStackLines.length - originStackLines.length)
+    Math.max(0, workerStackLines.length - originStackLines.length),
   );
 
   // Build a readable, merged stack trace
   const mergedStack = [
     `Thread Loader (Worker ${workerId})`,
     err?.message || 'Unknown error',
     extraWorkerLines.join('\n'),
-    ...originStackLines
+    ...originStackLines,
   ]
     .filter(Boolean)
     .join('\n');

test/integration/worker-error.test.js

@@ -5,39 +5,9 @@ import fixtureConfig from '../fixtures/error-project/webpack.config';
 test('WorkerError integration: wraps and formats worker error', (done) => {
   const config = fixtureConfig({ threads: 1 });
 
-  const compiler = webpack(config);
-
-  compiler.run((err, stats) => {
-    const finish = (cbErr) => {
-      compiler.close(() => {
-        try {
-          // Ensure any worker pools created by thread-loader are terminated so
-          // Jest can exit cleanly. Match the options used in the fixture.
-          // Require the runtime workerPools from dist and terminate the pool.
-          // eslint-disable-next-line global-require
-          const { getPool } = require('../../dist/workerPools');
-          const pool = getPool({ workers: 1, workerParallelJobs: 2, poolTimeout: 2000 });
-          if (pool && typeof pool.terminate === 'function') {
-            pool.terminate();
-          }
-        } catch (e) {
-          // ignore cleanup errors
-        }
-
-        // Allow a short grace period for child-process pipes and timeouts
-        // to unwind before finishing the test so Jest can exit cleanly.
-        setTimeout(() => {
-          if (cbErr) {
-            done(cbErr);
-          } else {
-            done();
-          }
-        }, 50);
-      });
-    };
-
+  webpack(config, (err, stats) => {
     if (err) {
-      finish(err);
+      done(err);
       return;
     }
 
@@ -59,9 +29,9 @@ test('WorkerError integration: wraps and formats worker error', (done) => {
       // Expect combined stack trace contains at least one 'at' line
       expect(stack).toMatch(/\n\s*at\s+/);
 
-      finish();
+      done();
     } catch (e) {
-      finish(e);
+      done(e);
     }
   });
 }, 30000);