From 147bfc9c56d363daaed9ae83d3abc9a61934ed06 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bartek=20Iwa=C5=84czuk?= Date: Wed, 17 Jan 2024 02:18:19 +0100 Subject: [PATCH] refactor: change tests to not rely on Deno.run() (#21961) For https://github.com/denoland/deno/pull/21939 --- .../registry/@denotest/child-process-fork/1.0.0/index.js | 7 ++++--- cli/tests/testdata/test/captured_output.ts | 9 +++------ 2 files changed, 7 insertions(+), 9 deletions(-) diff --git a/cli/tests/testdata/npm/registry/@denotest/child-process-fork/1.0.0/index.js b/cli/tests/testdata/npm/registry/@denotest/child-process-fork/1.0.0/index.js index 6f7dceb98e..9f84afe354 100644 --- a/cli/tests/testdata/npm/registry/@denotest/child-process-fork/1.0.0/index.js +++ b/cli/tests/testdata/npm/registry/@denotest/child-process-fork/1.0.0/index.js @@ -1,13 +1,14 @@ const path = require("path"); function childProcessFork(path) { - const p = Deno.run({ - cmd: [Deno.execPath(), "run", "--unstable", "-A", path], + const command = new Deno.Command(Deno.execPath(), { + args: ["run", "--unstable", "-A", path], env: { "DENO_DONT_USE_INTERNAL_NODE_COMPAT_STATE": Deno[Deno.internal].core.ops.op_npm_process_state(), } }); - p.status().then(() => { + const child = command.spawn(); + child.status.then(() => { console.log("Done."); }); } diff --git a/cli/tests/testdata/test/captured_output.ts b/cli/tests/testdata/test/captured_output.ts index 905156fd41..3eed249a2e 100644 --- a/cli/tests/testdata/test/captured_output.ts +++ b/cli/tests/testdata/test/captured_output.ts @@ -1,10 +1,7 @@ Deno.test("output", async () => { - // deno-lint-ignore no-deprecated-deno-api - const p = Deno.run({ - cmd: [Deno.execPath(), "eval", "console.log(0); console.error(1);"], - }); - await p.status(); - await p.close(); + await new Deno.Command(Deno.execPath(), { + args: ["eval", "console.log(0); console.error(1);"], + }).spawn().status; new Deno.Command(Deno.execPath(), { args: ["eval", "console.log(2); console.error(3);"], stdout: "inherit",