Update timer when removing timeout

mittwald · Sep 6, 2023 · 8835673 · 8835673
1 parent 2261ac0
commit 8835673
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/src/lib/ConsolidatedTimeout.test.ts b/src/lib/ConsolidatedTimeout.test.ts
@@ -81,3 +81,10 @@ test("Callback is triggered instantly when adding due timeout while already runn
   timeout.addTimeout({ milliseconds: 500 });
   expect(callback).toHaveBeenCalledTimes(1);
 });
+
+test("Removing last timeout will not trigger callback", (): void => {
+  timeout.start();
+  const removeTimeout = timeout.addTimeout({ milliseconds: 1000 });
+  removeTimeout();
+  testCallbackIsNotCalledAfter(1000);
+});
diff --git a/src/lib/ConsolidatedTimeout.ts b/src/lib/ConsolidatedTimeout.ts
@@ -35,6 +35,7 @@ export class ConsolidatedTimeout {
 
     return () => {
       this.timeoutMillis.delete(timeoutMs);
+      this.startNextTimeout();
     };
   }
-Original file line number
+Diff line change
@@ Expand Up / @@ -35,6 +35,7 @@ export class ConsolidatedTimeout { @@
         return () => {
           this.timeoutMillis.delete(timeoutMs);
+          this.startNextTimeout();
         };
       }
@@ Expand Down @@