change JitterTypes from enum to union (#19)

coveooss · Apr 3, 2020 · 16f3f46 · 16f3f46
1 parent f8aaeb8
commit 16f3f46
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 9 deletions.
diff --git a/src/jitter/jitter.factory.ts b/src/jitter/jitter.factory.ts
@@ -1,15 +1,15 @@
-import { IBackOffOptions, JitterTypes } from "../options";
+import { IBackOffOptions } from "../options";
 import { fullJitter } from "./full/full.jitter";
 import { noJitter } from "./no/no.jitter";
 
 export type Jitter = (delay: number) => number;
 
 export function JitterFactory(options: IBackOffOptions): Jitter {
     switch (options.jitter) {
-        case JitterTypes.Full:
+        case "full":
             return fullJitter
 
-        case JitterTypes.None:
+        case "none":
         default:
             return noJitter
     }

diff --git a/src/options.ts b/src/options.ts
@@ -1,11 +1,8 @@
-export enum JitterTypes {
-  None = "none",
-  Full = "full"
-}
+export type JitterType = "none" | "full";
 
 export interface IBackOffOptions {
   delayFirstAttempt: boolean;
-  jitter: JitterTypes;
+  jitter: JitterType;
   maxDelay: number;
   numOfAttempts: number;
   retry: (e: any, attemptNumber: number) => boolean;
@@ -15,7 +12,7 @@ export interface IBackOffOptions {
 
 const defaultOptions: IBackOffOptions = {
   delayFirstAttempt: false,
-  jitter: JitterTypes.None,
+  jitter: "none",
   maxDelay: Infinity,
   numOfAttempts: 10,
   retry: () => true,