[node:test] Infinite loop occurs when `files` is empty

[koh110]

Version

v20.4.0

Platform

Darwin MacBook-Pro.local 22.5.0 Darwin Kernel Version 22.5.0: Thu Jun 8 22:22:20 PDT 2023; root:xnu-8796.121.3~7/RELEASE_ARM64_T6000 arm64

Subsystem

No response

What steps will reproduce the bug?

This code occurs infinite loop.

import { run } from 'node:test' 
import { tap } from 'node:test/reporters';

const stream = run({
  // files: undefined
}).compose(tap);

stream.pipe(process.stdout);

How often does it reproduce? Is there a required condition?

everytime

What is the expected behavior? Why is that the expected behavior?

Default is written astest runner execution model. But it does not work.
https://nodejs.org/api/test.html#runoptions

files: An array containing the list of files to run. Default matching files from test runner execution model

It seems that tests that are run during node --test should be run when files is empty.

What do you see instead?

https://github.com/koh110/minimum-nodejs-test

$ node test-run-null-files.mjs
TAP version 13
# TAP version 13
# \# TAP version 13
# \# \\\# TAP version 13
# \# \\\# \\\\\\\# TAP version 13
# \# \\\# \\\\\\\# \\\\\\\\\\\\\\\# TAP version 13
# \# \\\# \\\\\\\# \\\\\\\\\\\\\\\# \\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\# TAP version 13

Additional information

createTestFileList gets its own file path.
It seems that it have to exclude own file path when executing in run.

const hasUserSuppliedPattern = process.argv.length > 1;
const patterns = hasUserSuppliedPattern ? ArrayPrototypeSlice(process.argv, 1) : [kDefaultPattern];
console.log('createTestFileList:', hasUserSuppliedPattern, patterns);

node/lib/internal/test_runner/runner.js

Lines 90 to 92 in 9988130

	const cwd = process.cwd();
	const hasUserSuppliedPattern = process.argv.length > 1;
	const patterns = hasUserSuppliedPattern ? ArrayPrototypeSlice(process.argv, 1) : [kDefaultPattern];

let testFiles = files ?? createTestFileList();

if (shard) {
  testFiles = ArrayPrototypeFilter(testFiles, (_, index) => index % shard.total === shard.index - 1);
}
console.log('testFiles: ', testFiles);

node/lib/internal/test_runner/runner.js

Lines 473 to 477 in 9988130

	let testFiles = files ?? createTestFileList();
	if (shard) {
	testFiles = ArrayPrototypeFilter(testFiles, (_, index) => index % shard.total === shard.index - 1);
	}

$ ~/dev/node/out/Release/node ./test-run-null-files.mjs
createTestFileList: true [ '/xxx/minimum-nodejs-test/test-run-null-files.mjs' ]
testFiles:  [ '/xxx/minimum-nodejs-test/test-run-null-files.mjs' ]
TAP version 13
# createTestFileList: true [ '/xxx/minimum-nodejs-test/test-run-null-files.mjs' ]
# testFiles:  [ '/xxx/minimum-nodejs-test/test-run-null-files.mjs' ]
# TAP version 13

[MoLow]

I am not sure I would consider this a bug, you should either specify files or name the root file in some way that the default execution model won't detect it as a test file.
and, due to the fact the the default will change to be a glob pattern starting node 21 - I don't think it should be excluded in any way

[koh110]

Does it mean that the files should be specified? Or does it mean that it should not be excluded own file path?

I don't think it should be excluded in any way

Would you prefer to modify the document in that case?

Default is written astest runner execution model

[MoLow]

the file calling run should not be excluded

[koh110]

So, I think Default matching in the document should be removed. What do you think about it?

[MoLow]

why? it is still correct

[koh110]

I think Default is the initial value that works that way if not specified.
For example, the initial values for concurrency and inspectPort, timeout are described in the document.
However, files will not work if it is excluded.
I think this is different compared to other Default.

So I think it should state that it cannot be omitted instead of Default.

[koh110]

How do I call the default file pattern like node --test for run?

[MoLow]

run({file: undefined}) or just run({}) will use the default behavior

[koh110]

When I run run({file: undefined}) or run({}), it's occurs Infinite loop. I want to solve it.

So, I think options.files has bug or Default in doc is not correct, when i create this issue.

Could you please exec this code?

$ node -v
v20.4.0

$ tree
.
└── run.js

$ cat run.js
require('node:test').run({ files: undefined }).compose(require('node:test/reporters').tap).pipe(process.stdout)

# this code occurs infinite loop
$ node run.js

run.js is included in process.argv in createFiles if i set files: undefined. It seems to refer to itself.
https://github.com/nodejs/node/blob/99881304c47138e09a06e6b2bd876cfdf840047e/lib/internal/test_runner/runner.js#L89C1-L93

This happens with any file name.
My first sample code with a file name starting with test- was bad. sorry.

[koshic]

run({file: undefined}) or just run({}) will use the default behavior

Is that mean that 'node run.js' (which files: undefined) is equal to 'node --test run.js', as @koh110 explained? I don't think that such 'expected' behaviour is really expected. We should have note in docs, or some asserts in test-runner code to brake the loop.

[koh110]

@MoLow Hi! Could you please help me with what is wrong with this sample code?

@koshic Thanks for the supplement!

[MoLow]

@koh110 if you experience an infinate loop - that means the file called by run has a name that is detected as a test file (as described here - https://nodejs.org/api/test.html#test-runner-execution-model it probably has the word test in its name.
you can avoid that by changing the file name or not using the default files argument

[koh110]

@MoLow It occurs with any file name. Could you please run this sample?
#48823 (comment)
This sample file is named run.js and it is located in the root directory.
I think this is not part of the pattern.