Commit 19cfa31
test_runner: close and flush destinations on forced exit
This commit updates the test runner to explicitly close and flush
all destination file streams when the --test-force-exit flag is
used.
Fixes: #54327
PR-URL: #55099
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Jake Yuesong Li <jake.yuesong@gmail.com>1 parent 9aeba48 commit 19cfa31Copy full SHA for 19cfa31
File tree
Expand file treeCollapse file tree
3 files changed
+74
-4
lines changedOpen diff view settings
Filter options
- lib/internal/test_runner
- test/parallel
Expand file treeCollapse file tree
3 files changed
+74
-4
lines changedOpen diff view settings
Collapse file
lib/internal/test_runner/test.js
Copy file name to clipboardExpand all lines: lib/internal/test_runner/test.js+21-3Lines changed: 21 additions & 3 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
14 | 14 | |
15 | 15 | |
16 | 16 | |
| 17 | + |
17 | 18 | |
18 | 19 | |
19 | 20 | |
20 | 21 | |
21 | 22 | |
22 | 23 | |
| 24 | + |
23 | 25 | |
24 | 26 | |
25 | 27 | |
| ||
46 | 48 | |
47 | 49 | |
48 | 50 | |
| 51 | + |
49 | 52 | |
50 | 53 | |
51 | 54 | |
| ||
973 | 976 | |
974 | 977 | |
975 | 978 | |
976 | | - |
977 | | - |
978 | | - |
| 979 | + |
| 980 | + |
| 981 | + |
| 982 | + |
| 983 | + |
| 984 | + |
| 985 | + |
| 986 | + |
| 987 | + |
| 988 | + |
| 989 | + |
| 990 | + |
| 991 | + |
| 992 | + |
| 993 | + |
| 994 | + |
979 | 995 | |
| 996 | + |
| 997 | + |
980 | 998 | |
981 | 999 | |
982 | 1000 | |
|
Collapse file
lib/internal/test_runner/utils.js
Copy file name to clipboardExpand all lines: lib/internal/test_runner/utils.js+4-1Lines changed: 4 additions & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
144 | 144 | |
145 | 145 | |
146 | 146 | |
147 | | - |
| 147 | + |
| 148 | + |
148 | 149 | |
149 | 150 | |
150 | 151 | |
| ||
301 | 302 | |
302 | 303 | |
303 | 304 | |
| 305 | + |
| 306 | + |
304 | 307 | |
305 | 308 | |
306 | 309 | |
|
Collapse file
test/parallel/test-runner-force-exit-flush.js
Copy file name to clipboard+49Lines changed: 49 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
| 1 | + |
| 2 | + |
| 3 | + |
| 4 | + |
| 5 | + |
| 6 | + |
| 7 | + |
| 8 | + |
| 9 | + |
| 10 | + |
| 11 | + |
| 12 | + |
| 13 | + |
| 14 | + |
| 15 | + |
| 16 | + |
| 17 | + |
| 18 | + |
| 19 | + |
| 20 | + |
| 21 | + |
| 22 | + |
| 23 | + |
| 24 | + |
| 25 | + |
| 26 | + |
| 27 | + |
| 28 | + |
| 29 | + |
| 30 | + |
| 31 | + |
| 32 | + |
| 33 | + |
| 34 | + |
| 35 | + |
| 36 | + |
| 37 | + |
| 38 | + |
| 39 | + |
| 40 | + |
| 41 | + |
| 42 | + |
| 43 | + |
| 44 | + |
| 45 | + |
| 46 | + |
| 47 | + |
| 48 | + |
| 49 | + |
0 commit comments