Commit e08a98f
test: always suffix tmpdir
This makes temp dir names consistent whether we run in stand-alone mode,
via `test.py` in single process, or in multi-process.
PR-URL: #28035
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>1 parent f336236 commit e08a98f
1 file changed
Lines changed: 1 addition & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -89,11 +89,7 @@ const testRoot = process.env.NODE_TEST_DIR ? | |||
| 89 | 89 | ||
| 90 | 90 | // Using a `.` prefixed name, which is the convention for "hidden" on POSIX, | |
| 91 | 91 | // gets tools to ignore it by default or by simple rules, especially eslint. | |
| 92 | - let tmpdirName = '.tmp'; | ||
| 93 | - if (process.env.TEST_THREAD_ID) { | ||
| 94 | - tmpdirName += `.${process.env.TEST_THREAD_ID}`; | ||
| 95 | - } | ||
| 96 | - | ||
| 92 | + const tmpdirName = '.tmp.' + (process.env.TEST_THREAD_ID || '0'); | ||
| 97 | 93 | const tmpPath = path.join(testRoot, tmpdirName); | |
| 98 | 94 | ||
| 99 | 95 | function refresh(opts = {}) { | |
0 commit comments