test: remove FIB environment variable from cpu-prof.js

PR-URL: #28183 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: Trott

test/common/README.md

@@ -438,10 +438,9 @@ The `cpu-prof` module provides utilities related to CPU profiling tests.
 
 ### env
 
-* Default:{...process.env, FIB, NODE_DEBUG_NATIVE: 'INSPECTOR_PROFILER' }
+* Default:{...process.env, NODE_DEBUG_NATIVE: 'INSPECTOR_PROFILER' }
 
-Environment variables used in profiled processes. FIB will be set to `40` on
-Windows and `30` elsewhere.
+Environment variables used in profiled processes.
 
 ### getCpuProfiles(dir)
 

test/common/cpu-prof.js

@@ -2,7 +2,7 @@
 
 'use strict';
 
-constcommon=require('./');
+require('./');
 constfs=require('fs');
 constpath=require('path');
 constassert=require('assert');
@@ -34,23 +34,12 @@ function verifyFrames(output, file, suffix){
 assert.notDeepStrictEqual(frames,[]);
 }
 
-letFIB=30;
-// 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.
-if(common.isWindows){
-FIB=40;
-}
-
 // We need to set --cpu-interval to a smaller value to make sure we can
 // find our workload in the samples. 50us should be a small enough sampling
 // interval for this.
 constkCpuProfInterval=50;
 constenv={
  ...process.env,
-FIB,
 NODE_DEBUG_NATIVE: 'INSPECTOR_PROFILER'
 };
 

test/fixtures/workload/fibonacci.js

@@ -4,5 +4,12 @@ function fib(n){
 returnfib(n-1)+fib(n-2);
 }
 
-constn=parseInt(process.env.FIB)||35;
+// 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.
+constFIB=process.platform==='win32' ? 40 : 30;
+
+constn=parseInt(process.env.FIB)||FIB;
 process.stdout.write(`${fib(n)}\n`);