fix: revert now unneeded workaround

[poolifier.git] / tests / worker / abstract-worker.test.js

diff --git a/tests/worker/abstract-worker.test.js b/tests/worker/abstract-worker.test.js
index b9ace12c41cbd32d17234faedcd3cba5b22d1b06..e5bf4c81005f94a33cee53bc2f04a10b48549847 100644 (file)
--- a/tests/worker/abstract-worker.test.js
+++ b/tests/worker/abstract-worker.test.js
@@ -1,25 +1,19 @@
 const { expect } = require('expect')
 const { ClusterWorker, KillBehaviors, 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', () => {
-    expect(() => new ClusterWorker()).toThrowError(
-      new Error('fn parameter is mandatory')
-    )
-  })
+  class StubWorkerWithMainWorker extends ThreadWorker {
+    constructor (fn, opts) {
+      super(fn, opts)
+      this.mainWorker = undefined
+    }
+  }
 
   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)
+    expect(worker.opts.async).toBe(undefined)
   })
 
   it('Verify that worker options are set at worker creation', () => {
@@ -28,20 +22,98 @@ 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)
+    expect(worker.opts.async).toBe(undefined)
+  })
+
+  it('Verify that taskFunctions parameter is mandatory', () => {
+    expect(() => new ClusterWorker()).toThrowError(
+      'taskFunctions parameter is mandatory'
+    )
+  })
+
+  it('Verify that taskFunctions parameter is a function or a plain object', () => {
+    expect(() => new ClusterWorker(0)).toThrowError(
+      new TypeError(
+        'taskFunctions parameter is not a function or a plain object'
+      )
+    )
+    expect(() => new ClusterWorker('')).toThrowError(
+      new TypeError(
+        'taskFunctions parameter is not a function or a plain object'
+      )
+    )
+    expect(() => new ClusterWorker(true)).toThrowError(
+      new TypeError(
+        'taskFunctions parameter is not a function or a plain object'
+      )
+    )
+    expect(() => new ClusterWorker([])).toThrowError(
+      new TypeError(
+        'taskFunctions parameter is not a function or a plain object'
+      )
+    )
+    expect(() => new ClusterWorker(new Map())).toThrowError(
+      new TypeError(
+        'taskFunctions parameter is not a function or a plain object'
+      )
+    )
+    expect(() => new ClusterWorker(new Set())).toThrowError(
+      new TypeError(
+        'taskFunctions parameter is not a function or a plain object'
+      )
+    )
+    expect(() => new ClusterWorker(new WeakMap())).toThrowError(
+      new TypeError(
+        'taskFunctions parameter is not a function or a plain object'
+      )
+    )
+    expect(() => new ClusterWorker(new WeakSet())).toThrowError(
+      new TypeError(
+        'taskFunctions parameter is not a function or a plain object'
+      )
+    )
+  })
+
+  it('Verify that taskFunctions parameter is not an empty object', () => {
+    expect(() => new ClusterWorker({})).toThrowError(
+      new Error('taskFunctions parameter object is empty')
+    )
+  })
+
+  it('Verify that taskFunctions parameter with multiple task functions contains function', () => {
+    const fn1 = () => {
+      return 1
+    }
+    const fn2 = ''
+    expect(() => new ThreadWorker({ fn1, fn2 })).toThrowError(
+      new TypeError('A taskFunctions parameter object value is not a function')
+    )
+  })
+
+  it('Verify that taskFunctions parameter with multiple task functions is taken', () => {
+    const fn1 = () => {
+      return 1
+    }
+    const fn2 = () => {
+      return 2
+    }
+    const worker = new ClusterWorker({ fn1, fn2 })
+    expect(typeof worker.taskFunctions.get('default') === 'function').toBe(true)
+    expect(typeof worker.taskFunctions.get('fn1') === 'function').toBe(true)
+    expect(typeof worker.taskFunctions.get('fn2') === 'function').toBe(true)
   })
 
-  it('Verify that handleError function is working properly', () => {
+  it('Verify that handleError() method is working properly', () => {
     const error = new Error('My error')
     const worker = new ThreadWorker(() => {})
-    expect(worker.handleError(error)).toBe(error)
+    expect(worker.handleError(error)).toStrictEqual(error)
   })
 
-  it('Verify that get main worker throw error if main worker is not set', () => {
+  it('Verify that getMainWorker() throw error if main worker is not set', () => {
     expect(() =>
-      new StubPoolWithIsMainWorker(() => {}).getMainWorker()
-    ).toThrowError(new Error('Main worker was not set'))
+      new StubWorkerWithMainWorker(() => {}).getMainWorker()
+    ).toThrowError('Main worker not set')
   })
 })