From f5e46c9bf2f50d66a953fa133161fc829cecff06 Mon Sep 17 00:00:00 2001 From: Matt Mastracci 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. --- cli/tests/testdata/npm/node_modules_import/main.out | 3 --- cli/tests/testdata/npm/node_modules_import/main.ts | 16 ---------------- .../testdata/npm/node_modules_import/main_check.out | 16 ---------------- cli/tests/testdata/npm/node_modules_import/package.json | 5 ----- 4 files changed, 40 deletions(-) delete mode 100644 cli/tests/testdata/npm/node_modules_import/main.out delete mode 100644 cli/tests/testdata/npm/node_modules_import/main.ts delete mode 100644 cli/tests/testdata/npm/node_modules_import/main_check.out delete mode 100644 cli/tests/testdata/npm/node_modules_import/package.json (limited to 'cli/tests/testdata/npm/node_modules_import') diff --git a/cli/tests/testdata/npm/node_modules_import/main.out b/cli/tests/testdata/npm/node_modules_import/main.out deleted file mode 100644 index 083edaac2..000000000 --- a/cli/tests/testdata/npm/node_modules_import/main.out +++ /dev/null @@ -1,3 +0,0 @@ -2 -2 -2 diff --git a/cli/tests/testdata/npm/node_modules_import/main.ts b/cli/tests/testdata/npm/node_modules_import/main.ts deleted file mode 100644 index 848ca0f81..000000000 --- a/cli/tests/testdata/npm/node_modules_import/main.ts +++ /dev/null @@ -1,16 +0,0 @@ -import * as myImport1 from "@denotest/esm-basic"; -import * as myImport2 from "./node_modules/@denotest/esm-basic/main.mjs"; -import * as myImport3 from "@denotest/esm-basic/main.mjs"; - -myImport1.setValue(5); -myImport2.setValue(2); - -// these should all give type errors -const value1: string = myImport1.getValue(); -const value2: string = myImport2.getValue(); -const value3: string = myImport3.getValue(); - -// these should all be equal because it should be mutating the same module -console.log(value1); -console.log(value2); -console.log(value3); diff --git a/cli/tests/testdata/npm/node_modules_import/main_check.out b/cli/tests/testdata/npm/node_modules_import/main_check.out deleted file mode 100644 index cf7cc110d..000000000 --- a/cli/tests/testdata/npm/node_modules_import/main_check.out +++ /dev/null @@ -1,16 +0,0 @@ -error: TS2322 [ERROR]: Type 'number' is not assignable to type 'string'. -const value1: string = myImport1.getValue(); - ~~~~~~ - at file:///[WILDCARD]/npm/node_modules_import/main.ts:9:7 - -TS2322 [ERROR]: Type 'number' is not assignable to type 'string'. -const value2: string = myImport2.getValue(); - ~~~~~~ - at file:///[WILDCARD]/npm/node_modules_import/main.ts:10:7 - -TS2322 [ERROR]: Type 'number' is not assignable to type 'string'. -const value3: string = myImport3.getValue(); - ~~~~~~ - at file:///[WILDCARD]/npm/node_modules_import/main.ts:11:7 - -Found 3 errors. diff --git a/cli/tests/testdata/npm/node_modules_import/package.json b/cli/tests/testdata/npm/node_modules_import/package.json deleted file mode 100644 index ed77298e0..000000000 --- a/cli/tests/testdata/npm/node_modules_import/package.json +++ /dev/null @@ -1,5 +0,0 @@ -{ - "dependencies": { - "@denotest/esm-basic": "^1" - } -} -- cgit v1.2.3