From 289009fd655da6d898de7aed76ed870e161a646f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C3=ABl=20Zasso?= Date: Tue, 4 Jun 2019 08:42:55 +0200 Subject: [PATCH] test: fix test-linux-perf - Set maxBuffer to Infinity because the output can exceed 1MB. - Fix error checking for perfScript --- test/v8-updates/test-linux-perf.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/test/v8-updates/test-linux-perf.js b/test/v8-updates/test-linux-perf.js index 40823b8e8bd3b1..c5a418d9146f48 100644 --- a/test/v8-updates/test-linux-perf.js +++ b/test/v8-updates/test-linux-perf.js @@ -41,6 +41,7 @@ const perfScriptArgs = [ const options = { cwd: tmpdir.path, encoding: 'utf-8', + maxBuffer: Infinity }; if (!common.isLinux) @@ -57,8 +58,8 @@ if (perf.status !== 0) { const perfScript = spawnSync('perf', perfScriptArgs, options); -if (perf.error) - common.skip(`perf script aborted: ${perf.error.errno}`); +if (perfScript.error) + common.skip(`perf script aborted: ${perfScript.error.errno}`); if (perfScript.status !== 0) { common.skip(`Failed to execute perf script: ${perfScript.stderr}`);