GeezerVison

The World Wide Web, restored to 800 pixels of glory.

GeezerVison is showing github.com — test_runner: refactor Promise chain in run() · nodejs/node@fb98fa4

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

lib/internal/test_runner
- runner.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

`‎lib/internal/test_runner/runner.js‎`

Copy file name to clipboardExpand all lines: lib/internal/test_runner/runner.js

+10-3Lines changed: 10 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,6 @@ const {`
`17`	`17`	`ArrayPrototypeSort,`
`18`	`18`	`ObjectAssign,`
`19`	`19`	`PromisePrototypeThen,`
`20`		`- PromiseResolve,`
`21`	`20`	`PromiseWithResolvers,`
`22`	`21`	`SafeMap,`
`23`	`22`	`SafePromiseAll,`
`@@ -798,9 +797,17 @@ function run(options = kEmptyObject) {`
`798`	`797`	`}`
`799`	`798`	`}`
`800`	`799`
`801`		`- const setupPromise = PromiseResolve(setup?.(root.reporter));`
`802`		`- PromisePrototypeThen(PromisePrototypeThen(PromisePrototypeThen(setupPromise, runFiles), postRun), teardown);`
	`800`	`+ const runChain = async () => {`
	`801`	`+ if (typeof setup === 'function') {`
	`802`	`+ await setup(root.reporter);`
	`803`	`+ }`
	`804`	`+`
	`805`	`+ await runFiles();`
	`806`	`+ postRun?.();`
	`807`	`+ teardown?.();`
	`808`	`+ };`
`803`	`809`
	`810`	`+ runChain();`
`804`	`811`	`return root.reporter;`
`805`	`812`	`}`
`806`	`813`

0 commit comments

Comments

0 (0)

You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.