build: cleanup eslint configuration

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

diff --git a/.eslintrc.json b/.eslintrc.json
index ddb47174049aff56a7589146429e75c4e1660902..408af99eb97b27a0ed461f02b49e44277c176e27 100644 (file)
--- a/.eslintrc.json
+++ b/.eslintrc.json
@@ -23,9 +23,7 @@
     "space-before-blocks": ["error", "always"],
     "curly": ["error", "all"],
     "brace-style": "error",
-    "eqeqeq": ["error", "always"],
     "no-else-return": "error",
-    "no-eq-null": "error",
     "no-extra-bind": "error",
     "no-lone-blocks": "error",
     "no-multi-spaces": "error",
@@ -41,14 +39,6 @@
       }
     ],
     "block-spacing": "error",
-    // "capitalized-comments": [
-    //   "error",
-    //   "always",
-    //   {
-    //     "ignoreConsecutiveComments": true,
-    //     "ignorePattern": "pragma|ignored"
-    //   }
-    // ],
     "eol-last": ["error", "always"],
     "consistent-this": ["error", "self"],
     "func-call-spacing": ["error", "never"],

diff --git a/src/utils/Utils.ts b/src/utils/Utils.ts
index d3bee6c6e6c31f09c55f8bb1c90597ae830054b8..d99dcc3a0da29e98f11882cb0b4d3726bbff4133 100644 (file)
--- a/src/utils/Utils.ts
+++ b/src/utils/Utils.ts
@@ -75,7 +75,9 @@ export const isValidTime = (date: unknown): boolean => {
   return false;
 };
 
-export const convertToDate = (value: Date | string | number | undefined): Date | undefined => {
+export const convertToDate = (
+  value: Date | string | number | null | undefined,
+): Date | null | undefined => {
   if (isNullOrUndefined(value)) {
     return value as undefined;
   }
@@ -127,7 +129,7 @@ export const convertToFloat = (value: unknown): number => {
 
 export const convertToBoolean = (value: unknown): boolean => {
   let result = false;
-  if (value) {
+  if (value != null) {
     // Check the type
     if (typeof value === 'boolean') {
       return value;
@@ -290,7 +292,6 @@ export const isUndefined = (value: unknown): boolean => {
 };
 
 export const isNullOrUndefined = (value: unknown): boolean => {
-  // eslint-disable-next-line eqeqeq, no-eq-null
   return value == null;
 };
 

diff --git a/src/worker/WorkerAbstract.ts b/src/worker/WorkerAbstract.ts
index 912f5256575bb2ca9a5b486bb8ed0cce8cd5bd9e..5e5e6fdbea05e01415415a852599e3a8ccb8ea27 100644 (file)
--- a/src/worker/WorkerAbstract.ts
+++ b/src/worker/WorkerAbstract.ts
@@ -21,7 +21,7 @@ export abstract class WorkerAbstract<T extends WorkerData> {
    * @param workerOptions -
    */
   constructor(workerScript: string, workerOptions: WorkerOptions) {
-    if (workerScript === null || workerScript === undefined) {
+    if (workerScript == null) {
       throw new Error('Worker script is not defined');
     }
     if (typeof workerScript === 'string' && workerScript.trim().length === 0) {

diff --git a/src/worker/WorkerSet.ts b/src/worker/WorkerSet.ts
index 62ddf8079f2f6377cd0f95fe62daac629a27a9bb..17b5b6dde201c85f4c0af0cbaf3574d33968fb03 100644 (file)
--- a/src/worker/WorkerSet.ts
+++ b/src/worker/WorkerSet.ts
@@ -105,7 +105,7 @@ export class WorkerSet extends WorkerAbstract<WorkerData> {
     if (!this.started) {
       throw new Error('Cannot add a WorkerSet element: not started');
     }
-    if (!this.workerSet) {
+    if (this.workerSet == null) {
       throw new Error("Cannot add a WorkerSet element: 'workerSet' property does not exist");
     }
     const workerSetElement = await this.getWorkerSetElement();

diff --git a/tests/utils/Utils.test.ts b/tests/utils/Utils.test.ts
index 82964803b87e0f4913a9929dc8125dc7e04d4420..615f76fa01f057c9bad1a1b60a7a0d16b4df54d4 100644 (file)
--- a/tests/utils/Utils.test.ts
+++ b/tests/utils/Utils.test.ts
@@ -91,6 +91,7 @@ await describe('Utils test suite', async () => {
 
   await it('Verify convertToDate()', () => {
     expect(convertToDate(undefined)).toBe(undefined);
+    expect(convertToDate(null)).toBe(null);
     expect(() => convertToDate('')).toThrow(new Error("Cannot convert to date: ''"));
     expect(() => convertToDate('00:70:61')).toThrow(
       new Error("Cannot convert to date: '00:70:61'"),