test: skip the test with proper TAP message

On Windows, the test simply returns which will be counted as passed. The test actually skips the rest of the test. So proper TAP message has to be included while skipping. This patch uses `common.skip` rather than `console.log` to print the skip messages. PR-URL: #11584 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
nodejs · Mar 7, 2017 · fdb4a6c · fdb4a6c
1 parent ed6d741
commit fdb4a6c
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/test/parallel/test-setproctitle.js b/test/parallel/test-setproctitle.js
@@ -4,8 +4,7 @@ const common = require('../common');
 
 // FIXME add sunos support
 if (common.isSunOS) {
-  console.log(`1..0 # Skipped: Unsupported platform [${process.platform}]`);
-  return;
+  return common.skip(`Unsupported platform [${process.platform}]`);
 }
 
 const assert = require('assert');
@@ -22,8 +21,9 @@ process.title = title;
 assert.strictEqual(process.title, title);
 
 // Test setting the title but do not try to run `ps` on Windows.
-if (common.isWindows)
-  return;
+if (common.isWindows) {
+  return common.skip('Windows does not have "ps" utility');
+}
 
 exec(`ps -p ${process.pid} -o args=`, function callback(error, stdout, stderr) {
   assert.ifError(error);