refactor: refine worker factory arguments checking
authorJérôme Benoit <jerome.benoit@sap.com>
Sat, 16 Dec 2023 19:25:43 +0000 (20:25 +0100)
committerJérôme Benoit <jerome.benoit@sap.com>
Sat, 16 Dec 2023 19:25:43 +0000 (20:25 +0100)
Signed-off-by: Jérôme Benoit <jerome.benoit@sap.com>
src/worker/WorkerAbstract.ts

index 5e5e6fdbea05e01415415a852599e3a8ccb8ea27..cd4dedf992fca40ff0b7cb787e91468d28020d55 100644 (file)
@@ -22,10 +22,13 @@ export abstract class WorkerAbstract<T extends WorkerData> {
    */
   constructor(workerScript: string, workerOptions: WorkerOptions) {
     if (workerScript == null) {
-      throw new Error('Worker script is not defined');
+      throw new TypeError('Worker script is not defined');
     }
-    if (typeof workerScript === 'string' && workerScript.trim().length === 0) {
-      throw new Error('Worker script is empty');
+    if (typeof workerScript !== 'string') {
+      throw new TypeError('Worker script must be a string');
+    }
+    if (workerScript.trim().length === 0) {
+      throw new Error('Worker script is an empty string');
     }
     if (!existsSync(workerScript)) {
       throw new Error('Worker script file does not exist');