Skip to content

Commit

Permalink
fix: sas9JobExecutor not returning response
Browse files Browse the repository at this point in the history
  • Loading branch information
medjedovicm committed Nov 26, 2021
1 parent 96aac0c commit bdf9e2f
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions src/job-execution/Sas9JobExecutor.ts
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,8 @@ export class Sas9JobExecutor extends BaseJobExecutor {
} catch (e: any) {
return Promise.reject(new ErrorResponse(e?.message, e))
}
} else {
data = ''
}

for (const key in requestParams) {
Expand All @@ -66,6 +68,7 @@ export class Sas9JobExecutor extends BaseJobExecutor {
config.password,
this.jobsPath
)

const contentType =
data && Object.keys(data).length
? 'multipart/form-data; boundary=' + (formData as any)._boundary
Expand All @@ -89,6 +92,8 @@ export class Sas9JobExecutor extends BaseJobExecutor {
}

this.requestClientSingle!.appendRequest(resString, sasJob, config.debug)

return res
})
.catch((err: any) => {
let errString = err
Expand All @@ -98,6 +103,8 @@ export class Sas9JobExecutor extends BaseJobExecutor {
}

this.requestClientSingle!.appendRequest(errString, sasJob, config.debug)

return err
})
}

Expand Down

1 comment on commit bdf9e2f

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverage report

Total coverage

Status Category Percentage Covered / Total
🔴 Statements 58.12% 1603/2758
🔴 Branches 39.55% 547/1383
🔴 Functions 45.63% 230/504
🟡 Lines 67.2% 2559/3808

Status of coverage: 🟢 - ok, 🟡 - slightly more than threshold, 🔴 - under the threshold

Report generated by 🧪jest coverage report action from bdf9e2f

Please sign in to comment.