Browse Source

test: refactor test-tick-processor

The test does some extra work that isn't necessary because of the way
temp directories are handled. The test removes all files from the temp
directory with `common.refreshTmpDir()` but still filters the results
even though only its files will be in the directory).

Refactor to remove that unneeded logic.

PR-URL: https://github.com/nodejs/node/pull/8180
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
v7.x
Rich Trott 8 years ago
parent
commit
2c41723e1c
  1. 10
      test/parallel/test-tick-processor.js

10
test/parallel/test-tick-processor.js

@ -45,12 +45,10 @@ runTest(/RunInDebugContext/,
function runTest(pattern, code) { function runTest(pattern, code) {
cp.execFileSync(process.execPath, ['-prof', '-pe', code]); cp.execFileSync(process.execPath, ['-prof', '-pe', code]);
var matches = fs.readdirSync(common.tmpDir).filter(function(file) { var matches = fs.readdirSync(common.tmpDir);
return /^isolate-/.test(file);
}); assert.strictEqual(matches.length, 1, 'There should be a single log file.');
if (matches.length != 1) {
common.fail('There should be a single log file.');
}
var log = matches[0]; var log = matches[0];
var out = cp.execSync(process.execPath + var out = cp.execSync(process.execPath +
' --prof-process --call-graph-size=10 ' + log, ' --prof-process --call-graph-size=10 ' + log,

Loading…
Cancel
Save