Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

test_runner: simplify hook running logic #55963

Merged
merged 1 commit into from
Nov 25, 2024
+21 −25
Merged

test_runner: simplify hook running logic #55963

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions lib/internal/test_runner/test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
const {
ArrayPrototypePush,
ArrayPrototypePushApply,
ArrayPrototypeReduce,
ArrayPrototypeShift,
ArrayPrototypeSlice,
ArrayPrototypeSome,
Expand Down Expand Up @@ -845,13 +844,14 @@ class Test extends AsyncResource {
async runHook(hook, args) {
validateOneOf(hook, 'hook name', kHookNames);
try {
await ArrayPrototypeReduce(this.hooks[hook], async (prev, hook) => {
await prev;
const hooks = this.hooks[hook];
for (let i = 0; i < hooks.length; ++i) {
const hook = hooks[i];
await hook.run(args);
if (hook.error) {
throw hook.error;
}
}, PromiseResolve());
}
} catch (err) {
const error = new ERR_TEST_FAILURE(`failed running ${hook} hook`, kHookFailure);
error.cause = isTestFailureError(err) ? err.cause : err;
Expand Down
4 changes: 2 additions & 2 deletions test/fixtures/test-runner/output/abort_hooks.snapshot
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ not ok 2 - 2 after describe
*
*
*
async Promise.all (index 0)
pmarchini marked this conversation as resolved.
Show resolved Hide resolved
*
...
# Subtest: test 2
not ok 2 - test 2
Expand All @@ -122,7 +122,7 @@ not ok 2 - 2 after describe
*
*
*
async Promise.all (index 0)
*
...
1..2
not ok 3 - 3 beforeEach describe
Expand Down
1 change: 0 additions & 1 deletion test/fixtures/test-runner/output/global_after_should_fail_the_test.snapshot
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ not ok 2 - /test/fixtures/test-runner/output/global_after_should_fail_the_test.j
*
*
*
*
...
1..1
# tests 1
Expand Down
11 changes: 5 additions & 6 deletions test/fixtures/test-runner/output/hooks.snapshot
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,6 @@ not ok 3 - before throws
*
*
*
*
...
# Subtest: before throws - no subtests
not ok 4 - before throws - no subtests
Expand All @@ -97,7 +96,6 @@ not ok 4 - before throws - no subtests
*
*
*
*
...
# Subtest: after throws
# Subtest: 1
Expand Down Expand Up @@ -129,6 +127,7 @@ not ok 5 - after throws
*
*
*
*
...
# Subtest: after throws - no subtests
not ok 6 - after throws - no subtests
Expand All @@ -149,6 +148,7 @@ not ok 6 - after throws - no subtests
*
*
*
*
...
# Subtest: beforeEach throws
# Subtest: 1
Expand All @@ -167,9 +167,9 @@ not ok 6 - after throws - no subtests
*
*
*
async Promise.all (index 0)
*
*
new Promise (<anonymous>)
...
# Subtest: 2
not ok 2 - 2
Expand All @@ -188,6 +188,8 @@ not ok 6 - after throws - no subtests
*
*
*
*
async Promise.all (index 0)
...
1..2
not ok 7 - beforeEach throws
Expand Down Expand Up @@ -482,7 +484,6 @@ not ok 15 - t.after throws
*
*
*
*
...
# Subtest: t.after throws - no subtests
not ok 16 - t.after throws - no subtests
Expand All @@ -502,7 +503,6 @@ not ok 16 - t.after throws - no subtests
*
*
*
*
...
# Subtest: t.beforeEach throws
# Subtest: 1
Expand Down Expand Up @@ -765,7 +765,6 @@ not ok 24 - run after when before throws
*
*
*
*
...
# Subtest: test hooks - async
# Subtest: 1
Expand Down
22 changes: 10 additions & 12 deletions test/fixtures/test-runner/output/hooks_spec_reporter.snapshot
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@
*
*
*
*

before throws - no subtests (*ms)
Error: before
Expand All @@ -38,7 +37,6 @@
*
*
*
*

after throws
1 (*ms)
Expand All @@ -55,6 +53,7 @@
*
*
*
*

after throws - no subtests (*ms)
Error: after
Expand All @@ -67,6 +66,7 @@
*
*
*
*

beforeEach throws
1 (*ms)
Expand All @@ -78,9 +78,9 @@
*
*
*
at async Promise.all (index 0)
*
*
at new Promise (<anonymous>)

2 (*ms)
Error: beforeEach
Expand All @@ -92,6 +92,8 @@
*
*
*
*
at async Promise.all (index 0)

beforeEach throws (*ms)
afterEach throws
Expand Down Expand Up @@ -242,7 +244,6 @@
*
*
*
*

t.after throws - no subtests (*ms)
Error: after
Expand All @@ -255,7 +256,6 @@
*
*
*
*

t.beforeEach throws
1 (*ms)
Expand Down Expand Up @@ -390,7 +390,6 @@
*
*
*
*

test hooks - async
1 (*ms)
Expand Down Expand Up @@ -430,7 +429,6 @@
*
*
*
*

*
before throws - no subtests (*ms)
Expand All @@ -443,7 +441,6 @@
*
*
*
*

*
after throws (*ms)
Expand All @@ -457,6 +454,7 @@
*
*
*
*

*
after throws - no subtests (*ms)
Expand All @@ -470,6 +468,7 @@
*
*
*
*

*
1 (*ms)
Expand All @@ -481,9 +480,9 @@
*
*
*
at async Promise.all (index 0)
*
*
at new Promise (<anonymous>)

*
2 (*ms)
Expand All @@ -496,6 +495,8 @@
*
*
*
*
at async Promise.all (index 0)

*
1 (*ms)
Expand Down Expand Up @@ -633,7 +634,6 @@
*
*
*
*

*
t.after throws - no subtests (*ms)
Expand All @@ -647,7 +647,6 @@
*
*
*
*

*
1 (*ms)
Expand Down Expand Up @@ -776,4 +775,3 @@
*
*
*
*
Loading