Commit 02ebde3
test: use common/tmpdir in watch-mode ipc test
Using `path.join(os.tmpdir(), 'file')` on a multi-user system can lead
to file collisions where the file already exists but is owned by a
different user and cannot be removed or overwritten. Other tests in
`parallel/test-watch-mode-files_watcher` use `common/tmpdir` instead
so switch to that for consistency.
PR-URL: #45211
Refs: #44366
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>1 parent 49be13c commit 02ebde3Copy full SHA for 02ebde3
File tree
Expand file treeCollapse file tree
2 files changed
+3
-4
lines changedOpen diff view settings
Filter options
- test
- fixtures/watch-mode
- parallel
Expand file treeCollapse file tree
2 files changed
+3
-4
lines changedOpen diff view settings
Collapse file
test/fixtures/watch-mode/ipc.js
Copy file name to clipboardExpand all lines: test/fixtures/watch-mode/ipc.js+2-2Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
1 | 1 | |
2 | 2 | |
3 | | - |
4 | 3 | |
| 4 | + |
5 | 5 | |
6 | | - |
| 6 | + |
7 | 7 | |
8 | 8 | |
9 | 9 | |
|
Collapse file
test/parallel/test-watch-mode-files_watcher.mjs
Copy file name to clipboardExpand all lines: test/parallel/test-watch-mode-files_watcher.mjs+1-2Lines changed: 1 addition & 2 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
5 | 5 | |
6 | 6 | |
7 | 7 | |
8 | | - |
9 | 8 | |
10 | 9 | |
11 | 10 | |
| ||
153 | 152 | |
154 | 153 | |
155 | 154 | |
156 | | - |
| 155 | + |
157 | 156 | |
158 | 157 | |
159 | 158 | |
|
0 commit comments