@orion-js/dogs 3.1.0-alpha.5 → 3.1.0-alpha.8

package/lib/events.test.js

@@ -74,8 +74,7 @@ describe('Event tests', () => {
         const job = (0, _1.defineJob)({
             type: 'event',
             async resolve(params, context) {
-                if (ranCount === 1) {
-                    expect(context.tries).toBe(2);
+                if (context.tries === 2) {
                     context.extendLockTime(10000);
                 }
                 await (0, helpers_1.sleep)(100);
@@ -88,20 +87,20 @@ describe('Event tests', () => {
         });
         const instance = (0, _1.startWorkers)({
             jobs: { [jobId]: job },
-            workersCount: 1,
+            workersCount: 2,
             pollInterval: 10,
             cooldownPeriod: 10,
-            lockTime: 40,
+            lockTime: 10,
             logLevel: 'info'
         });
         await (0, _1.scheduleJob)({
             name: jobId,
             runIn: 1
         });
-        await (0, helpers_1.sleep)(200);
+        await (0, helpers_1.sleep)(300);
         await instance.stop();
-        expect(ranCount).toBe(2);
         expect(staleCount).toBe(1);
+        expect(ranCount).toBe(2);
     });
     it('Should only schedule one job with uniqueIdentifier', async () => {
         const jobId = (0, helpers_1.generateId)();

package/lib/history.test.js

@@ -34,7 +34,7 @@ describe('Test Jobs History', () => {
             _id: expect.any(String),
             executionId: expect.any(String),
             jobName: jobId,
-            isRecurrent: false,
+            type: 'event',
             priority: 1,
             tries: 1,
             startedAt: expect.any(Date),
@@ -75,7 +75,7 @@ describe('Test Jobs History', () => {
             _id: expect.any(String),
             executionId: expect.any(String),
             jobName: jobId,
-            isRecurrent: false,
+            type: 'event',
             priority: 1,
             tries: 1,
             startedAt: expect.any(Date),
@@ -119,7 +119,7 @@ describe('Test Jobs History', () => {
             _id: expect.any(String),
             executionId: expect.any(String),
             jobName: jobId,
-            isRecurrent: false,
+            type: 'event',
             priority: 1,
             tries: 1,
             startedAt: expect.any(Date),
@@ -137,7 +137,7 @@ describe('Test Jobs History', () => {
             _id: expect.any(String),
             executionId: expect.any(String),
             jobName: jobId,
-            isRecurrent: false,
+            type: 'event',
             priority: 1,
             tries: 1,
             startedAt: expect.any(Date),

package/lib/index.d.ts

@@ -3,8 +3,9 @@ import { StartWorkersConfig } from './types/StartConfig';
 import { ScheduleJobOptions } from './types/Events';
 import { JobsHistoryRepo } from './repos/JobsHistoryRepo';
 import { JobsRepo } from './repos/JobsRepo';
+export * from './types';
 declare const jobsHistoryRepo: JobsHistoryRepo;
 declare const jobsRepo: JobsRepo;
-declare const startWorkers: (config: Partial<StartWorkersConfig>) => import("./types/Worker").WorkersInstance;
+declare const startWorkers: (config: Partial<StartWorkersConfig>) => import("./types").WorkersInstance;
 declare const scheduleJob: (options: ScheduleJobOptions) => Promise<void>;
 export { defineJob, startWorkers, scheduleJob, jobsHistoryRepo, jobsRepo };

package/lib/index.js

@@ -1,4 +1,14 @@
 "use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    Object.defineProperty(o, k2, { enumerable: true, get: function() { return m[k]; } });
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __exportStar = (this && this.__exportStar) || function(m, exports) {
+    for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
+};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.jobsRepo = exports.jobsHistoryRepo = exports.scheduleJob = exports.startWorkers = exports.defineJob = void 0;
 const services_1 = require("@orion-js/services");
@@ -8,6 +18,7 @@ const EventsService_1 = require("./services/EventsService");
 const WorkerService_1 = require("./services/WorkerService");
 const JobsHistoryRepo_1 = require("./repos/JobsHistoryRepo");
 const JobsRepo_1 = require("./repos/JobsRepo");
+__exportStar(require("./types"), exports);
 const workerService = (0, services_1.getInstance)(WorkerService_1.WorkerService);
 const eventsService = (0, services_1.getInstance)(EventsService_1.EventsService);
 const jobsHistoryRepo = (0, services_1.getInstance)(JobsHistoryRepo_1.JobsHistoryRepo);

package/lib/repos/JobsRepo.d.ts

@@ -10,6 +10,7 @@ export declare class JobsRepo {
         nextRunAt: Date;
         addTries: boolean;
     }): Promise<void>;
+    deleteEventJob(jobId: string): Promise<void>;
     extendLockTime(jobId: string, extraTime: number): Promise<void>;
     ensureJobRecord(job: JobDefinitionWithName): Promise<void>;
     scheduleJob(options: ScheduleJobRecordOptions): Promise<void>;

package/lib/repos/JobsRepo.js

@@ -31,7 +31,7 @@ let JobsRepo = class JobsRepo {
                     },
                     options: {
                         unique: true,
-                        partialFilterExpression: { isRecurrent: true }
+                        partialFilterExpression: { type: 'recurrent' }
                     }
                 },
                 {
@@ -53,7 +53,7 @@ let JobsRepo = class JobsRepo {
             nextRunAt: { $lte: new Date() },
             $or: [{ lockedUntil: { $exists: false } }, { lockedUntil: { $lte: new Date() } }]
         }, {
-            $set: { lockedUntil }
+            $set: { lockedUntil, lastRunAt: new Date() }
         }, {
             mongoOptions: {
                 sort: {
@@ -75,7 +75,7 @@ let JobsRepo = class JobsRepo {
             jobId: job._id,
             name: job.jobName,
             params: job.params,
-            isRecurrent: job.isRecurrent,
+            type: job.type,
             tries,
             lockTime,
             priority: job.priority,
@@ -92,6 +92,9 @@ let JobsRepo = class JobsRepo {
         }
         await this.jobs.updateOne(options.jobId, updator);
     }
+    async deleteEventJob(jobId) {
+        await this.jobs.deleteOne({ _id: jobId, type: 'event' });
+    }
     async extendLockTime(jobId, extraTime) {
         const lockedUntil = new Date(Date.now() + extraTime);
         await this.jobs.updateOne({
@@ -105,7 +108,7 @@ let JobsRepo = class JobsRepo {
             jobName: job.name
         }, {
             $set: {
-                isRecurrent: true,
+                type: job.type,
                 priority: job.priority || 1
             },
             $setOnInsert: {
@@ -127,7 +130,7 @@ let JobsRepo = class JobsRepo {
                 params: options.params,
                 nextRunAt: options.nextRunAt,
                 priority: options.priority,
-                isRecurrent: false
+                type: 'event'
             });
         }
         catch (error) {

package/lib/services/Executor.d.ts

@@ -15,5 +15,6 @@ export declare class Executor {
         job: JobDefinition;
         jobToRun: JobToRun;
     }): Promise<void>;
+    afterExecutionSuccess(job: JobDefinition, jobToRun: JobToRun): Promise<void>;
     executeJob(jobs: JobsDefinition, jobToRun: JobToRun): Promise<void>;
 }

package/lib/services/Executor.js

@@ -32,12 +32,8 @@ let Executor = class Executor {
     }
     getJobDefinition(jobToRun, jobs) {
         const job = jobs[jobToRun.name];
-        if (!jobToRun.isRecurrent && job.type === 'recurrent') {
-            (0, log_1.log)('warn', `Job record ${jobToRun.name} is event but definition is recurrent`);
-            return;
-        }
-        if (jobToRun.isRecurrent && job.type === 'event') {
-            (0, log_1.log)('warn', `Job record ${jobToRun.name} is recurrent but definition is event`);
+        if (jobToRun.type !== job.type) {
+            (0, log_1.log)('warn', `Job record "${jobToRun.name}" is "${jobToRun.type}" but definition is "${job.type}"`);
             return;
         }
         return job;
@@ -81,7 +77,7 @@ let Executor = class Executor {
             await this.jobsHistoryRepo.saveExecution({
                 executionId: jobToRun.jobId,
                 jobName: jobToRun.name,
-                isRecurrent: jobToRun.isRecurrent,
+                type: jobToRun.type,
                 priority: jobToRun.priority,
                 tries: jobToRun.tries,
                 uniqueIdentifier: jobToRun.uniqueIdentifier,
@@ -96,6 +92,20 @@ let Executor = class Executor {
             });
         }
     }
+    async afterExecutionSuccess(job, jobToRun) {
+        if (job.type === 'recurrent') {
+            (0, log_1.log)('debug', `Scheduling next run for recurrent job "${jobToRun.name}"`);
+            await this.jobsRepo.scheduleNextRun({
+                jobId: jobToRun.jobId,
+                nextRunAt: (0, getNextRunDate_1.getNextRunDate)(job),
+                addTries: false
+            });
+        }
+        if (job.type === 'event') {
+            (0, log_1.log)('debug', `Removing event job after success "${jobToRun.name}"`);
+            await this.jobsRepo.deleteEventJob(jobToRun.jobId);
+        }
+    }
     async executeJob(jobs, jobToRun) {
         const job = this.getJobDefinition(jobToRun, jobs);
         if (!job)
@@ -130,13 +140,7 @@ let Executor = class Executor {
                 job,
                 jobToRun
             });
-            if (job.type === 'recurrent') {
-                await this.jobsRepo.scheduleNextRun({
-                    jobId: jobToRun.jobId,
-                    nextRunAt: (0, getNextRunDate_1.getNextRunDate)(job),
-                    addTries: false
-                });
-            }
+            await this.afterExecutionSuccess(job, jobToRun);
         }
         catch (error) {
             context.clearStaleTimeout();

package/lib/types/HistoryRecord.d.ts

@@ -5,7 +5,7 @@ export declare class HistoryRecord {
     _id: string;
     executionId: string;
     jobName: string;
-    isRecurrent: boolean;
+    type: 'recurrent' | 'event';
     priority: number;
     tries: number;
     uniqueIdentifier?: string;

package/lib/types/HistoryRecord.js

@@ -27,8 +27,8 @@ __decorate([
 ], HistoryRecord.prototype, "jobName", void 0);
 __decorate([
     (0, typed_model_1.Prop)(),
-    __metadata("design:type", Boolean)
-], HistoryRecord.prototype, "isRecurrent", void 0);
+    __metadata("design:type", String)
+], HistoryRecord.prototype, "type", void 0);
 __decorate([
     (0, typed_model_1.Prop)(),
     __metadata("design:type", Number)

package/lib/types/JobRecord.d.ts

@@ -2,7 +2,7 @@ import { PlainObject } from './HistoryRecord';
 export declare class JobRecord {
     _id: string;
     jobName: string;
-    isRecurrent: boolean;
+    type: 'recurrent' | 'event';
     priority: number;
     uniqueIdentifier?: string;
     nextRunAt: Date;

package/lib/types/JobRecord.js

@@ -23,8 +23,8 @@ __decorate([
 ], JobRecord.prototype, "jobName", void 0);
 __decorate([
     (0, typed_model_1.Prop)(),
-    __metadata("design:type", Boolean)
-], JobRecord.prototype, "isRecurrent", void 0);
+    __metadata("design:type", String)
+], JobRecord.prototype, "type", void 0);
 __decorate([
     (0, typed_model_1.Prop)(),
     __metadata("design:type", Number)

package/lib/types/Worker.d.ts

@@ -3,7 +3,7 @@ import { JobDefinition } from './JobsDefinition';
 export interface JobToRun {
     jobId: string;
     name: string;
-    isRecurrent: boolean;
+    type: 'event' | 'recurrent';
     params: PlainObject;
     tries: number;
     lockTime: number;

package/lib/types/index.d.ts

@@ -0,0 +1,6 @@
+export * from './Events';
+export * from './HistoryRecord';
+export * from './JobRecord';
+export * from './JobsDefinition';
+export * from './StartConfig';
+export * from './Worker';

package/lib/types/index.js

@@ -0,0 +1,18 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    Object.defineProperty(o, k2, { enumerable: true, get: function() { return m[k]; } });
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __exportStar = (this && this.__exportStar) || function(m, exports) {
+    for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+__exportStar(require("./Events"), exports);
+__exportStar(require("./HistoryRecord"), exports);
+__exportStar(require("./JobRecord"), exports);
+__exportStar(require("./JobsDefinition"), exports);
+__exportStar(require("./StartConfig"), exports);
+__exportStar(require("./Worker"), exports);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@orion-js/dogs",
-  "version": "3.1.0-alpha.5",
+  "version": "3.1.0-alpha.8",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",
   "files": [
@@ -35,5 +35,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "a5a1a46dd58b86b6bfb9523d1352b3d57891cb1c"
+  "gitHead": "f11193ee2cc10b9a7c99ef618ae284d319a34de4"
 }