Skip to content

Commit

Permalink
test: refactor pummel/test-dtrace-jsstack
Browse files Browse the repository at this point in the history
* `os.type()` check -> `common.isSunOS`
* != -> !==
* == -> ===

PR-URL: #8175
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
  • Loading branch information
Trott authored and evanlucas committed Aug 24, 2016
1 parent cfbafd7 commit c568dfa
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/pummel/test-dtrace-jsstack.js
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}
Expand Down Expand Up @@ -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);
}
Expand All @@ -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);
Expand Down

0 comments on commit c568dfa

Please sign in to comment.