test: cleanup error handling expectations

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

diff --git a/tests/pools/cluster/fixed.test.mjs b/tests/pools/cluster/fixed.test.mjs
index 975e6e16a60aaf5a46579aad66acf3c9d070e855..094e506d19d73b1e44ac596eadd910444e933d67 100644 (file)
--- a/tests/pools/cluster/fixed.test.mjs
+++ b/tests/pools/cluster/fixed.test.mjs
@@ -176,9 +176,7 @@ describe('Fixed cluster pool test suite', () => {
     } catch (e) {
       inError = e
     }
-    expect(inError).toBeDefined()
-    expect(typeof inError === 'string').toBe(true)
-    expect(inError).toBe('Error Message from ClusterWorker')
+    expect(inError).toStrictEqual('Error Message from ClusterWorker')
     expect(taskError).toStrictEqual({
       data,
       message: 'Error Message from ClusterWorker',
@@ -207,9 +205,7 @@ describe('Fixed cluster pool test suite', () => {
     } catch (e) {
       inError = e
     }
-    expect(inError).toBeDefined()
-    expect(typeof inError === 'string').toBe(true)
-    expect(inError).toBe('Error Message from ClusterWorker:async')
+    expect(inError).toStrictEqual('Error Message from ClusterWorker:async')
     expect(taskError).toStrictEqual({
       data,
       message: 'Error Message from ClusterWorker:async',

diff --git a/tests/pools/thread/fixed.test.mjs b/tests/pools/thread/fixed.test.mjs
index 7473ab5490150bd52f992fdff3035cd5adc2d907..2ffc6c93a72897e601ee0a66af00ba45a4de9fde 100644 (file)
--- a/tests/pools/thread/fixed.test.mjs
+++ b/tests/pools/thread/fixed.test.mjs
@@ -202,11 +202,8 @@ describe('Fixed thread pool test suite', () => {
     } catch (e) {
       inError = e
     }
-    expect(inError).toBeDefined()
     expect(inError).toBeInstanceOf(Error)
-    expect(inError.message).toBeDefined()
-    expect(typeof inError.message === 'string').toBe(true)
-    expect(inError.message).toBe('Error Message from ThreadWorker')
+    expect(inError.message).toStrictEqual('Error Message from ThreadWorker')
     expect(taskError).toStrictEqual({
       data,
       message: new Error('Error Message from ThreadWorker'),
@@ -235,11 +232,10 @@ describe('Fixed thread pool test suite', () => {
     } catch (e) {
       inError = e
     }
-    expect(inError).toBeDefined()
     expect(inError).toBeInstanceOf(Error)
-    expect(inError.message).toBeDefined()
-    expect(typeof inError.message === 'string').toBe(true)
-    expect(inError.message).toBe('Error Message from ThreadWorker:async')
+    expect(inError.message).toStrictEqual(
+      'Error Message from ThreadWorker:async'
+    )
     expect(taskError).toStrictEqual({
       data,
       message: new Error('Error Message from ThreadWorker:async'),

diff --git a/tests/worker/cluster-worker.test.mjs b/tests/worker/cluster-worker.test.mjs
index fcb6564e83dbf832cb83f9ae9fc432040972afe6..732ba2544721012b2a5b56ee437541fb8b0ca174 100644 (file)
--- a/tests/worker/cluster-worker.test.mjs
+++ b/tests/worker/cluster-worker.test.mjs
@@ -93,7 +93,6 @@ describe('Cluster worker test suite', () => {
   it('Verify that handleError() method is working properly', () => {
     const error = new Error('Error as an error')
     const worker = new ClusterWorker(() => {})
-    expect(worker.handleError(error)).not.toBeInstanceOf(Error)
     expect(worker.handleError(error)).toStrictEqual(error.message)
     const errorMessage = 'Error as a string'
     expect(worker.handleError(errorMessage)).toStrictEqual(errorMessage)

diff --git a/tests/worker/thread-worker.test.mjs b/tests/worker/thread-worker.test.mjs
index ed9f16ac318694c90b8074daef9caddc0af215d0..8f146b7114565dc915346f25a872126202a3facf 100644 (file)
--- a/tests/worker/thread-worker.test.mjs
+++ b/tests/worker/thread-worker.test.mjs
@@ -93,7 +93,6 @@ describe('Thread worker test suite', () => {
   it('Verify that handleError() method is working properly', () => {
     const error = new Error('Error as an error')
     const worker = new ThreadWorker(() => {})
-    expect(worker.handleError(error)).toBeInstanceOf(Error)
     expect(worker.handleError(error)).toStrictEqual(error)
     const errorMessage = 'Error as a string'
     expect(worker.handleError(errorMessage)).toStrictEqual(errorMessage)