From f5e46c9bf2f50d66a953fa133161fc829cecff06 Mon Sep 17 00:00:00 2001
From: Matt Mastracci <matthew@mastracci.com>
Date: Sat, 10 Feb 2024 13:22:13 -0700
Subject: chore: move cli/tests/ -> tests/  (#22369)

This looks like a massive PR, but it's only a move from cli/tests ->
tests, and updates of relative paths for files.

This is the first step towards aggregate all of the integration test
files under tests/, which will lead to a set of integration tests that
can run without the CLI binary being built.

While we could leave these tests under `cli`, it would require us to
keep a more complex directory structure for the various test runners. In
addition, we have a lot of complexity to ignore various test files in
the `cli` project itself (cargo publish exclusion rules, autotests =
false, etc).

And finally, the `tests/` folder will eventually house the `test_ffi`,
`test_napi` and other testing code, reducing the size of the root repo
directory.

For easier review, the extremely large and noisy "move" is in the first
commit (with no changes -- just a move), while the remainder of the
changes to actual files is in the second commit.
---
 tests/testdata/check/node_builtin_modules/mod.js     |  3 +++
 tests/testdata/check/node_builtin_modules/mod.js.out |  5 +++++
 tests/testdata/check/node_builtin_modules/mod.ts     |  9 +++++++++
 tests/testdata/check/node_builtin_modules/mod.ts.out | 13 +++++++++++++
 4 files changed, 30 insertions(+)
 create mode 100644 tests/testdata/check/node_builtin_modules/mod.js
 create mode 100644 tests/testdata/check/node_builtin_modules/mod.js.out
 create mode 100644 tests/testdata/check/node_builtin_modules/mod.ts
 create mode 100644 tests/testdata/check/node_builtin_modules/mod.ts.out

(limited to 'tests/testdata/check/node_builtin_modules')

diff --git a/tests/testdata/check/node_builtin_modules/mod.js b/tests/testdata/check/node_builtin_modules/mod.js
new file mode 100644
index 000000000..196fb9be9
--- /dev/null
+++ b/tests/testdata/check/node_builtin_modules/mod.js
@@ -0,0 +1,3 @@
+// @ts-check
+import fs from "node:fs";
+const _data = fs.readFileSync("./node_builtin.js", 123);
diff --git a/tests/testdata/check/node_builtin_modules/mod.js.out b/tests/testdata/check/node_builtin_modules/mod.js.out
new file mode 100644
index 000000000..97786ebae
--- /dev/null
+++ b/tests/testdata/check/node_builtin_modules/mod.js.out
@@ -0,0 +1,5 @@
+error: TS2769 [ERROR]: No overload matches this call.
+  [WILDCARD]
+const _data = fs.readFileSync("./node_builtin.js", 123);
+                                                   ~~~
+    at file:///[WILDCARD]/node_builtin_modules/mod.js:3:52
diff --git a/tests/testdata/check/node_builtin_modules/mod.ts b/tests/testdata/check/node_builtin_modules/mod.ts
new file mode 100644
index 000000000..0e62353fe
--- /dev/null
+++ b/tests/testdata/check/node_builtin_modules/mod.ts
@@ -0,0 +1,9 @@
+import fs from "node:fs";
+const _data = fs.readFileSync("./node_builtin.js", 123);
+
+// check node:module specifically because for deno check it should
+// resolve to the @types/node package, but at runtime it uses a different
+// builtin object than deno_std
+import { builtinModules } from "node:module";
+// should error about being string[]
+const _testString: number[] = builtinModules;
diff --git a/tests/testdata/check/node_builtin_modules/mod.ts.out b/tests/testdata/check/node_builtin_modules/mod.ts.out
new file mode 100644
index 000000000..49b762cff
--- /dev/null
+++ b/tests/testdata/check/node_builtin_modules/mod.ts.out
@@ -0,0 +1,13 @@
+error: TS2769 [ERROR]: No overload matches this call.
+  [WILDCARD]
+const _data = fs.readFileSync("./node_builtin.js", 123);
+                                                   ~~~
+    at file:///[WILDCARD]/node_builtin_modules/mod.ts:2:52
+
+TS2322 [ERROR]: Type 'string[]' is not assignable to type 'number[]'.
+  Type 'string' is not assignable to type 'number'.
+const _testString: number[] = builtinModules;
+      ~~~~~~~~~~~
+    at file:///[WILDCARD]/node_builtin_modules/mod.ts:9:7
+
+Found 2 errors.
-- 
cgit v1.2.3