Skip to content

Commit d5e95e1

Browse files
committed
Refactor to make it clear we don't double release
1 parent 6d0dd13 commit d5e95e1

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

run-tests.js

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -794,17 +794,16 @@ ${ENDGROUP}`)
794794
// once CI runs with Node.js 24+.
795795
if (dirSema) await dirSema.acquire()
796796
await sema.acquire()
797-
if (testSignal.aborted) {
798-
const error = new Error(`Skipped due to abort.`, {
799-
cause: testSignal.reason,
800-
})
801-
error.name = test.file
802-
if (dirSema) dirSema.release()
803-
sema.release()
804-
throw error
805-
}
806797

807798
try {
799+
if (testSignal.aborted) {
800+
const error = new Error(`Skipped due to abort.`, {
801+
cause: testSignal.reason,
802+
})
803+
error.name = test.file
804+
throw error
805+
}
806+
808807
await runTest(test)
809808
} finally {
810809
sema.release()

0 commit comments

Comments
 (0)