From c568dfad3969df7df8863c602068fc80da746ad7 Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Thu, 18 Aug 2016 16:06:55 -0700 Subject: [PATCH] test: refactor pummel/test-dtrace-jsstack * `os.type()` check -> `common.isSunOS` * != -> !== * == -> === PR-URL: https://github.com/nodejs/node/pull/8175 Reviewed-By: James M Snell Reviewed-By: Colin Ihrig --- test/pummel/test-dtrace-jsstack.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/pummel/test-dtrace-jsstack.js b/test/pummel/test-dtrace-jsstack.js index b50794bd7b78cf..1949045888c138 100644 --- a/test/pummel/test-dtrace-jsstack.js +++ b/test/pummel/test-dtrace-jsstack.js @@ -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);