Dylan/fix some failing tests (#2544)

* handle `umask()` invalid arguments

* move `bktree-fast` to test root, fix postinstall

* fix fs test

* could be baseline

* handle different timezones

* accidentally deleted tests

* fix hang in `AbortSignal.timeout`

* bring abort tests back

* Revert "bring abort tests back"

This reverts commit 0ff2ad5bf408694ac719b8ba0a38b16070e10201.

* bunx node-gyp

* bun x

* fix typecheck

* test

* Update inspect.test.js

---------

Co-authored-by: Dylan Conway <dylan.conway567@gmail.com>

5 files changed, 34 insertions, 23 deletions

diff --git a/test/js/bun/util/inspect.test.js b/test/js/bun/util/inspect.test.js
index 8409a07c4..5d2f27c61 100644
--- a/test/js/bun/util/inspect.test.js
+++ b/test/js/bun/util/inspect.test.js
@@ -339,11 +339,18 @@ it("possibly formatted emojis log", () => {
 });
 
 it("new Date(..)", () => {
-  expect(Bun.inspect(new Date(1679911059000))).toBe("2023-03-27T09:54:00.000Z");
-  expect(Bun.inspect(new Date("March 27, 2023 09:54:00"))).toBe("2023-03-27T09:54:00.000Z");
-  expect(Bun.inspect(new Date("2023-03-27T09:54:00"))).toBe("2023-03-27T09:54:00.000Z");
-  expect(Bun.inspect(new Date(2023, 02, 27))).toBe("2023-03-27T00:00:00.000Z");
-  expect(Bun.inspect(new Date(2023, 02, 27, 09, 54, 0))).toBe("2023-03-27T09:54:00.000Z");
+  let s = Bun.inspect(new Date(1679911059000 - new Date().getTimezoneOffset()));
+  expect(s).toContain("2023-03-27T");
+  expect(s).toHaveLength(24);
+  let offset = new Date().getTimezoneOffset() / 60;
+  let hour = (9 - offset).toString();
+  if (hour.length === 1) {
+    hour = "0" + hour;
+  }
+  expect(Bun.inspect(new Date("March 27, 2023 " + hour + ":54:00"))).toBe("2023-03-27T09:54:00.000Z");
+  expect(Bun.inspect(new Date("2023-03-27T" + hour + ":54:00"))).toBe("2023-03-27T09:54:00.000Z");
+  expect(Bun.inspect(new Date(2023, 02, 27, -offset))).toBe("2023-03-27T00:00:00.000Z");
+  expect(Bun.inspect(new Date(2023, 02, 27, 09 - offset, 54, 0))).toBe("2023-03-27T09:54:00.000Z");
 
   expect(Bun.inspect(new Date("1679911059000"))).toBe("Invalid Date");
   expect(Bun.inspect(new Date("hello world"))).toBe("Invalid Date");
diff --git a/test/js/node/fs/fs.test.ts b/test/js/node/fs/fs.test.ts
index 80f6c7dfe..fa27ab246 100644
--- a/test/js/node/fs/fs.test.ts
+++ b/test/js/node/fs/fs.test.ts
@@ -530,7 +530,7 @@ describe("rmdir", () => {
     } catch (e) {}
     expect(existsSync(path + "/file.txt")).toBe(true);
     rmdir(path, err => {
-      expect("ENOTEMPTY").toContain(err!.code);
+      expect("ENOTEMPTY EPERM").toContain(err!.code);
       done();
     });
     expect(existsSync(path + "/file.txt")).toBe(true);
diff --git a/test/js/node/process/process.test.js b/test/js/node/process/process.test.js
index 68d72e056..414392a79 100644
--- a/test/js/node/process/process.test.js
+++ b/test/js/node/process/process.test.js
@@ -66,10 +66,10 @@ it("process.hrtime.bigint()", () => {
 
 it("process.release", () => {
   expect(process.release.name).toBe("bun");
-  expect(process.release.sourceUrl).toBe(
+  expect(process.release.sourceUrl).toContain(
     `https://github.com/oven-sh/bun/release/bun-v${process.versions.bun}/bun-${process.platform}-${
       { arm64: "aarch64", x64: "x64" }[process.arch] || process.arch
-    }.zip`,
+    }`,
   );
 });
 
@@ -117,9 +117,26 @@ it("process.uptime()", () => {
 });
 
 it("process.umask()", () => {
-  const orig = process.umask(777);
+  let notNumbers = [265n, "string", true, false, null, {}, [], () => {}, Symbol("symbol"), BigInt(1)];
+  for (let notNumber of notNumbers) {
+    expect(() => {
+      process.umask(notNumber);
+    }).toThrow('The "mask" argument must be a number');
+  }
+
+  let rangeErrors = [NaN, -1.4, Infinity, -Infinity, -1, 1.3, 4294967296];
+  for (let rangeError of rangeErrors) {
+    expect(() => {
+      process.umask(rangeError);
+    }).toThrow(RangeError);
+  }
+
+  const orig = process.umask(0o777);
   expect(orig).toBeGreaterThan(0);
-  expect(process.umask(orig)).toBe(777);
+  expect(process.umask()).toBe(0o777);
+  expect(process.umask(undefined)).toBe(0o777);
+  expect(process.umask(Number(orig))).toBe(0o777);
+  expect(process.umask()).toBe(orig);
 });
 
 const versions = existsSync(import.meta.dir + "/../../src/generated_versions_list.zig");
diff --git a/test/js/third_party/napi_create_external/bun.lockb b/test/js/third_party/napi_create_external/bun.lockb
deleted file mode 100755
index 7f6d3e95d..000000000
--- a/test/js/third_party/napi_create_external/bun.lockb
+++ /dev/null
diff --git a/test/js/third_party/napi_create_external/package.json b/test/js/third_party/napi_create_external/package.json
deleted file mode 100644
index 659b279b2..000000000
--- a/test/js/third_party/napi_create_external/package.json
+++ /dev/null
@@ -1,13 +0,0 @@
-{
-  "name": "napi-create-external-test",
-  "type": "module",
-  "version": "1.0.0",
-  "description": "Test for napi_create_external",
-  "dependencies": {
-    "bktree-fast": "0.0.7",
-    "lodash": "^4.17.21"
-  },
-  "scripts": {
-    "postinstall": "cd node_modules/bktree-fast && node-gyp configure"
-  }
-}