Commit ca51ff8
test: fix recursive rm test to actually use tmpdir
Previously folders were created in the root of the project and would
therefore stay there in case of errors. This changes the test to create
all directories in the temp directory (`tmpdir.path`).
PR-URL: #31250
Reviewed-By: Bryan English <bryan@bryanenglish.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>1 parent 0b88c3d commit ca51ff8Copy full SHA for ca51ff8
File tree
Expand file treeCollapse file tree
1 file changed
+9
-10
lines changedOpen diff view settings
Filter options
- test/parallel
Expand file treeCollapse file tree
1 file changed
+9
-10
lines changedOpen diff view settings
Collapse file
test/parallel/test-fs-rmdir-recursive.js
Copy file name to clipboardExpand all lines: test/parallel/test-fs-rmdir-recursive.js+9-10Lines changed: 9 additions & 10 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
6 | 6 | |
7 | 7 | |
8 | 8 | |
9 | | - |
10 | 9 | |
11 | 10 | |
12 | 11 | |
| 12 | + |
| 13 | + |
| 14 | + |
| 15 | + |
13 | 16 | |
14 | 17 | |
15 | 18 | |
| ||
89 | 92 | |
90 | 93 | |
91 | 94 | |
92 | | - |
| 95 | + |
93 | 96 | |
94 | 97 | |
95 | 98 | |
96 | 99 | |
97 | | - |
98 | | - |
| 100 | + |
99 | 101 | |
100 | 102 | |
101 | 103 | |
102 | 104 | |
103 | | - |
104 | | - |
| 105 | + |
105 | 106 | |
106 | 107 | |
107 | 108 | |
108 | 109 | |
109 | 110 | |
110 | 111 | |
111 | | - |
112 | | - |
| 112 | + |
113 | 113 | |
114 | 114 | |
115 | 115 | |
| ||
132 | 132 | |
133 | 133 | |
134 | 134 | |
135 | | - |
136 | | - |
| 135 | + |
137 | 136 | |
138 | 137 | |
139 | 138 | |
|
0 commit comments