Skip to content

test-runner: run tests as soon as find them #48584

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
rluvaton wants to merge 20 commits into from
Closed

test-runner: run tests as soon as find them #48584

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
1d80a2b
test_runner: run tests as soon as find them
rluvaton Jun 28, 2023
20e7ad9
test-runner: fix tests as files not run in consistent order...
rluvaton Jun 28, 2023
299cd6d
test: use hard-coded pattern
rluvaton Jun 28, 2023
13f003e
Update lib/internal/test_runner/runner.js
rluvaton Jun 28, 2023
a3f7575
Update lib/internal/test_runner/runner.js
rluvaton Jun 28, 2023
00259ee
test_runner: check node stream
rluvaton Jun 28, 2023
1164a4a
Merge remote-tracking branch 'origin/run-tests-as-soon-as-get-a-file'…
rluvaton Jun 28, 2023
afd0658
test_runner: lint
rluvaton Jun 28, 2023
cebf726
test_runner: add comment
rluvaton Jun 28, 2023
1d68c63
test_runner: fix concurrent test
rluvaton Jun 28, 2023
0ea199f
test_runner: format
rluvaton Jun 28, 2023
98a9614
test_runner: improve error message in test
rluvaton Jun 28, 2023
004e915
test_runner: fix exit code test
rluvaton Jun 28, 2023
0cb655f
test_runner: revert termination handler process.exit(1)
rluvaton Jun 28, 2023
2062f53
test_runner: termination handler process.exit(1)
rluvaton Jun 28, 2023
517bdbc
test_runner: remove todo
rluvaton Jun 28, 2023
5d34f67
test_runner: add signal support for glob
rluvaton Jun 28, 2023
4461789
test_runner: simplify runner
rluvaton Jun 28, 2023
773f3cf
test_runner: remove comment
rluvaton Jun 28, 2023
dd09109
test_runner: remove todo
rluvaton Jun 28, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
test_runner: check node stream
  • Loading branch information
@rluvaton
rluvaton committed Jun 28, 2023
commit 00259ee3cf0b67417b3c518f5b33bd2d006456b8
22 changes: 11 additions & 11 deletions lib/internal/test_runner/runner.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ const {
const { spawn } = require('child_process');
const { Readable } = require('stream');
const { finished } = require('internal/streams/end-of-stream');
const { isNodeStream } = require('internal/streams/utils');
const { DefaultDeserializer, DefaultSerializer } = require('v8');
// TODO(aduh95): switch to internal/readline/interface when backporting to Node.js 16.x is no longer a concern.
const { createInterface } = require('readline');
Expand Down Expand Up @@ -478,21 +479,20 @@ function run(options) {
const runFiles = () => {
root.harness.bootstrapComplete = true;

if (ArrayIsArray(testFiles)) {
testFiles = PromiseResolve(testFiles);
if(isNodeStream(testFiles)) {
return testFiles
.map(runSingleFile, { signal, concurrency })
.toArray();
}

// TODO - find better way to check if promise
if (typeof testFiles.then === 'function') {
return PromisePrototypeThen(
testFiles,
(testFilesArray) => SafePromiseAllSettledReturnVoid(testFilesArray, runSingleFile),
);
if (ArrayIsArray(testFiles)) {
testFiles = PromiseResolve(testFiles);
}

return testFiles
.map(runSingleFile, { signal, concurrency })
.toArray();
return PromisePrototypeThen(
testFiles,
(testFilesArray) => SafePromiseAllSettledReturnVoid(testFilesArray, runSingleFile),
);
};

PromisePrototypeThen(PromisePrototypeThen(PromiseResolve(setup?.(root)), runFiles), postRun);
Expand Down