From eff3e432966f6bc9ed909ba22fcafc0978c924d7 Mon Sep 17 00:00:00 2001 From: Tareque Md Hanif Date: Sat, 11 Nov 2023 03:29:09 +0600 Subject: chore(cli): Migrate some unit tests to "Promise.withResolvers()" (#21128) Migrate to use `Promise.withResolvers()` instead of `deferred` in some of the tests in `cli/tests/unit/`. Issue: #21041 --- cli/tests/unit/signal_test.ts | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) (limited to 'cli/tests/unit/signal_test.ts') diff --git a/cli/tests/unit/signal_test.ts b/cli/tests/unit/signal_test.ts index 10ed32489..e4e0b0227 100644 --- a/cli/tests/unit/signal_test.ts +++ b/cli/tests/unit/signal_test.ts @@ -1,5 +1,5 @@ // Copyright 2018-2023 the Deno authors. All rights reserved. MIT license. -import { assertEquals, assertThrows, deferred, delay } from "./test_util.ts"; +import { assertEquals, assertThrows, delay } from "./test_util.ts"; Deno.test( { ignore: Deno.build.os !== "windows" }, @@ -110,7 +110,7 @@ Deno.test( permissions: { run: true }, }, async function signalListenerTest() { - const resolvable = deferred(); + const { promise, resolve } = Promise.withResolvers(); let c = 0; const listener = () => { c += 1; @@ -124,10 +124,10 @@ Deno.test( } await delay(20); Deno.removeSignalListener("SIGUSR1", listener); - resolvable.resolve(); + resolve(); }); - await resolvable; + await promise; assertEquals(c, 3); }, ); @@ -138,7 +138,7 @@ Deno.test( permissions: { run: true }, }, async function multipleSignalListenerTest() { - const resolvable = deferred(); + const { promise, resolve } = Promise.withResolvers(); let c = ""; const listener0 = () => { c += "0"; @@ -169,10 +169,10 @@ Deno.test( } await delay(20); Deno.removeSignalListener("SIGUSR2", listener0); - resolvable.resolve(); + resolve(); }); - await resolvable; + await promise; // The first 3 events are handled by both handlers // The last 3 events are handled only by handler0 assertEquals(c, "010101000"); -- cgit v1.2.3