Revert Promise.all usage in tests
authorJérôme Benoit <jerome.benoit@sap.com>
Sun, 9 Oct 2022 23:07:09 +0000 (01:07 +0200)
committerJérôme Benoit <jerome.benoit@sap.com>
Sun, 9 Oct 2022 23:07:09 +0000 (01:07 +0200)
Signed-off-by: Jérôme Benoit <jerome.benoit@sap.com>
tests/pools/cluster/dynamic.test.js
tests/pools/cluster/fixed.test.js
tests/pools/thread/dynamic.test.js
tests/pools/thread/fixed.test.js

index 9e4a6b235134408f83537dec6a60652acdb52a58..02a022870d4911eced33b2dd529ae695c7280210 100644 (file)
@@ -26,7 +26,7 @@ describe('Dynamic cluster pool test suite', () => {
     for (let i = 0; i < max * 2; i++) {
       promises.push(pool.execute({ test: 'test' }))
     }
-    await Promise.all(promises)
+    // await Promise.all(promises)
     expect(pool.workers.length).toBeLessThanOrEqual(max)
     expect(pool.workers.length).toBeGreaterThan(min)
     // The `busy` event is triggered when the number of submitted tasks at once reach the max number of workers in the dynamic pool.
index 52ba58d7114cd78f89b4d46f8f5b7b8c992d5501..2ef709c1d0fc71fa3d6da9af508d8afa3859d5cf 100644 (file)
@@ -74,7 +74,6 @@ describe('Fixed cluster pool test suite', () => {
     for (let i = 0; i < numberOfWorkers * 2; i++) {
       promises.push(pool.execute({ test: 'test' }))
     }
-    await Promise.all(promises)
     // The `busy` event is triggered when the number of submitted tasks at once reach the number of fixed pool workers.
     // So in total numberOfWorkers + 1 times for a loop submitting up to numberOfWorkers * 2 tasks to the fixed pool.
     expect(poolBusy).toBe(numberOfWorkers + 1)
index f345a0ee665a78543e085bad4dd32248c70bd7d7..8558cb6f16468f773df94edb3ff0fadb4b0877e3 100644 (file)
@@ -26,7 +26,7 @@ describe('Dynamic thread pool test suite', () => {
     for (let i = 0; i < max * 2; i++) {
       promises.push(pool.execute({ test: 'test' }))
     }
-    await Promise.all(promises)
+    // await Promise.all(promises)
     expect(pool.workers.length).toBeLessThanOrEqual(max)
     expect(pool.workers.length).toBeGreaterThan(min)
     // The `busy` event is triggered when the number of submitted tasks at once reach the max number of workers in the dynamic pool.
index f89977b3a5be536f71972c2db66e2837eac4f3b4..9af4417c2fa5b61fcd130a00319adca7c3fe05ff 100644 (file)
@@ -74,7 +74,7 @@ describe('Fixed thread pool test suite', () => {
     for (let i = 0; i < numberOfThreads * 2; i++) {
       promises.push(pool.execute({ test: 'test' }))
     }
-    await Promise.all(promises)
+    // await Promise.all(promises)
     // The `busy` event is triggered when the number of submitted tasks at once reach the number of fixed pool workers.
     // So in total numberOfThreads + 1 times for a loop submitting up to numberOfThreads * 2 tasks to the fixed pool.
     expect(poolBusy).toBe(numberOfThreads + 1)