refactor: check for null and undefined

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

diff --git a/prepare.js b/prepare.js
index 12846ca298a878bef2e719bf46fa43914f3b6f18..eef41216657fd21ba1a650cf36e38e2713f42579 100644 (file)
--- a/prepare.js
+++ b/prepare.js
@@ -1,4 +1,4 @@
-const isCIEnvironment = process.env.CI !== undefined
+const isCIEnvironment = process.env.CI != null
 if (isCIEnvironment === false) {
   require('husky').install()
 }

diff --git a/src/pools/abstract-pool.ts b/src/pools/abstract-pool.ts
index afd7914eceaef3e531e87f4fe7fdcd92b88dd27d..ad21effb3bbb9837977a54c2f8c159c85af6b3c8 100644 (file)
--- a/src/pools/abstract-pool.ts
+++ b/src/pools/abstract-pool.ts
@@ -380,9 +380,9 @@ export abstract class AbstractPool<
    */
   protected workerListener (): (message: MessageValue<Response>) => void {
     return message => {
-      if (message.id !== undefined) {
+      if (message.id != null) {
         const promiseResponse = this.promiseResponseMap.get(message.id)
-        if (promiseResponse !== undefined) {
+        if (promiseResponse != null) {
           if (message.error != null) {
             promiseResponse.reject(message.error)
           } else {

diff --git a/src/worker/abstract-worker.ts b/src/worker/abstract-worker.ts
index 6a7a43259f11392fc764bf74e4cada6e20bc6518..5165417924b66cafbb9dceb6e2cab53c24ee7965 100644 (file)
--- a/src/worker/abstract-worker.ts
+++ b/src/worker/abstract-worker.ts
@@ -81,18 +81,18 @@ export abstract class AbstractWorker<
     value: MessageValue<Data, MainWorker>,
     fn: (data: Data) => Response
   ): void {
-    if (value.data !== undefined && value.id !== undefined) {
+    if (value.data != null && value.id != null) {
       // Here you will receive messages
       if (this.opts.async === true) {
         this.runInAsyncScope(this.runAsync.bind(this), this, fn, value)
       } else {
         this.runInAsyncScope(this.run.bind(this), this, fn, value)
       }
-    } else if (value.parent !== undefined) {
+    } else if (value.parent != null) {
       // Save a reference of the main worker to communicate with it
       // This will be received once
       this.mainWorker = value.parent
-    } else if (value.kill !== undefined) {
+    } else if (value.kill != null) {
       // Here is time to kill this worker, just clearing the interval
       this.aliveInterval != null && clearInterval(this.aliveInterval)
       this.emitDestroy()