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/check/jsx_not_checked/main.jsx | 21 +++++++++++++++++++++ tests/testdata/check/jsx_not_checked/main.out | 13 +++++++++++++ tests/testdata/check/jsx_not_checked/other.ts | 5 +++++ 3 files changed, 39 insertions(+) create mode 100644 tests/testdata/check/jsx_not_checked/main.jsx create mode 100644 tests/testdata/check/jsx_not_checked/main.out create mode 100644 tests/testdata/check/jsx_not_checked/other.ts (limited to 'tests/testdata/check/jsx_not_checked') diff --git a/tests/testdata/check/jsx_not_checked/main.jsx b/tests/testdata/check/jsx_not_checked/main.jsx new file mode 100644 index 000000000..8de05f9f7 --- /dev/null +++ b/tests/testdata/check/jsx_not_checked/main.jsx @@ -0,0 +1,21 @@ +// should not error about jsx-runtime not being found in types here +/** @jsxImportSource npm:react@18.2.0 */ + +import "./other.ts"; + +export default ( + <> +

Hello world

+

This is a JSX page

+ +); + +/** + * @param {number} a + * @param {number} b + */ +function add(a, b) { + return a + b; +} + +console.log(add("1", "2")); diff --git a/tests/testdata/check/jsx_not_checked/main.out b/tests/testdata/check/jsx_not_checked/main.out new file mode 100644 index 000000000..82c1c2358 --- /dev/null +++ b/tests/testdata/check/jsx_not_checked/main.out @@ -0,0 +1,13 @@ +[UNORDERED_START] +Download http://localhost:4545/npm/registry/react +Download http://localhost:4545/npm/registry/loose-envify +Download http://localhost:4545/npm/registry/js-tokens +Download http://localhost:4545/npm/registry/react/react-18.2.0.tgz +Download http://localhost:4545/npm/registry/loose-envify/loose-envify-1.4.0.tgz +Download http://localhost:4545/npm/registry/js-tokens/js-tokens-4.0.0.tgz +[UNORDERED_END] +Check file:///[WILDCARD]/jsx_not_checked/main.jsx +error: TS2345 [ERROR]: Argument of type 'string' is not assignable to parameter of type 'number'. +console.log(add("1", "2")); + ~~~ + at file:///[WILDCARD]/other.ts:5:17 diff --git a/tests/testdata/check/jsx_not_checked/other.ts b/tests/testdata/check/jsx_not_checked/other.ts new file mode 100644 index 000000000..47995cb0f --- /dev/null +++ b/tests/testdata/check/jsx_not_checked/other.ts @@ -0,0 +1,5 @@ +function add(a: number, b: number) { + return a + b; +} + +console.log(add("1", "2")); -- cgit v1.2.3