simplify DelayTimer API

src/script/E2EIdentity/DelayTimer/DelayTimer.test.ts

@@ -18,7 +18,7 @@
  */
 
 import {FIFTEEN_MINUTES, FOUR_HOURS, ONE_HOUR, ONE_MINUTE} from './delay';
-import {DelayTimerService} from './DelayTimer'; // Update this with your module's actual path
+import {DelayTimerService} from './DelayTimer';
 
 describe('createGracePeriodTimer', () => {
   let timer: DelayTimerService | undefined;
@@ -27,16 +27,14 @@ describe('createGracePeriodTimer', () => {
     jest.clearAllMocks();
     jest.useFakeTimers();
     global.localStorage.clear();
-    timer = DelayTimerService?.getInstance({
+    timer = new DelayTimerService({
       gracePeriodInMS: 0,
       gracePeriodExpiredCallback: jest.fn(),
       delayPeriodExpiredCallback: jest.fn(),
     });
   });
 
   afterEach(() => {
-    timer?.resetInstance();
-    timer = undefined;
     jest.useRealTimers();
   });
 

src/script/E2EIdentity/DelayTimer/DelayTimer.ts

@@ -32,42 +32,21 @@ interface CreateGracePeriodTimerParams {
   delayPeriodExpiredCallback: () => void;
 }
 
-class DelayTimerService {
-  private static instance: DelayTimerService | null = null;
+export class DelayTimerService {
   private gracePeriodInMS: number;
   private gracePeriodExpiredCallback: () => void;
   private delayPeriodExpiredCallback: () => void;
   private readonly logger = logdown('@wireapp/core/DelayTimer');
   private delayPeriodTimerKey: string = 'E2EIdentity_DelayTimer';
   private gracePeriodTimerKey: string = 'E2EIdentity_GracePeriodTimer';
 
-  private constructor({
-    gracePeriodInMS,
-    gracePeriodExpiredCallback,
-    delayPeriodExpiredCallback,
-  }: CreateGracePeriodTimerParams) {
+  constructor({gracePeriodInMS, gracePeriodExpiredCallback, delayPeriodExpiredCallback}: CreateGracePeriodTimerParams) {
     this.gracePeriodInMS = gracePeriodInMS;
     this.gracePeriodExpiredCallback = gracePeriodExpiredCallback;
     this.delayPeriodExpiredCallback = delayPeriodExpiredCallback;
     this.initialize();
   }
 
-  /**
-   * Get the singleton instance of GracePeriodTimer or create a new one
-   * For the first time, params are required to create the instance
-   * @param params The params to create the grace period timer
-   * @returns The singleton instance of GracePeriodTimer
-   */
-  public static getInstance(params?: CreateGracePeriodTimerParams) {
-    if (!DelayTimerService.instance) {
-      if (!params) {
-        throw new Error('DelayTimerService is not initialized. Please call getInstance with params.');
-      }
-      DelayTimerService.instance = new DelayTimerService(params);
-    }
-    return DelayTimerService.instance;
-  }
-
   /**
    * @param CreateGracePeriodTimerParams The params to create the grace period timer
    */
@@ -263,16 +242,6 @@ class DelayTimerService {
       : 0;
   }
 
-  /**
-   * Reset the instance
-   */
-  public resetInstance() {
-    DelayTimerStore.clear.all();
-    this.clearGracePeriodTimer();
-    this.clearDelayPeriodTimer();
-    DelayTimerService.instance = null;
-  }
-
   public isDelayTimerActive() {
     return TaskScheduler.hasActiveTask(this.delayPeriodTimerKey);
   }
@@ -283,5 +252,3 @@ class DelayTimerService {
     return remainingTime - delayTime > 0;
   }
 }
-
-export {DelayTimerService};

src/script/E2EIdentity/E2EIdentityEnrollment.ts

@@ -126,7 +126,7 @@ export class E2EIHandler extends TypedEventEmitter<Events> {
       this.config = {
         discoveryUrl,
         gracePeriodInMs,
-        timer: DelayTimerService.getInstance({
+        timer: new DelayTimerService({
           gracePeriodInMS: gracePeriodInMs,
           gracePeriodExpiredCallback: () => null,
           delayPeriodExpiredCallback: () => null,