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. --- .../test/parallel/test-url-parse-invalid-input.js | 83 ---------------------- 1 file changed, 83 deletions(-) delete mode 100644 cli/tests/node_compat/test/parallel/test-url-parse-invalid-input.js (limited to 'cli/tests/node_compat/test/parallel/test-url-parse-invalid-input.js') diff --git a/cli/tests/node_compat/test/parallel/test-url-parse-invalid-input.js b/cli/tests/node_compat/test/parallel/test-url-parse-invalid-input.js deleted file mode 100644 index 345e8d338..000000000 --- a/cli/tests/node_compat/test/parallel/test-url-parse-invalid-input.js +++ /dev/null @@ -1,83 +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'); -const url = require('url'); - -// https://github.com/joyent/node/issues/568 -[ - [undefined, 'undefined'], - [null, 'object'], - [true, 'boolean'], - [false, 'boolean'], - [0.0, 'number'], - [0, 'number'], - [[], 'object'], - [{}, 'object'], - [() => {}, 'function'], - [Symbol('foo'), 'symbol'], -].forEach(([val, type]) => { - assert.throws(() => { - url.parse(val); - }, { - code: 'ERR_INVALID_ARG_TYPE', - name: 'TypeError', - message: 'The "url" argument must be of type string.' + - common.invalidArgTypeHelper(val) - }); -}); - -assert.throws(() => { url.parse('http://%E0%A4%A@fail'); }, - (e) => { - // The error should be a URIError. - if (!(e instanceof URIError)) - return false; - - // The error should be from the JS engine and not from Node.js. - // JS engine errors do not have the `code` property. - return e.code === undefined; - }); - -assert.throws(() => { url.parse('http://[127.0.0.1\x00c8763]:8000/'); }, - { code: 'ERR_INVALID_URL', input: 'http://[127.0.0.1\x00c8763]:8000/' } -); - -if (common.hasIntl) { - // An array of Unicode code points whose Unicode NFKD contains a "bad - // character". - const badIDNA = (() => { - const BAD_CHARS = '#%/:?@[\\]^|'; - const out = []; - for (let i = 0x80; i < 0x110000; i++) { - const cp = String.fromCodePoint(i); - for (const badChar of BAD_CHARS) { - if (cp.normalize('NFKD').includes(badChar)) { - out.push(cp); - } - } - } - return out; - })(); - - // The generation logic above should at a minimum produce these two - // characters. - assert(badIDNA.includes('℀')); - assert(badIDNA.includes('@')); - - for (const badCodePoint of badIDNA) { - const badURL = `http://fail${badCodePoint}fail.com/`; - assert.throws(() => { url.parse(badURL); }, - (e) => e.code === 'ERR_INVALID_URL', - `parsing ${badURL}`); - } - - assert.throws(() => { url.parse('http://\u00AD/bad.com/'); }, - (e) => e.code === 'ERR_INVALID_URL', - 'parsing http://\u00AD/bad.com/'); -} -- cgit v1.2.3