fix: callback type checking fixes #304

src/SASjs.ts

@@ -550,31 +550,41 @@ export default class SASjs {
       ...this.sasjsConfig,
       ...config
     }
-
+    if (
-    if (config.serverType === ServerType.SasViya && config.contextName) {
+      typeof loginRequiredCallback === 'function' ||
-      if (config.useComputeApi) {
+      typeof loginRequiredCallback === 'undefined'
-        return await this.computeJobExecutor!.execute(
+    ) {
-          sasJob,
+      if (config.serverType === ServerType.SasViya && config.contextName) {
-          data,
+        if (config.useComputeApi) {
-          config,
+          return await this.computeJobExecutor!.execute(
-          loginRequiredCallback,
+            sasJob,
-          accessToken
+            data,
-        )
+            config,
+            loginRequiredCallback,
+            accessToken
+          )
+        } else {
+          return await this.jesJobExecutor!.execute(
+            sasJob,
+            data,
+            config,
+            loginRequiredCallback,
+            accessToken
+          )
+        }
       } else {
-        return await this.jesJobExecutor!.execute(
+        return await this.webJobExecutor!.execute(
           sasJob,
           data,
           config,
-          loginRequiredCallback,
+          loginRequiredCallback
-          accessToken
         )
       }
     } else {
-      return await this.webJobExecutor!.execute(
+      return Promise.reject(
-        sasJob,
+        new ErrorResponse(
-        data,
+          `Invalid loginRequiredCallback parameter was provided. Expected Callback function but found ${typeof loginRequiredCallback}`
-        config,
+        )
-        loginRequiredCallback
       )
     }
   }