fix(backoff): change optional BackoffStrategy params to use union (#2365

)

src/interfaces/repeat-options.ts

@@ -30,7 +30,7 @@ export interface RepeatOptions extends Omit<ParserOptions, 'iterator'> {
   every?: number;
   /**
    * Repeated job should start right now
-   * ( work only with every settings)
+   * (work only with every settings)
    */
   immediately?: boolean;
   /**

src/types/backoff-strategy.ts

@@ -1,8 +1,30 @@
 import { MinimalJob } from '../interfaces/minimal-job';
 
-export type BackoffStrategy = (
+export type BackoffStrategy4 = (
   attemptsMade: number,
-  type?: string,
-  err?: Error,
-  job?: MinimalJob,
+  type: string,
+  err: Error,
+  job: MinimalJob,
 ) => Promise<number> | number;
+
+export type BackoffStrategy3 = (
+  attemptsMade: number,
+  type: string,
+  err: Error,
+) => Promise<number> | number;
+
+export type BackoffStrategy2 = (
+  attemptsMade: number,
+  type: string,
+) => Promise<number> | number;
+
+export type BackoffStrategy1 = (attemptsMade: number) => Promise<number> | number;
+
+export type BackoffStrategy0 = () => Promise<number> | number;
+
+export type BackoffStrategy =
+  | BackoffStrategy4
+  | BackoffStrategy3
+  | BackoffStrategy2
+  | BackoffStrategy1
+  | BackoffStrategy0;

tests/test_worker.ts

@@ -3589,12 +3589,7 @@ describe('workers', function () {
             connection,
             prefix,
             settings: {
-              backoffStrategy: (
-                attemptsMade: number,
-                type: string,
-                err: Error,
-                job: MinimalJob,
-              ) => {
+              backoffStrategy: (attemptsMade: number, type: string) => {
                 switch (type) {
                   case 'custom1': {
                     return attemptsMade * 1000;