From f5e46c9bf2f50d66a953fa133161fc829cecff06 Mon Sep 17 00:00:00 2001
From: Matt Mastracci <matthew@mastracci.com>
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-querystring-escape.js       | 48 ----------------------
 1 file changed, 48 deletions(-)
 delete mode 100644 cli/tests/node_compat/test/parallel/test-querystring-escape.js

(limited to 'cli/tests/node_compat/test/parallel/test-querystring-escape.js')

diff --git a/cli/tests/node_compat/test/parallel/test-querystring-escape.js b/cli/tests/node_compat/test/parallel/test-querystring-escape.js
deleted file mode 100644
index f4f635af1..000000000
--- a/cli/tests/node_compat/test/parallel/test-querystring-escape.js
+++ /dev/null
@@ -1,48 +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';
-require('../common');
-const assert = require('assert');
-
-const qs = require('querystring');
-
-assert.strictEqual(qs.escape(5), '5');
-assert.strictEqual(qs.escape('test'), 'test');
-assert.strictEqual(qs.escape({}), '%5Bobject%20Object%5D');
-assert.strictEqual(qs.escape([5, 10]), '5%2C10');
-assert.strictEqual(qs.escape('Ŋōđĕ'), '%C5%8A%C5%8D%C4%91%C4%95');
-assert.strictEqual(qs.escape('testŊōđĕ'), 'test%C5%8A%C5%8D%C4%91%C4%95');
-assert.strictEqual(qs.escape(`${String.fromCharCode(0xD800 + 1)}test`),
-                   '%F0%90%91%B4est');
-
-assert.throws(
-  () => qs.escape(String.fromCharCode(0xD800 + 1)),
-  {
-    code: 'ERR_INVALID_URI',
-    name: 'URIError',
-    message: 'URI malformed'
-  }
-);
-
-// Using toString for objects
-assert.strictEqual(
-  qs.escape({ test: 5, toString: () => 'test', valueOf: () => 10 }),
-  'test'
-);
-
-// `toString` is not callable, must throw an error.
-// Error message will vary between different JavaScript engines, so only check
-// that it is a `TypeError`.
-assert.throws(() => qs.escape({ toString: 5 }), TypeError);
-
-// Should use valueOf instead of non-callable toString.
-assert.strictEqual(qs.escape({ toString: 5, valueOf: () => 'test' }), 'test');
-
-// Error message will vary between different JavaScript engines, so only check
-// that it is a `TypeError`.
-assert.throws(() => qs.escape(Symbol('test')), TypeError);
-- 
cgit v1.2.3