test: improve performance of test-crypto-timing-safe-equal-benchmarks

Using `eval()` rather than `require()`'ing a fixture and deleting it from the cache results in a roughtly 10x improvement in running time. Fixes: #25984 Refs: #26229 PR-URL: #26237 Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Yang Guo <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Teddy Katz <[email protected]> Reviewed-By: Richard Lau <[email protected]>

Author: Trott

test/fixtures/crypto-timing-safe-equal-benchmark-func.js

@@ -6,21 +6,24 @@ if (!common.hasCrypto)
 if(!common.enoughTestMem)
 common.skip('memory-intensive test');
 
-constfixtures=require('../common/fixtures');
 constassert=require('assert');
 constcrypto=require('crypto');
 
-constBENCHMARK_FUNC_PATH=
-`${fixtures.fixturesDir}/crypto-timing-safe-equal-benchmark-func`;
-functionrunOneBenchmark(...args){
-constbenchmarkFunc=require(BENCHMARK_FUNC_PATH);
-constresult=benchmarkFunc(...args);
-
-// Don't let the comparison function get cached. This avoid a timing
-// inconsistency due to V8 optimization where the function would take
-// less time when called with a specific set of parameters.
-deleterequire.cache[require.resolve(BENCHMARK_FUNC_PATH)];
-returnresult;
+functionrunOneBenchmark(compareFunc,firstBufFill,secondBufFill,bufSize){
+returneval(`
+ const firstBuffer = Buffer.alloc(bufSize, firstBufFill);
+ const secondBuffer = Buffer.alloc(bufSize, secondBufFill);
+
+ const startTime = process.hrtime();
+ const result = compareFunc(firstBuffer, secondBuffer);
+ const endTime = process.hrtime(startTime);
+
+ // Ensure that the result of the function call gets used, so it doesn't
+ // get discarded due to engine optimizations.
+ assert.strictEqual(result, firstBufFill === secondBufFill);
+
+ endTime[0] * 1e9 + endTime[1];
+ `);
 }
 
 functiongetTValue(compareFunc){