Skip to content

Commit

Permalink
benchmark: add simple module-require benchmark
Browse files Browse the repository at this point in the history
  • Loading branch information
zhangyongsheng committed Dec 30, 2020
1 parent 452cf13 commit 2ec982a
Show file tree
Hide file tree
Showing 2 changed files with 81 additions and 1 deletion.
80 changes: 80 additions & 0 deletions benchmark/module/module-require.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,80 @@
'use strict';

const fs = require('fs');
const path = require('path');
const common = require('../common.js');
const tmpdir = require('../../test/common/tmpdir');
const benchmarkDirectory = path.join(tmpdir.path, 'nodejs-benchmark-module');

const bench = common.createBenchmark(main, {
type: ['.js', '.json', 'dir'],
n: [1e4],
});

function main({ type, n }) {
tmpdir.refresh();
createEntryPoint(n);

switch (type) {
case '.js':
measureJSFile(n);
break;
case '.json':
measureJSONFile(n);
break;
case 'dir':
measureDir(n);
}

tmpdir.refresh();
}

function measureJSFile(n) {
bench.start();
for (let i = 0; i < n; i++) {
require(`${benchmarkDirectory}/${i}`);
}
bench.end(n);
}

function measureJSONFile(n) {
bench.start();
for (let i = 0; i < n; i++) {
require(`${benchmarkDirectory}/json_${i}.json`);
}
bench.end(n);
}

function measureDir(n) {
bench.start();
for (let i = 0; i < n; i++) {
require(`${benchmarkDirectory}${i}`);
}
bench.end(n);
}

function createEntryPoint(n) {
fs.mkdirSync(benchmarkDirectory);

const JSFileContent = 'module.exports = [];';
const JSONFileContent = '[]';

for (let i = 0; i < n; i++) {
// JS file.
fs.writeFileSync(`${benchmarkDirectory}/${i}.js`, JSFileContent);

// JSON file.
fs.writeFileSync(`${benchmarkDirectory}/json_${i}.json`, JSONFileContent);

// Dir.
fs.mkdirSync(`${benchmarkDirectory}${i}`);
fs.writeFileSync(
`${benchmarkDirectory}${i}/package.json`,
'{"main": "index.js"}'
);
fs.writeFileSync(
`${benchmarkDirectory}${i}/index.js`,
JSFileContent
);
}
}
2 changes: 1 addition & 1 deletion lib/internal/modules/cjs/loader.js
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@ function stat(filename) {
if (result !== undefined) return result;
}
const result = internalModuleStat(filename);
if (statCache !== null && result === 0) {
if (statCache !== null && result >= 0) {
// Only set cache when `internalModuleStat(filename)` succeeds.
statCache.set(filename, result);
}
Expand Down

0 comments on commit 2ec982a

Please sign in to comment.