1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: refactor pummel/test-dtrace-jsstack

Browse Source 
* `os.type()` check -> `common.isSunOS`
* != -> !==
* == -> ===

PR-URL: https://github.com/nodejs/node/pull/8175
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
This commit is contained in:
Rich Trott 2016-08-18 16:06:55 -07:00
parent 9604d29659
commit 25703dc4fc
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
test/pummel/test-dtrace-jsstack.js
View File

@ -3,7 +3,7 @@ const common = require('../common');
var assert = require('assert');
var os = require('os');
if (os.type() != 'SunOS') {
if (!common.isSunOS) {
common.skip('no DTRACE support');
return;
}
@ -52,7 +52,7 @@ dtrace.stdout.on('data', function(data) {
});
dtrace.on('exit', function(code) {
if (code != 0) {
if (code !== 0) {
console.error('dtrace exited with code ' + code);
process.exit(code);
}
@ -65,7 +65,7 @@ dtrace.on('exit', function(code) {
for (var i = 0; i < lines.length; i++) {
var line = lines[i];
if (line.indexOf(sentinel) == -1 || frames.length === 0)
if (line.indexOf(sentinel) === -1 || frames.length === 0)
continue;
var frame = line.substr(line.indexOf(sentinel) + sentinel.length);