Skip to content

Commit 50a5c85

Browse files
authored
Merge pull request #20 from windingtree/fix/queue-test
fix: 🐛 Fixed queue test
2 parents fc58218 + bbec616 commit 50a5c85

File tree

2 files changed

+13
-6
lines changed

2 files changed

+13
-6
lines changed

src/shared/queue.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ export const createJobHandler =
108108
<OfferData = any, HandlerOptions extends object = object>(
109109
handler: JobHandler<OfferData, HandlerOptions>,
110110
) =>
111-
(options: HandlerOptions) =>
111+
(options: HandlerOptions = {} as HandlerOptions) =>
112112
(job: Job<OfferData>) =>
113113
handler(job, options);
114114
/* eslint-disable @typescript-eslint/no-explicit-any */

test/shared.queue.spec.ts

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,13 @@
11
import './setup.js';
22
import { memoryStorage } from '../src/storage/index.js';
3-
import { Queue, JobHandler, JobStatus, createJobHandler } from '../src/shared/queue.js';
3+
import {
4+
Queue,
5+
JobHandler,
6+
JobStatus,
7+
createJobHandler,
8+
JobHandlerClosure,
9+
Job,
10+
} from '../src/shared/queue.js';
411
import { nowSec } from '../src/utils/time.js';
512
import { expect } from 'chai';
613

@@ -110,7 +117,7 @@ describe('Shared.Queue', () => {
110117

111118
it('should process recurrent jobs', (done) => {
112119
const counter = 5;
113-
const handler: JobHandler = async () => {
120+
const handler: JobHandlerClosure = async () => {
114121
await new Promise((resolve) => setTimeout(resolve, 10));
115122
};
116123

@@ -139,7 +146,7 @@ describe('Shared.Queue', () => {
139146

140147
it('should cancel recurrent job using handler return', (done) => {
141148
const counter = 5;
142-
const handler: JobHandler = async (job) => {
149+
const handler: JobHandlerClosure = async (job: Job) => {
143150
await new Promise((resolve) => setTimeout(resolve, 10));
144151

145152
if (job.state.attempts >= counter) {
@@ -168,7 +175,7 @@ describe('Shared.Queue', () => {
168175

169176
it('should cancel recurrent job using max attempts option', (done) => {
170177
const counter = 5;
171-
const handler: JobHandler = async () => {
178+
const handler: JobHandlerClosure = async () => {
172179
await new Promise((resolve) => setTimeout(resolve, 10));
173180
};
174181

@@ -191,7 +198,7 @@ describe('Shared.Queue', () => {
191198
});
192199

193200
it('should cancel expired job', (done) => {
194-
const handler: JobHandler = async () => {
201+
const handler: JobHandlerClosure = async () => {
195202
await new Promise((resolve) => setTimeout(resolve, 10));
196203
};
197204

0 commit comments

Comments
 (0)