From 99301469ed4b8a4632d651ca3be45c1b135bea84 Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Mon, 21 Feb 2022 12:30:06 -0800 Subject: [PATCH] test: increase Fibonacci argument to 40 Fixes: https://github.com/nodejs/node/issues/34031 PR-URL: https://github.com/nodejs/node/pull/42055 Reviewed-By: Luigi Pinca Reviewed-By: Akhil Marsonya Reviewed-By: Darshan Sen Reviewed-By: Ricky Zhou <0x19951125@gmail.com> --- test/fixtures/workload/fibonacci.js | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/test/fixtures/workload/fibonacci.js b/test/fixtures/workload/fibonacci.js index 0326914d048203..90ef3e99fb39a3 100644 --- a/test/fixtures/workload/fibonacci.js +++ b/test/fixtures/workload/fibonacci.js @@ -4,12 +4,5 @@ function fib(n) { return fib(n - 1) + fib(n - 2); } -// This is based on emperial values - in the CI, on Windows the program -// tend to finish too fast then we won't be able to see the profiled script -// in the samples, so we need to bump the values a bit. On slower platforms -// like the Pis it could take more time to complete, we need to use a -// smaller value so the test would not time out. -const FIB = process.platform === 'win32' ? 40 : 30; - -const n = parseInt(process.env.FIB) || FIB; +const n = parseInt(process.env.FIB, 10) || 40; process.stdout.write(`${fib(n)}\n`);