From 66e6ed65e9c9281a68b548ab01a2214d5a0aa1e3 Mon Sep 17 00:00:00 2001 From: Yusuke Tanaka Date: Thu, 1 Feb 2024 12:48:27 +0900 Subject: fix(node): add `ppid` getter for `node:process` (#22167) This commit adds `ppid` getter for `node:process` to improve Node compatibility one step further. There is one problem though, which is that `Deno.ppid`, which `process.ppid` internally calls, is actually of type `bigint` although it's supposed to be `number`. I filed an issue for this (#22166). For the time being, explciit type conversion from `bigint` to `number` is applied to match the Node.js behavior. --- cli/tests/unit_node/process_test.ts | 8 ++++++++ 1 file changed, 8 insertions(+) (limited to 'cli') diff --git a/cli/tests/unit_node/process_test.ts b/cli/tests/unit_node/process_test.ts index 23bf73864..21f873260 100644 --- a/cli/tests/unit_node/process_test.ts +++ b/cli/tests/unit_node/process_test.ts @@ -121,6 +121,14 @@ Deno.test({ }, }); +Deno.test({ + name: "process.ppid", + fn() { + assertEquals(typeof process.ppid, "number"); + assertEquals(process.ppid, Deno.ppid); + }, +}); + Deno.test({ name: "process.on", async fn() { -- cgit v1.2.3