test: show more info on failure in test-cli-syntax-require.js by joyeecheung · Pull Request #49561 · nodejs/node

@joyeecheung

Use spawnSyncAndExit() to show more info when the tes fails.

@nodejs-github-bot added needs-ci

PRs that need a full CI run.

test

Issues and PRs related to the tests.

labels

Sep 8, 2023

@joyeecheung

ruyadorno

RafaelGSS

Linkgoron

@debadree25 debadree25 added the author ready

PRs that have at least one approval, no pending requests for changes, and a CI started.

label

Sep 12, 2023

This was referenced

Sep 16, 2023

ruyadorno pushed a commit that referenced this pull request

Sep 28, 2023
Use spawnSyncAndExit() to show more info when the tes fails.

PR-URL: #49561
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Ruy Adorno <ruyadorno@google.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Nitzan Uziely <linkgoron@gmail.com>

This was referenced

Sep 28, 2023

alexfernandez pushed a commit to alexfernandez/node that referenced this pull request

Nov 1, 2023
Use spawnSyncAndExit() to show more info when the tes fails.

PR-URL: nodejs#49561
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Ruy Adorno <ruyadorno@google.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Nitzan Uziely <linkgoron@gmail.com>