test: fix flaky test-policy-integrity

Split the test into seven tests so that it doesn't time out.
Fixes: #40694Fixes: #38088
PR-URL: #40763
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Bradley Farias <bradley.meck@gmail.com>

Author: Trott

test/pummel/pummel.status

@@ -7,8 +7,6 @@ prefix pummel
 [true] # This section applies to all platforms
 
 [$system==win32]
-# https://github.com/nodejs/node/issues/40694
-test-policy-integrity: PASS,FLAKY
 
 [$system==linux]
 # https://github.com/nodejs/node/issues/38226

test/pummel/test-policy-integrity.js test/pummel/test-policy-integrity-dep.jstest/pummel/test-policy-integrity.js renamed to test/pummel/test-policy-integrity-dep.js

@@ -60,24 +60,6 @@ const parentBody = {
  import(process.env.DEP_FILE)
  `,
 };
-constworkerSpawningBody=`
- const path = require('path');
- const { Worker } = require('worker_threads');
- if (!process.env.PARENT_FILE) {
- console.error(
- 'missing required PARENT_FILE env to determine worker entry point'
- );
- process.exit(33);
- }
- if (!process.env.DELETABLE_POLICY_FILE) {
- console.error(
- 'missing required DELETABLE_POLICY_FILE env to check reloading'
- );
- process.exit(33);
- }
- const w = new Worker(path.resolve(process.env.PARENT_FILE));
- w.on('exit', (status) => process.exit(status === 0 ? 0 : 1));
-`;
 
 letnextTestId=1;
 functionnewTestId(){
@@ -100,12 +82,11 @@ function drainQueue() {
 if(toSpawn.length){
 constconfig=toSpawn.shift();
 const{
- shouldSucceed,// = (() => { throw new Error('required')})(),
- preloads,// = (() =>{ throw new Error('required')})(),
- entryPath,// = (() => { throw new Error('required')})(),
- willDeletePolicy,// = (() => { throw new Error('required')})(),
- onError,// = (() => { throw new Error('required')})(),
- resources,// = (() => { throw new Error('required')})(),
+ shouldSucceed,
+ preloads,
+ entryPath,
+ onError,
+ resources,
  parentPath,
  depPath,
 }=config;
@@ -118,7 +99,7 @@ function drainQueue() {
 tmpdir.path,
 `deletable-policy-${testId}.json`
 );
-constcliPolicy=willDeletePolicy ? tmpPolicyPath : policyPath;
+
 fs.rmSync(configDirPath,{maxRetries: 3,recursive: true,force: true});
 fs.mkdirSync(configDirPath,{recursive: true});
 constmanifest={
@@ -140,15 +121,15 @@ function drainQueue() {
 }
 constmanifestBody=JSON.stringify(manifest);
 fs.writeFileSync(manifestPath,manifestBody);
-if(cliPolicy===tmpPolicyPath){
+if(policyPath===tmpPolicyPath){
 fs.writeFileSync(tmpPolicyPath,manifestBody);
 }
 constspawnArgs=[
 process.execPath,
 [
 '--unhandled-rejections=strict',
 '--experimental-policy',
-cliPolicy,
+policyPath,
  ...preloads.flatMap((m)=>['-r',m]),
 entryPath,
 '--',
@@ -255,7 +236,6 @@ function fileExtensionFormat(extension, packageType) {
 thrownewError('unknown format '+extension);
 }
 for(constpermutationofpermutations({
-entry: ['worker','parent','dep'],
 preloads: [[],['parent'],['dep']],
 onError: ['log','exit'],
 parentExtension: ['.js','.mjs','.cjs'],
@@ -282,14 +262,9 @@ for (const permutation of permutations({
 continue;
 }
 constdepPath=`./dep${permutation.depExtension}`;
-constworkerSpawnerPath='./worker-spawner.cjs';
-constentryPath={
-dep: depPath,
-parent: parentPath,
-worker: workerSpawnerPath,
-}[permutation.entry];
+
 constpackageJSON={
-main: entryPath,
+main: depPath,
 type: permutation.packageType,
 };
 if(permutation.packageType==='no-field'){
@@ -314,8 +289,7 @@ for (const permutation of permutations({
 if(parentFormat!=='commonjs'){
 permutation.preloads=permutation.preloads.filter((_)=>_!=='parent');
 }
-consthasParent=
-permutation.entry!=='dep'||permutation.preloads.includes('parent');
+consthasParent=permutation.preloads.includes('parent');
 if(hasParent){
 resources[parentPath]={
 body: parentBody[parentFormat],
@@ -332,12 +306,7 @@ for (const permutation of permutations({
 thrownewError('unreachable');
 }
 }
-if(permutation.entry==='worker'){
-resources[workerSpawnerPath]={
-body: workerSpawningBody,
-integrities: hash('sha256',workerSpawningBody),
-};
-}
+
 if(permutation.packageType!=='no-package-json'){
 letpackageBody=JSON.stringify(packageJSON,null,2);
 letpackageIntegrities=hash('sha256',packageBody);
@@ -364,18 +333,15 @@ for (const permutation of permutations({
 integrities: packageIntegrities,
 };
 }
-constwillDeletePolicy=permutation.entry==='worker';
+
 if(permutation.onError==='log'){
 shouldSucceed=true;
 }
 tests.add(
 JSON.stringify({
-// hasParent,
-// original: permutation,
 onError: permutation.onError,
  shouldSucceed,
- entryPath,
- willDeletePolicy,
+entryPath: depPath,
 preloads: permutation.preloads
 .map((_)=>{
 return{