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. --- .../parallel/test-process-env-allowed-flags.js | 109 --------------------- 1 file changed, 109 deletions(-) delete mode 100644 cli/tests/node_compat/test/parallel/test-process-env-allowed-flags.js (limited to 'cli/tests/node_compat/test/parallel/test-process-env-allowed-flags.js') diff --git a/cli/tests/node_compat/test/parallel/test-process-env-allowed-flags.js b/cli/tests/node_compat/test/parallel/test-process-env-allowed-flags.js deleted file mode 100644 index 1b2c96e68..000000000 --- a/cli/tests/node_compat/test/parallel/test-process-env-allowed-flags.js +++ /dev/null @@ -1,109 +0,0 @@ -// deno-fmt-ignore-file -// deno-lint-ignore-file - -// Copyright Joyent and Node contributors. All rights reserved. MIT license. -// Taken from Node 18.12.1 -// This file is automatically generated by `tools/node_compat/setup.ts`. Do not modify this file manually. - -'use strict'; - -const common = require('../common'); -const assert = require('assert'); - -// Assert legit flags are allowed, and bogus flags are disallowed -{ - const goodFlags = [ - '--perf_basic_prof', - '--perf-basic-prof', - 'perf-basic-prof', - '--perf_basic-prof', - 'perf_basic-prof', - 'perf_basic_prof', - '-r', - 'r', - '--stack-trace-limit=100', - '--stack-trace-limit=-=xX_nodejs_Xx=-', - ].concat(process.features.inspector ? [ - '--inspect-brk', - 'inspect-brk', - '--inspect_brk', - ] : []); - - const badFlags = [ - 'INSPECT-BRK', - '--INSPECT-BRK', - '--r', - '-R', - '---inspect-brk', - '--cheeseburgers', - ]; - - goodFlags.forEach((flag) => { - assert.strictEqual( - process.allowedNodeEnvironmentFlags.has(flag), - true, - `flag should be in set: ${flag}` - ); - }); - - badFlags.forEach((flag) => { - assert.strictEqual( - process.allowedNodeEnvironmentFlags.has(flag), - false, - `flag should not be in set: ${flag}` - ); - }); -} - -// Assert all "canonical" flags begin with dash(es) -{ - process.allowedNodeEnvironmentFlags.forEach((flag) => { - assert.match(flag, /^--?[a-zA-Z0-9._-]+$/); - }); -} - -// Assert immutability of process.allowedNodeEnvironmentFlags -{ - assert.strictEqual(Object.isFrozen(process.allowedNodeEnvironmentFlags), - true); - - process.allowedNodeEnvironmentFlags.add('foo'); - assert.strictEqual(process.allowedNodeEnvironmentFlags.has('foo'), false); - Set.prototype.add.call(process.allowedNodeEnvironmentFlags, 'foo'); - assert.strictEqual(process.allowedNodeEnvironmentFlags.has('foo'), false); - - const thisArg = {}; - process.allowedNodeEnvironmentFlags.forEach( - common.mustCallAtLeast(function(flag, _, set) { - assert.notStrictEqual(flag, 'foo'); - assert.strictEqual(this, thisArg); - assert.strictEqual(set, process.allowedNodeEnvironmentFlags); - }), - thisArg - ); - - for (const flag of process.allowedNodeEnvironmentFlags.keys()) { - assert.notStrictEqual(flag, 'foo'); - } - for (const flag of process.allowedNodeEnvironmentFlags.values()) { - assert.notStrictEqual(flag, 'foo'); - } - for (const flag of process.allowedNodeEnvironmentFlags) { - assert.notStrictEqual(flag, 'foo'); - } - for (const [flag] of process.allowedNodeEnvironmentFlags.entries()) { - assert.notStrictEqual(flag, 'foo'); - } - - const size = process.allowedNodeEnvironmentFlags.size; - - process.allowedNodeEnvironmentFlags.clear(); - assert.strictEqual(process.allowedNodeEnvironmentFlags.size, size); - Set.prototype.clear.call(process.allowedNodeEnvironmentFlags); - assert.strictEqual(process.allowedNodeEnvironmentFlags.size, size); - - process.allowedNodeEnvironmentFlags.delete('-r'); - assert.strictEqual(process.allowedNodeEnvironmentFlags.size, size); - Set.prototype.delete.call(process.allowedNodeEnvironmentFlags, '-r'); - assert.strictEqual(process.allowedNodeEnvironmentFlags.size, size); -} -- cgit v1.2.3