test: revise test-policy-integrity

* eliminate unneeded Set deletion/cleanup * use number of CPUs as limit for processes spawned rather than hard-coding the limit PR-URL: #35101 Reviewed-By: Bradley Farias <[email protected]> Reviewed-By: Andrey Pechkurov <[email protected]>
nodejs · Sep 22, 2020 · 43c0174 · 43c0174
1 parent 3954b8f
commit 43c0174
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/test/pummel/test-policy-integrity.js b/test/pummel/test-policy-integrity.js
@@ -13,6 +13,8 @@ const fs = require('fs');
 const path = require('path');
 const { pathToFileURL } = require('url');
 
+const cpus = require('os').cpus().length;
+
 function hash(algo, body) {
   const values = [];
   {
@@ -82,7 +84,7 @@ function queueSpawn(opts) {
 }
 
 function drainQueue() {
-  if (spawned > 50) {
+  if (spawned > cpus) {
     return;
   }
   if (toSpawn.length) {
@@ -383,6 +385,5 @@ debug(`spawning ${tests.size} policy integrity permutations`);
 
 for (const config of tests) {
   const parsed = JSON.parse(config);
-  tests.delete(config);
   queueSpawn(parsed);
 }