Skip to content

Commit c635b81

Browse files
committed
instead of Promise.all, resolve sequentially in test run because not concurrent-safe
Signed-off-by: Kipras Melnikovas <kipras@kipras.org>
1 parent e17957f commit c635b81

File tree

2 files changed

+11
-3
lines changed

2 files changed

+11
-3
lines changed

test/run.ts

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,11 +3,14 @@
33
import { testCase } from "./experiment.spec";
44
import reducePathTC from "../reducePath.spec";
55

6+
import { sequentialResolve } from "../util/sequentialResolve";
7+
68
main();
79
function main() {
8-
Promise.all([
9-
testCase(), //
10-
reducePathTC(),
10+
// TODO Promise.all
11+
sequentialResolve([
12+
testCase, //
13+
async () => reducePathTC(),
1114
])
1215
.then(() => process.stdout.write("\nsuccess\n\n"))
1316
.catch((e) => {

util/sequentialResolve.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
export const sequentialResolve = <T>(xs: (() => Promise<T>)[]): Promise<T> =>
2+
xs.reduce(
3+
(prev, curr) => prev.then(curr), //
4+
(Promise.resolve() as unknown) as Promise<T>
5+
);

0 commit comments

Comments
 (0)