Skip to content

doc(proposal): un/break --test #13

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

Open
JakobJingleheimer wants to merge 4 commits into
base: main
Choose a base branch
from
Open

doc(proposal): un/break --test #13

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
86af8d6
doc(proposal): un/break `--test`
JakobJingleheimer Feb 1, 2026
f34de60
add "option 3"
JakobJingleheimer Feb 9, 2026
fea8e41
remove `entrypoint` from places that don't make sense, add item for i…
JakobJingleheimer Feb 12, 2026
826cd96
remove erroneous code sample with `--watch` occurring before `--test`
JakobJingleheimer Feb 24, 2026
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
remove entrypoint from places that don't make sense, add item for i… 
…ncluding defaults plus extras
  • Loading branch information
@JakobJingleheimer
JakobJingleheimer committed Feb 12, 2026
commit fea8e41c59578005edcb85b8552b95022d9f6fae
86 changes: 39 additions & 47 deletions proposals/proposal - unbreak test cli arg.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,8 @@ This is similar to another existing feature with which we want to improve intero
* enables `watch` mode
* optionally accepts 1 value to override the entrypoint
Comment thread
Ethan-Arrowood marked this conversation as resolved.

`--watch-path` sets additional paths to include as well as the entrypoint.

```sh
node
--watch
Expand Down Expand Up @@ -76,10 +78,7 @@ node
</td>
<td>

`**/*.test.{cjs,cts,mjs,mts,js,ts}`
etc

https://nodejs.org/api/test.html#running-tests-from-the-command-line
`**/*.test.{cjs,cts,mjs,mts,js,ts}`, [etc](https://nodejs.org/api/test.html#running-tests-from-the-command-line)

</td>
</tr>
Expand All @@ -101,6 +100,29 @@ node

`./src/foo/*.test.js`

</td>
</tr>
</tr>

<tr>
<td>
Test & watch modes enabled, using default & an additional path
</td>
<td>

```sh
node
--test
--test ./src/foo/*.test.js
--watch
```

</td>
<td>

`./src/foo/*.test.js`
`**/*.test.{cjs,cts,mjs,mts,js,ts}`, [etc](https://nodejs.org/api/test.html#running-tests-from-the-command-line)

</td>
</tr>

Expand Down Expand Up @@ -156,31 +178,24 @@ node --watch
```sh title='override derived entrypoint'
node --watch ./src/not-pjson-main.js
```
```sh title='package.json "main" + an additional path'
node --watch ./src/**/*.js
```
```sh title='explicit entrypoint + additional paths'
```sh title='package.json "main" + additional paths'
node
--watch ./assets/**
--watch
--watch ./src/**/*.js
./src/not-pjson-main.js
--watch ./src/not-pjson-main.js
```

An explicit entrypoint, like all node commands, must come last and must be a relative or absolute path (not a glob/pattern).

`test` would then work the same way:

```sh title='reads "main" etc from package.json'
```sh title='use default'
node --test
```
```sh title='package.json "main" + an additional path'
node --test ./src/**/*.test.js
```sh title='override default with one or more paths'
node --test ./src/bar/*.test.js
```
```sh title='explicit entrypoint + an additional path'
```sh title='use default with one or more additional paths'
node
--test
--test ./src/foo/*.test.js
--test ./src/bar/*.test.js
./src/not-pjson-main.js
```

#### All together
Expand All @@ -189,51 +204,28 @@ node

All examples are sequence-independent (all within the same heading behave the same).

##### Watch + test paths & main entrypoint:
##### Test & watch mode enabled:

```sh
node
--test ./src/foo/*.test.js
--test ./src/bar/*.test.js
--watch
./src/not-pjson-main.js
```
```sh
node
--watch
--test ./src/foo/*.test.js
--test ./src/bar/*.test.js
./src/not-pjson-main.js
```

##### Without an entrypoint (just paths):
##### Test & watch mode enabled with _additional_ watch path:

```sh
node
--test ./src/foo/*.test.js
--test ./src/bar/*.test.js
--watch
```
```sh
node
--watch
--test ./src/foo/*.test.js
--test ./src/bar/*.test.js
```

##### _Additional_ watch paths

```sh
node
--test ./src/foo/*.test.js
--test ./src/bar/*.test.js
--watch ./src/**/*.js
```
```sh
node
--watch ./src/**/*.js
--test ./src/foo/*.test.js
--test ./src/bar/*.test.js
--watch ./src/something-else.js
```

### Option: Apply `--watch`'s current design to `--test`
Expand Down Expand Up @@ -306,11 +298,11 @@ node
</td>
<td>

`./src/**/*.test.(c|m)?(j|t)s`
`**/*.test.{cjs,cts,mjs,mts,js,ts}`, [etc](https://nodejs.org/api/test.html#running-tests-from-the-command-line)
~`./src/foo/*.test.js`~
~`./src/bar/*.test.js`~

which reduces to only the default: `./src/**/*.test.(c|m)?(j|t)s`
which reduces to only the default

</td>
</tr>
Expand Down