test: add and use spawnSyncAndExit() and spawnSyncAndExitWithoutError() by joyeecheung · Pull Request #49200 · nodejs/node

@joyeecheung

Replaces expectSyncExit() and expectSyncExitWithoutError(). Since
we usually just check the child process right after its spawned,
these shorthands also takes care of the spawning. This makes the
tests more concise.

@nodejs-github-bot added needs-ci

PRs that need a full CI run.

test

Issues and PRs related to the tests.

labels

Aug 16, 2023

LiviaMedeiros

@joyeecheung

LiviaMedeiros

This was referenced

Aug 21, 2023

UlisesGascon pushed a commit that referenced this pull request

Sep 10, 2023
Replaces expectSyncExit() and expectSyncExitWithoutError(). Since
we usually just check the child process right after its spawned,
these shorthands also takes care of the spawning. This makes the
tests more concise.

PR-URL: #49200
Reviewed-By: LiviaMedeiros <livia@cirno.name>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>