fix: improve job timeout notification by giving the job name and id in the error message (#387)

airhorns · web-flow · commit ca886b1f8540 · 2021-02-02T09:10:24.000+01:00
diff --git a/src/classes/job.ts b/src/classes/job.ts
@@ -363,7 +363,7 @@ export class Job<T = any, R = any, N extends string = string> {
     return new Promise<any>(async (resolve, reject) => {
       let timeout: NodeJS.Timeout;
       if (ttl) {
-        timeout = setTimeout(() => onFailed('timedout'), ttl);
+        timeout = setTimeout(() => onFailed(`Job wait ${this.name} timed out before finishing, no finish notification arrived after ${ttl}ms (id=${jobId})`), ttl);
       }
 
       function onCompleted(args: any) {

Original file line number	Diff line number	Diff line change
`@@ -363,7 +363,7 @@ export class Job<T = any, R = any, N extends string = string> {`
`363`	`363`	`return new Promise<any>(async (resolve, reject) => {`
`364`	`364`	`let timeout: NodeJS.Timeout;`
`365`	`365`	`if (ttl) {`
`366`		`- timeout = setTimeout(() => onFailed('timedout'), ttl);`
	`366`	+ timeout = setTimeout(() => onFailed(`Job wait ${this.name} timed out before finishing, no finish notification arrived after ${ttl}ms (id=${jobId})`), ttl);
`367`	`367`	`}`
`368`	`368`
`369`	`369`	`function onCompleted(args: any) {`