diff options
author | Matt Mastracci <matthew@mastracci.com> | 2024-02-10 13:22:13 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-02-10 20:22:13 +0000 |
commit | f5e46c9bf2f50d66a953fa133161fc829cecff06 (patch) | |
tree | 8faf2f5831c1c7b11d842cd9908d141082c869a5 /cli/tests/node_compat/test/parallel/test-fs-read-stream-concurrent-reads.js | |
parent | d2477f780630a812bfd65e3987b70c0d309385bb (diff) |
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.
Diffstat (limited to 'cli/tests/node_compat/test/parallel/test-fs-read-stream-concurrent-reads.js')
-rw-r--r-- | cli/tests/node_compat/test/parallel/test-fs-read-stream-concurrent-reads.js | 54 |
1 files changed, 0 insertions, 54 deletions
diff --git a/cli/tests/node_compat/test/parallel/test-fs-read-stream-concurrent-reads.js b/cli/tests/node_compat/test/parallel/test-fs-read-stream-concurrent-reads.js deleted file mode 100644 index ac52d66a8..000000000 --- a/cli/tests/node_compat/test/parallel/test-fs-read-stream-concurrent-reads.js +++ /dev/null @@ -1,54 +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 fixtures = require('../common/fixtures'); -const assert = require('assert'); -const fs = require('fs'); - -// Test that concurrent file read streams don’t interfere with each other’s -// contents, and that the chunks generated by the reads only retain a -// 'reasonable' amount of memory. - -// Refs: https://github.com/nodejs/node/issues/21967 - -const filename = fixtures.path('loop.js'); // Some small non-homogeneous file. -const content = fs.readFileSync(filename); - -const N = 2000; -let started = 0; -let done = 0; - -const arrayBuffers = new Set(); - -function startRead() { - ++started; - const chunks = []; - fs.createReadStream(filename) - .on('data', (chunk) => { - chunks.push(chunk); - arrayBuffers.add(chunk.buffer); - }) - .on('end', common.mustCall(() => { - if (started < N) - startRead(); - assert.deepStrictEqual(Buffer.concat(chunks), content); - if (++done === N) { - const retainedMemory = - [...arrayBuffers].map((ab) => ab.byteLength).reduce((a, b) => a + b); - assert(retainedMemory / (N * content.length) <= 3, - `Retaining ${retainedMemory} bytes in ABs for ${N} ` + - `chunks of size ${content.length}`); - } - })); -} - -// Don’t start the reads all at once – that way we would have to allocate -// a large amount of memory upfront. -for (let i = 0; i < 6; ++i) - startRead(); |