From e622bc658e3bf2a1dec957791acdde4cfae8f7ed Mon Sep 17 00:00:00 2001 From: Steven Date: Wed, 23 Mar 2022 17:37:37 -0400 Subject: [PATCH] Fix `next info` accidentally printing stderr --- packages/next/cli/next-info.ts | 5 ++++- test/integration/cli/test/index.test.js | 2 ++ 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/packages/next/cli/next-info.ts b/packages/next/cli/next-info.ts index 4fd15e44dfc5..9ac5cb350225 100755 --- a/packages/next/cli/next-info.ts +++ b/packages/next/cli/next-info.ts @@ -99,7 +99,10 @@ function getPackageVersion(packageName: string) { function getBinaryVersion(binaryName: string) { try { - return childProcess.execSync(`${binaryName} --version`).toString().trim() + return childProcess + .execFileSync(binaryName, ['--version']) + .toString() + .trim() } catch { return 'N/A' } diff --git a/test/integration/cli/test/index.test.js b/test/integration/cli/test/index.test.js index 2fbae8c6ce2c..c3d7aad3092b 100644 --- a/test/integration/cli/test/index.test.js +++ b/test/integration/cli/test/index.test.js @@ -489,7 +489,9 @@ describe('CLI Usage', () => { test('should print output', async () => { const info = await runNextCommand(['info'], { stdout: true, + stderr: true, }) + expect(info.stderr || '').toBe('') expect(info.stdout).toMatch( new RegExp(` Operating System: