Skip to content

Commit

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

PR-URL: nodejs#42055
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Akhil Marsonya <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Ricky Zhou <[email protected]>
  • Loading branch information
Trott authored and sxa committed Mar 7, 2022
1 parent e3347db commit 9930146
Showing 1 changed file with 1 addition and 8 deletions.
9 changes: 1 addition & 8 deletions test/fixtures/workload/fibonacci.js
Original file line number Diff line number Diff line change
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 9930146

Please sign in to comment.