diff --git a/test/fixtures/crypto-timing-safe-equal-benchmark-func.js b/test/fixtures/crypto-timing-safe-equal-benchmark-func.js deleted file mode 100644 index 96470e3e4434a8..00000000000000 --- a/test/fixtures/crypto-timing-safe-equal-benchmark-func.js +++ /dev/null @@ -1,17 +0,0 @@ -'use strict'; - -const assert = require('assert'); -module.exports = (compareFunc, firstBufFill, secondBufFill, bufSize) => { - 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); - - return endTime[0] * 1e9 + endTime[1]; -}; diff --git a/test/pummel/test-crypto-timing-safe-equal-benchmarks.js b/test/pummel/test-crypto-timing-safe-equal-benchmarks.js index 0592b9a0d63674..b649b071e1e49d 100644 --- a/test/pummel/test-crypto-timing-safe-equal-benchmarks.js +++ b/test/pummel/test-crypto-timing-safe-equal-benchmarks.js @@ -6,21 +6,24 @@ if (!common.hasCrypto) if (!common.enoughTestMem) common.skip('memory-intensive test'); -const fixtures = require('../common/fixtures'); const assert = require('assert'); const crypto = require('crypto'); -const BENCHMARK_FUNC_PATH = - `${fixtures.fixturesDir}/crypto-timing-safe-equal-benchmark-func`; -function runOneBenchmark(...args) { - const benchmarkFunc = require(BENCHMARK_FUNC_PATH); - const result = 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. - delete require.cache[require.resolve(BENCHMARK_FUNC_PATH)]; - return result; +function runOneBenchmark(compareFunc, firstBufFill, secondBufFill, bufSize) { + return eval(` + 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]; + `); } function getTValue(compareFunc) {