Merge pull request #785 from sasjs/issue-783

Sasjs executor not re-sending waiting requests

.github/issue_template.md

@@ -0,0 +1,12 @@
+## Expected behaviour
+*Describe what should be happening*
+
+## Current behaviour
+*Describe what is actually happening* 
+
+## Environment info
+**Client tech stack**: *Angular, React, Vue, VanillaJS, NodeJS etc.*
+**Server type**: SASJS|SASVIYA|SAS9
+**Login mechanism**: Default|Redirected
+**Debug**: true|false
+**Use Compute Api (relevant only on VIYA)**: true|false

sasjs-tests/src/testSuites/Basic.ts

@@ -53,9 +53,7 @@ export const basicTests = (
         return await newAdapterIns.checkSession()
       },
       assertion: (response: any) =>
-        adapter.getSasjsConfig().serverType === ServerType.Sas9
-          ? response?.isLoggedIn
-          : response?.isLoggedIn && response?.userName === userName
+        response?.isLoggedIn && response?.userName === userName
     },
     {
       title: 'Multiple Log in attempts',

src/SASjs.ts

@@ -895,6 +895,7 @@ export default class SASjs {
     await this.computeJobExecutor?.resendWaitingRequests()
     await this.jesJobExecutor?.resendWaitingRequests()
     await this.fileUploader?.resendWaitingRequests()
+    await this.sasjsJobExecutor?.resendWaitingRequests()
   }
 
   /**

src/auth/AuthManager.ts

@@ -141,7 +141,7 @@ export class AuthManager {
         await this.performCASSecurityCheck()
       }
 
-      await this.loginCallback()
+      this.loginCallback()
       this.userName = loginParams.username
     }
 

src/job-execution/WebJobExecutor.ts

@@ -16,8 +16,7 @@ import { SASViyaApiClient } from '../SASViyaApiClient'
 import {
   isRelativePath,
   parseSasViyaDebugResponse,
-  appendExtraResponseAttributes,
-  getValidJson
+  appendExtraResponseAttributes
 } from '../utils'
 import { BaseJobExecutor } from './JobExecutor'
 import { parseWeboutResponse } from '../utils/parseWeboutResponse'
@@ -184,10 +183,6 @@ export class WebJobExecutor extends BaseJobExecutor {
             }
           }
 
-          if (typeof jsonResponse === 'string') {
-            jsonResponse = getValidJson(jsonResponse)
-          }
-
           const responseObject = appendExtraResponseAttributes(
             { result: jsonResponse, log: res.log },
             extraResponseAttributes