fixup! actor alarms tests: Reject waiting promise on alarm handler ex…

…ception
cloudflare · Sep 17, 2024 · d2417f9 · d2417f9
1 parent 5a8ac6b
commit d2417f9
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 10 deletions.
diff --git a/src/workerd/api/actor-alarms-delete-test.js b/src/workerd/api/actor-alarms-delete-test.js
@@ -14,13 +14,12 @@ export class DurableObjectExample {
 
   async waitForAlarm(scheduledTime) {
     let self = this;
-    let prom = new Promise((resolve, reject) => {
-      self.resolve = resolve;
-      self.reject = reject;
-    });
+    const { promise, resolve, reject } = Promise.withResolvers();
+    self.resolve = resolve;
+    self.reject = reject;
 
     try {
-      await prom;
+      await promise;
       if (Date.now() < scheduledTime.valueOf()) {
         throw new Error(
           `Date.now() is before scheduledTime! ${Date.now()} vs ${scheduledTime.valueOf()}`

diff --git a/src/workerd/api/actor-alarms-test.js b/src/workerd/api/actor-alarms-test.js
@@ -11,13 +11,12 @@ export class DurableObjectExample {
 
   async waitForAlarm(scheduledTime) {
     let self = this;
-    let prom = new Promise((resolve, reject) => {
-      self.resolve = resolve;
-      self.reject = reject;
-    });
+    const { promise, resolve, reject } = Promise.withResolvers();
+    self.resolve = resolve;
+    self.reject = reject;
 
     try {
-      await prom;
+      await promise;
       if (Date.now() < scheduledTime.valueOf()) {
         throw new Error(
           `Date.now() is before scheduledTime! ${Date.now()} vs ${scheduledTime.valueOf()}`