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. --- .../node_modules/imports_exports/import_export.js | 6 ------ .../node_modules/imports_exports/import_polyfill.js | 3 --- .../commonjs/node_modules/imports_exports/package.json | 17 ----------------- .../node_modules/imports_exports/require_export.cjs | 6 ------ .../node_modules/imports_exports/require_polyfill.js | 3 --- 5 files changed, 35 deletions(-) delete mode 100644 cli/tests/testdata/commonjs/node_modules/imports_exports/import_export.js delete mode 100644 cli/tests/testdata/commonjs/node_modules/imports_exports/import_polyfill.js delete mode 100644 cli/tests/testdata/commonjs/node_modules/imports_exports/package.json delete mode 100644 cli/tests/testdata/commonjs/node_modules/imports_exports/require_export.cjs delete mode 100644 cli/tests/testdata/commonjs/node_modules/imports_exports/require_polyfill.js (limited to 'cli/tests/testdata/commonjs/node_modules/imports_exports') diff --git a/cli/tests/testdata/commonjs/node_modules/imports_exports/import_export.js b/cli/tests/testdata/commonjs/node_modules/imports_exports/import_export.js deleted file mode 100644 index 3ebd222ea..000000000 --- a/cli/tests/testdata/commonjs/node_modules/imports_exports/import_export.js +++ /dev/null @@ -1,6 +0,0 @@ -import dep from "#dep"; - -export default { - bar: "bar", - dep, -}; diff --git a/cli/tests/testdata/commonjs/node_modules/imports_exports/import_polyfill.js b/cli/tests/testdata/commonjs/node_modules/imports_exports/import_polyfill.js deleted file mode 100644 index 76716a3ef..000000000 --- a/cli/tests/testdata/commonjs/node_modules/imports_exports/import_polyfill.js +++ /dev/null @@ -1,3 +0,0 @@ -export default { - polyfill: "import", -}; diff --git a/cli/tests/testdata/commonjs/node_modules/imports_exports/package.json b/cli/tests/testdata/commonjs/node_modules/imports_exports/package.json deleted file mode 100644 index 5d26359db..000000000 --- a/cli/tests/testdata/commonjs/node_modules/imports_exports/package.json +++ /dev/null @@ -1,17 +0,0 @@ -{ - "version": "1.0.0", - "name": "imports_exports", - "main": "./require_export.cjs", - "imports": { - "#dep": { - "import": "./import_polyfill.js", - "require": "./require_polyfill.js" - } - }, - "exports": { - ".": { - "import": "./import_export.js", - "require": "./require_export.cjs" - } - } -} diff --git a/cli/tests/testdata/commonjs/node_modules/imports_exports/require_export.cjs b/cli/tests/testdata/commonjs/node_modules/imports_exports/require_export.cjs deleted file mode 100644 index 48923b8d8..000000000 --- a/cli/tests/testdata/commonjs/node_modules/imports_exports/require_export.cjs +++ /dev/null @@ -1,6 +0,0 @@ -const dep = require("#dep"); - -module.exports = { - foo: "foo", - dep, -}; diff --git a/cli/tests/testdata/commonjs/node_modules/imports_exports/require_polyfill.js b/cli/tests/testdata/commonjs/node_modules/imports_exports/require_polyfill.js deleted file mode 100644 index 1023fd65c..000000000 --- a/cli/tests/testdata/commonjs/node_modules/imports_exports/require_polyfill.js +++ /dev/null @@ -1,3 +0,0 @@ -module.exports = { - polyfill: "require", -}; -- cgit v1.2.3