Repository navigation
Commit b2f8440
committed
fix: observe every async entry together, and gate the cases on Node.js 14.8
Awaiting the entries one by one leaves the rest unobserved once one rejects,
so a second failing entry became an unhandled rejection. Await them in a
single `Promise.all`.
The two cases need top-level await, which the runner's `vm.SourceTextModule`
cannot parse before Node.js 14.8, and both fixtures now settle on a task so
they fail when startup returns too early.1 parent d5a4fca commit b2f8440
5 files changed
Lines changed: 28 additions & 5 deletions
File tree
- lib/javascript
- test/configCases/module
- async-entry-await-multiple
- async-entry-await
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1872 | 1872 | | |
1873 | 1873 | | |
1874 | 1874 | | |
1875 | | - | |
1876 | | - | |
1877 | | - | |
| 1875 | + | |
| 1876 | + | |
| 1877 | + | |
| 1878 | + | |
| 1879 | + | |
| 1880 | + | |
1878 | 1881 | | |
1879 | 1882 | | |
1880 | 1883 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
| 2 | + | |
| 3 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
| 2 | + | |
| 3 | + | |
2 | 4 | | |
3 | 5 | | |
4 | 6 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
0 commit comments