The Wayback Machine - https://web.archive.org/web/20220419203643/https://github.com/nodejs/node/commit/97c68ddaad
Skip to content
Permalink
Browse files
test: improve error message in test-tick-processor
Provide additional information about values that indicate test failed.

PR-URL: #7693
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
  • Loading branch information
Trott authored and MylesBorins committed Oct 26, 2016
1 parent 7766997 commit 97c68ddaad19c3e87d579caa8606f30e8454059f
Showing with 6 additions and 6 deletions.
  1. +6 −6 test/parallel/test-tick-processor.js
@@ -1,8 +1,8 @@
'use strict';
var fs = require('fs');
var assert = require('assert');
var cp = require('child_process');
var common = require('../common');
const fs = require('fs');
const assert = require('assert');
const cp = require('child_process');
const common = require('../common');

// TODO(mhdawson) Currently the test-tick-processor functionality in V8
// depends on addresses being smaller than a full 64 bits. Aix supports
@@ -49,12 +49,12 @@ function runTest(pattern, code) {
return /^isolate-/.test(file);
});
if (matches.length != 1) {
assert.fail(null, null, 'There should be a single log file.');
common.fail('There should be a single log file.');
}
var log = matches[0];
var out = cp.execSync(process.execPath +
' --prof-process --call-graph-size=10 ' + log,
{encoding: 'utf8'});
assert(pattern.test(out));
assert(pattern.test(out), `${pattern} not matching ${out}`);
fs.unlinkSync(log);
}

0 comments on commit 97c68dd

Please sign in to comment.
Morty Proxy This is a proxified and sanitized view of the page, visit original site.