test: adjust elapsed max time at pool destroy

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

diff --git a/SECURITY.md b/SECURITY.md
index d42dfb7aa7c9f9d4c99575e0b934fff6e0e6200f..31e1bbe2a39aae7ef434c56b8e114ebac4eb13c2 100644 (file)
--- a/SECURITY.md
+++ b/SECURITY.md
@@ -6,7 +6,8 @@ Security matrix, currently there are no security vulnerabilities.
 
 | Version | Supported          |
 | ------- | ------------------ |
-| 4.x.x   | :white_check_mark: |
+| 5.x.x   | :white_check_mark: |
+| 4.x.x   | :x:                |
 | 3.x.x   | :x:                |
 | 2.x.x   | :x:                |
 | 1.x.x   | :x:                |

diff --git a/src/pools/abstract-pool.ts b/src/pools/abstract-pool.ts
index 298bc2a288b5a493cf7459c3b662c4e3fc58d604..f69921b5c451cc3ec7755733664eca459fe22c69 100644 (file)
--- a/src/pools/abstract-pool.ts
+++ b/src/pools/abstract-pool.ts
@@ -2180,7 +2180,7 @@ export abstract class AbstractPool<
 
   private async sendKillMessageToWorker (
     workerNodeKey: number,
-    timeout = 2000
+    timeout = 1000
   ): Promise<void> {
     let timeoutHandle: NodeJS.Timeout | undefined
     let killMessageListener:

diff --git a/tests/pools/abstract-pool.test.mjs b/tests/pools/abstract-pool.test.mjs
index 452d5b55d52d9991aa89c6aab84e53a2862f0e11..1e40178c090128680d1ef5ddbe2e03b9d6256652 100644 (file)
--- a/tests/pools/abstract-pool.test.mjs
+++ b/tests/pools/abstract-pool.test.mjs
@@ -1447,7 +1447,10 @@ describe('Abstract pool test suite', () => {
     const elapsedTime = performance.now() - startTime
     expect(tasksFinished).toBeLessThanOrEqual(numberOfWorkers * maxMultiplier)
     expect(elapsedTime).toBeGreaterThanOrEqual(2000)
-    expect(elapsedTime).toBeLessThanOrEqual(tasksFinishedTimeout + 800)
+    // Worker kill message response timeout is 1000ms
+    expect(elapsedTime).toBeLessThanOrEqual(
+      tasksFinishedTimeout + 1000 * tasksFinished + 100
+    )
   })
 
   it('Verify that destroy() waits until the tasks finished timeout is reached', async () => {