Coverage increased (#226)

* Coverage increased

* Few changes

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

diff --git a/tests/pools/thread/fixed.test.js b/tests/pools/thread/fixed.test.js
index cc60a9b5a021253819445d46ff49fab6e91abe2f..b70d4c624c5d9744494c6ac87ed9f290e1633221 100644 (file)
--- a/tests/pools/thread/fixed.test.js
+++ b/tests/pools/thread/fixed.test.js
@@ -11,7 +11,8 @@ const pool = new FixedThreadPool(
 )
 const emptyPool = new FixedThreadPool(
   1,
-  './tests/worker-files/thread/emptyWorker.js'
+  './tests/worker-files/thread/emptyWorker.js',
+  { exitHandler: () => console.log('WORKER EXITED') }
 )
 const echoPool = new FixedThreadPool(
   1,

diff --git a/tests/worker/abstract-worker.test.js b/tests/worker/abstract-worker.test.js
index 0c9b665ee75ae0de6a6a5fd4d910fc2bccff1fd3..1d0c03d9c398c75a5c88b21cb8a7ff1cb0dfc2c1 100644 (file)
--- a/tests/worker/abstract-worker.test.js
+++ b/tests/worker/abstract-worker.test.js
@@ -1,5 +1,12 @@
 const expect = require('expect')
-const { ClusterWorker } = require('../../lib')
+const { ClusterWorker, ThreadWorker } = require('../../lib')
+
+class StubPoolWithIsMainWorker extends ThreadWorker {
+  constructor (fn, opts) {
+    super(fn, opts)
+    this.mainWorker = false
+  }
+}
 
 describe('Abstract worker test suite', () => {
   it('Verify that fn function is mandatory', () => {
@@ -7,4 +14,16 @@ describe('Abstract worker test suite', () => {
       new Error('fn parameter is mandatory')
     )
   })
+
+  it('Verify that handle Error function is working properly', () => {
+    const error = new Error('My error')
+    const worker = new ThreadWorker(() => {})
+    expect(worker.handleError(error)).toBe(error)
+  })
+
+  it('Verify that get main worker throw error if main worker is not set', () => {
+    expect(() =>
+      new StubPoolWithIsMainWorker(() => {}).getMainWorker()
+    ).toThrowError()
+  })
 })

diff --git a/tests/worker/cluster-worker.test.js b/tests/worker/cluster-worker.test.js
index 5dec6076f8e34ec7f0bdf7eec42c4b5202d3d980..196564a48c51aee9e32a2f4af7cd91208d9b92de 100644 (file)
--- a/tests/worker/cluster-worker.test.js
+++ b/tests/worker/cluster-worker.test.js
@@ -6,4 +6,10 @@ describe('Cluster worker test suite', () => {
     const worker = new ClusterWorker(() => {})
     expect(worker.maxInactiveTime).toEqual(60_000)
   })
+
+  it('Verify that handleError function works properly', () => {
+    const errorMessage = 'Error as a string'
+    const worker = new ClusterWorker(() => {})
+    expect(worker.handleError(errorMessage)).toBe(errorMessage)
+  })
 })