Skip to content

Commit

Permalink
test: increase Fibonacci argument to 40
Browse files Browse the repository at this point in the history
Fixes: #34031

PR-URL: #42055
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Akhil Marsonya <akhil.marsonya27@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Ricky Zhou <0x19951125@gmail.com>
  • Loading branch information
Trott authored and danielleadams committed Apr 24, 2022
1 parent b969a27 commit 17af006
Showing 1 changed file with 1 addition and 8 deletions.
9 changes: 1 addition & 8 deletions test/fixtures/workload/fibonacci.js
Expand Up @@ -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`);

0 comments on commit 17af006

Please sign in to comment.