Tests: be more strict on number expectation

Signed-off-by: Jérôme Benoit <jerome.benoit@sap.com>

diff --git a/benchmarks/versus-external-pools/dynamic-piscina.js b/benchmarks/versus-external-pools/dynamic-piscina.js
index d78dd568c2c28ae9a9f0fd04d79e4e9c1ecd774e..83190d43cd927e708de2c356bea727958aee507b 100644 (file)
--- a/benchmarks/versus-external-pools/dynamic-piscina.js
+++ b/benchmarks/versus-external-pools/dynamic-piscina.js
@@ -12,7 +12,7 @@ const piscina = new Piscina({
   filename: './workers/piscina/function-to-bench-worker.js',
   minThreads: size,
   maxThreads: size * 3,
-  idleTimeout: 1000 * 60 // this is the same as poolifier default
+  idleTimeout: 60000 // this is the same as poolifier default
 })
 
 async function run () {

diff --git a/benchmarks/versus-external-pools/fixed-piscina.js b/benchmarks/versus-external-pools/fixed-piscina.js
index d1f785edeb31f1ff21870d13b987e8234597a22b..c89a951d2c70be00147764eeba7e78eece53bccd 100644 (file)
--- a/benchmarks/versus-external-pools/fixed-piscina.js
+++ b/benchmarks/versus-external-pools/fixed-piscina.js
@@ -11,7 +11,7 @@ const data = {
 const piscina = new Piscina({
   filename: './workers/piscina/function-to-bench-worker.js',
   minThreads: size,
-  idleTimeout: 1000 * 60 // this is the same as poolifier default
+  idleTimeout: 60000 // this is the same as poolifier default
 })
 
 async function run () {

diff --git a/src/worker/abstract-worker.ts b/src/worker/abstract-worker.ts
index 4aad8e62669335c3940ba3fb88cc08afda062eb3..52143fe3099c36c284cf56c3a8c6746b6c76c1dc 100644 (file)
--- a/src/worker/abstract-worker.ts
+++ b/src/worker/abstract-worker.ts
@@ -6,7 +6,7 @@ import { EMPTY_FUNCTION } from '../utils'
 import type { KillBehavior, WorkerOptions } from './worker-options'
 import { KillBehaviors } from './worker-options'
 
-const DEFAULT_MAX_INACTIVE_TIME = 1000 * 60
+const DEFAULT_MAX_INACTIVE_TIME = 60000
 const DEFAULT_KILL_BEHAVIOR: KillBehavior = KillBehaviors.SOFT
 
 /**

diff --git a/tests/worker/abstract-worker.test.js b/tests/worker/abstract-worker.test.js
index 9c8dd0973e9fe43ab0a9fe50e13b89b2fbe669be..c75227cc0bd8fcb8f0de0fe3f4b1b025e513245a 100644 (file)
--- a/tests/worker/abstract-worker.test.js
+++ b/tests/worker/abstract-worker.test.js
@@ -17,7 +17,7 @@ describe('Abstract worker test suite', () => {
 
   it('Verify worker options default values', () => {
     const worker = new ThreadWorker(() => {})
-    expect(worker.opts.maxInactiveTime).toBe(1000 * 60)
+    expect(worker.opts.maxInactiveTime).toStrictEqual(60000)
     expect(worker.opts.killBehavior).toBe(KillBehaviors.SOFT)
     expect(worker.opts.async).toBe(false)
   })
@@ -28,7 +28,7 @@ describe('Abstract worker test suite', () => {
       async: true,
       killBehavior: KillBehaviors.HARD
     })
-    expect(worker.opts.maxInactiveTime).toBe(6000)
+    expect(worker.opts.maxInactiveTime).toStrictEqual(6000)
     expect(worker.opts.killBehavior).toBe(KillBehaviors.HARD)
     expect(worker.opts.async).toBe(true)
   })