test: refactor pummel/test-dtrace-jsstack

* `os.type()` check -> `common.isSunOS` * != -> !== * == -> === PR-URL: #8175 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
nodejs · Oct 26, 2016 · 11f761ab1a1e1d4fa435d8defe2d67d672b286ed · 11f761a
1 parent 2997b79
commit 11f761ab1a1e1d4fa435d8defe2d67d672b286ed
Showing with 3 additions and 3 deletions.

+3 −3 test/pummel/test-dtrace-jsstack.js
diff --git 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);