Commit 863bfc4
test: redirect stderr EnvironmentWithNoESMLoader
This commit adds a suggestion to redirect stderr for
EnvironmentTest.EnvironmentWithNoESMLoader.
The motivation for this is that currently this tests prints the
following error (which is expected):
vm:module(0):1
globalThis.importResult = import("")
^
Error: Not supported
at vm:module(0):1:1
at SourceTextModule.evaluate (node:internal/vm/module:229:23)
at node:embedder_main_12:1:328
at processTicksAndRejections (node:internal/process/task_queues:93:5)
It might not be obvious which test caused this error just by looking at
the output above and it would be nice if it was not displayed.
PR-URL: #36548
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>1 parent d20235b commit 863bfc4Copy full SHA for 863bfc4
File tree
Expand file treeCollapse file tree
1 file changed
+29
-0
lines changedOpen diff view settings
Filter options
- test/cctest
Expand file treeCollapse file tree
1 file changed
+29
-0
lines changedOpen diff view settings
Collapse file
test/cctest/test_environment.cc
Copy file name to clipboardExpand all lines: test/cctest/test_environment.cc+29Lines changed: 29 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
5 | 5 | |
6 | 6 | |
7 | 7 | |
| 8 | + |
| 9 | + |
8 | 10 | |
9 | 11 | |
10 | 12 | |
| ||
66 | 68 | |
67 | 69 | |
68 | 70 | |
| 71 | + |
| 72 | + |
| 73 | + |
| 74 | + |
| 75 | + |
| 76 | + |
| 77 | + |
| 78 | + |
| 79 | + |
| 80 | + |
| 81 | + |
| 82 | + |
| 83 | + |
| 84 | + |
| 85 | + |
| 86 | + |
| 87 | + |
| 88 | + |
| 89 | + |
| 90 | + |
| 91 | + |
| 92 | + |
| 93 | + |
| 94 | + |
69 | 95 | |
| 96 | + |
| 97 | + |
| 98 | + |
70 | 99 | |
71 | 100 | |
72 | 101 | |
|
0 commit comments