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. --- tests/testdata/npm/node_modules_import/main.out | 3 +++ tests/testdata/npm/node_modules_import/main.ts | 16 ++++++++++++++++ tests/testdata/npm/node_modules_import/main_check.out | 16 ++++++++++++++++ tests/testdata/npm/node_modules_import/package.json | 5 +++++ 4 files changed, 40 insertions(+) create mode 100644 tests/testdata/npm/node_modules_import/main.out create mode 100644 tests/testdata/npm/node_modules_import/main.ts create mode 100644 tests/testdata/npm/node_modules_import/main_check.out create mode 100644 tests/testdata/npm/node_modules_import/package.json (limited to 'tests/testdata/npm/node_modules_import') diff --git a/tests/testdata/npm/node_modules_import/main.out b/tests/testdata/npm/node_modules_import/main.out new file mode 100644 index 000000000..083edaac2 --- /dev/null +++ b/tests/testdata/npm/node_modules_import/main.out @@ -0,0 +1,3 @@ +2 +2 +2 diff --git a/tests/testdata/npm/node_modules_import/main.ts b/tests/testdata/npm/node_modules_import/main.ts new file mode 100644 index 000000000..848ca0f81 --- /dev/null +++ b/tests/testdata/npm/node_modules_import/main.ts @@ -0,0 +1,16 @@ +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/tests/testdata/npm/node_modules_import/main_check.out b/tests/testdata/npm/node_modules_import/main_check.out new file mode 100644 index 000000000..cf7cc110d --- /dev/null +++ b/tests/testdata/npm/node_modules_import/main_check.out @@ -0,0 +1,16 @@ +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/tests/testdata/npm/node_modules_import/package.json b/tests/testdata/npm/node_modules_import/package.json new file mode 100644 index 000000000..ed77298e0 --- /dev/null +++ b/tests/testdata/npm/node_modules_import/package.json @@ -0,0 +1,5 @@ +{ + "dependencies": { + "@denotest/esm-basic": "^1" + } +} -- cgit v1.2.3