From d10c59fc600c4766de8918f350b4850a1fae8f6a Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Fri, 17 Jul 2020 06:00:03 -0700 Subject: [PATCH] benchmark,test: remove output from readable-async-iterator benchmark Extra output makes test-benchmark-streams fail. Change console.log() to console.assert(). Fixes: https://github.com/nodejs/node/issues/34409 PR-URL: https://github.com/nodejs/node/pull/34411 Reviewed-By: Richard Lau Reviewed-By: Robert Nagy --- benchmark/streams/readable-async-iterator.js | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/benchmark/streams/readable-async-iterator.js b/benchmark/streams/readable-async-iterator.js index 0455f67195fcd6..3d06eda7458105 100644 --- a/benchmark/streams/readable-async-iterator.js +++ b/benchmark/streams/readable-async-iterator.js @@ -34,9 +34,8 @@ async function main({ n, sync }) { } } - // Side effect to ensure V8 does not optimize away the - // loop as a noop. - console.log(x); + // Use x to ensure V8 does not optimize away the loop as a noop. + console.assert(x); bench.end(n); }