fix: isRunning for non job processor calls

Author: simllll

src/Job.ts

@@ -29,23 +29,26 @@ export class Job<DATA = unknown | void> {
 		args: Partial<IJobParameters<void>> & {
 			name: string;
 			type: 'normal' | 'single';
-		}
+		},
+		byJobProcessor?
 	);
 	constructor(
 		agenda: Agenda,
 		args: Partial<IJobParameters<DATA>> & {
 			name: string;
 			type: 'normal' | 'single';
 			data: DATA;
-		}
+		},
+		byJobProcessor?
 	);
 	constructor(
 		readonly agenda: Agenda,
 		args: Partial<IJobParameters<DATA>> & {
 			name: string;
 			type: 'normal' | 'single';
 			data: DATA;
-		}
+		},
+		private readonly byJobProcessor = false
 	) {
 		// Set attrs to args
 		this.attrs = {
@@ -161,8 +164,7 @@ export class Job<DATA = unknown | void> {
 	}
 
 	async isRunning(): Promise<boolean> {
-		const definition = this.agenda.definitions[this.attrs.name];
-		if (!definition || !this.agenda.isActiveJobProcessor()) {
+		if (!this.byJobProcessor) {
 			// we have no job definition, therfore we are not the job processor, but a client call
 			// so we get the real state from database
 			await this.fetchStatus();
@@ -197,13 +199,14 @@ export class Job<DATA = unknown | void> {
 	}
 
 	async isDead(): Promise<boolean> {
-		const definition = this.agenda.definitions[this.attrs.name];
-		if (!definition || !this.agenda.isActiveJobProcessor()) {
+		if (!this.byJobProcessor) {
 			// we have no job definition, therfore we are not the job processor, but a client call
 			// so we get the real state from database
 			await this.fetchStatus();
 		}
 
+		const definition = this.agenda.definitions[this.attrs.name];
+
 		const lockDeadline = new Date(Date.now() - definition.lockLifetime);
 
 		// This means a job has "expired", as in it has not been "touched" within the lockoutTime

src/JobProcessor.ts

@@ -239,7 +239,7 @@ export class JobProcessor {
 						);
 					}
 
-					const jobToEnqueue = new Job(this.agenda, resp) as JobWithId;
+					const jobToEnqueue = new Job(this.agenda, resp, true) as JobWithId;
 
 					// Before en-queing job make sure we haven't exceed our lock limits
 					if (!this.shouldLock(jobToEnqueue.attrs.name)) {
@@ -291,7 +291,7 @@ export class JobProcessor {
 				'found a job available to lock, creating a new job on Agenda with id [%s]',
 				result._id
 			);
-			return new Job(this.agenda, result) as JobWithId;
+			return new Job(this.agenda, result, true) as JobWithId;
 		}
 
 		return undefined;

test/job.test.ts

@@ -1479,4 +1479,17 @@ describe('Job', () => {
 			});
 		});
 	});
+
+	it('checks database for running job on "client"', async () => {
+		agenda.define('test', async () => {
+			await new Promise(resolve => setTimeout(resolve, 30000));
+		});
+
+		const job = await agenda.now('test');
+		await agenda.start();
+
+		await new Promise(resolve => agenda.on('start:test', resolve));
+
+		expect(await job.isRunning()).to.be.equal(true);
+	});
 });