test: formatting skip messages for TAP parsing

This patch makes the skip messages consistent so that the TAP plugin
in CI can parse the messages properly. The format will be

    1..0 # Skipped: [Actual reason why the test is skipped]

PR-URL: https://github.com/nodejs/io.js/pull/2109
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>

test/parallel/test-cluster-bind-privileged-port.js

@@ -5,7 +5,7 @@ var cluster = require('cluster');
 var net = require('net');
 
 if (process.platform === 'win32') {
-  console.log('Skipping test, not reliable on Windows.');
+  console.log('1..0 # Skipped: not reliable on Windows.');
   process.exit(0);
 }
 

test/parallel/test-cluster-disconnect-unshared-udp.js

@@ -1,6 +1,6 @@
 'use strict';
 if (process.platform === 'win32') {
-  console.log('skipping test on windows, where clustered dgram is ENOTSUP');
+  console.log('1..0 # Skipped: on windows, because clustered dgram is ENOTSUP');
   process.exit(0);
 }
 

test/parallel/test-cluster-shared-handle-bind-privileged-port.js

@@ -5,12 +5,12 @@ var cluster = require('cluster');
 var net = require('net');
 
 if (process.platform === 'win32') {
-  console.log('Skipping test, not reliable on Windows.');
+  console.log('1..0 # Skipped: not reliable on Windows');
   process.exit(0);
 }
 
 if (process.getuid() === 0) {
-  console.log('Do not run this test as root.');
+  console.log('1..0 # Skipped: as this test should not be run as `root`');
   process.exit(0);
 }
 

test/parallel/test-crypto-authenticated.js

@@ -56,7 +56,7 @@ for (var i in TEST_CASES) {
   var test = TEST_CASES[i];
 
   if (ciphers.indexOf(test.algo) == -1) {
-    console.log('skipping unsupported ' + test.algo + ' test');
+    console.log('1..0 # Skipped: unsupported ' + test.algo + ' test');
     continue;
   }
 

test/parallel/test-dgram-bind-default-address.js

@@ -16,7 +16,7 @@ dgram.createSocket('udp4').bind(common.PORT + 0, common.mustCall(function() {
 }));
 
 if (!common.hasIPv6) {
-  console.error('Skipping udp6 part of test, no IPv6 support');
+  console.log('1..0 # Skipped: udp6 part of test, because no IPv6 support');
   return;
 }
 

test/parallel/test-dgram-empty-packet.js

@@ -9,7 +9,7 @@ var client;
 var timer;
 
 if (process.platform === 'darwin') {
-  console.error('Test is disabled due to 17894467 Apple bug');
+  console.log('1..0 # Skipped: because of 17894467 Apple bug');
   return;
 }
 

test/parallel/test-dgram-send-empty-buffer.js

@@ -8,7 +8,7 @@ var callbacks = 0;
 var client, timer, buf;
 
 if (process.platform === 'darwin') {
-  console.error('Test is disabled due to 17894467 Apple bug');
+  console.log('1..0 # Skipped: because of 17894467 Apple bug');
   return;
 }
 

test/parallel/test-fs-readfile-error.js

@@ -7,7 +7,7 @@ var path = require('path');
 // `fs.readFile('/')` does not fail on FreeBSD, because you can open and read
 // the directory there.
 if (process.platform === 'freebsd') {
-  console.error('Skipping test, platform not supported.');
+  console.log('1..0 # Skipped: platform not supported.');
   process.exit();
 }
 

test/parallel/test-fs-readfile-pipe-large.js

@@ -6,7 +6,7 @@ var path = require('path');
 // simulate `cat readfile.js | node readfile.js`
 
 if (process.platform === 'win32') {
-  console.error('No /dev/stdin on windows.  Skipping test.');
+  console.log('1..0 # Skipped: No /dev/stdin on windows.');
   process.exit();
 }
 

test/parallel/test-fs-readfile-pipe.js

@@ -5,7 +5,7 @@ var assert = require('assert');
 // simulate `cat readfile.js | node readfile.js`
 
 if (process.platform === 'win32') {
-  console.error('No /dev/stdin on windows.  Skipping test.');
+  console.log('1..0 # Skipped: No /dev/stdin on windows.');
   process.exit();
 }
 

test/parallel/test-fs-readfilesync-pipe-large.js

@@ -6,7 +6,7 @@ var path = require('path');
 // simulate `cat readfile.js | node readfile.js`
 
 if (process.platform === 'win32') {
-  console.error('No /dev/stdin on windows.  Skipping test.');
+  console.log('1..0 # Skipped: No /dev/stdin on windows.');
   process.exit();
 }
 

test/parallel/test-fs-realpath.js

@@ -80,7 +80,7 @@ function test_simple_error_callback(cb) {
 function test_simple_relative_symlink(callback) {
   console.log('test_simple_relative_symlink');
   if (skipSymlinks) {
-    console.log('skipping symlink test (no privs)');
+    console.log('1..0 # Skipped: symlink test (no privs)');
     return runNextTest();
   }
   var entry = common.tmpDir + '/symlink',
@@ -143,7 +143,7 @@ function test_simple_absolute_symlink(callback) {
 function test_deep_relative_file_symlink(callback) {
   console.log('test_deep_relative_file_symlink');
   if (skipSymlinks) {
-    console.log('skipping symlink test (no privs)');
+    console.log('1..0 # Skipped: symlink test (no privs)');
     return runNextTest();
   }
 
@@ -175,7 +175,7 @@ function test_deep_relative_file_symlink(callback) {
 function test_deep_relative_dir_symlink(callback) {
   console.log('test_deep_relative_dir_symlink');
   if (skipSymlinks) {
-    console.log('skipping symlink test (no privs)');
+    console.log('1..0 # Skipped: symlink test (no privs)');
     return runNextTest();
   }
   var expected = path.join(common.fixturesDir, 'cycles', 'folder');
@@ -207,7 +207,7 @@ function test_deep_relative_dir_symlink(callback) {
 function test_cyclic_link_protection(callback) {
   console.log('test_cyclic_link_protection');
   if (skipSymlinks) {
-    console.log('skipping symlink test (no privs)');
+    console.log('1..0 # Skipped: symlink test (no privs)');
     return runNextTest();
   }
   var entry = common.tmpDir + '/cycles/realpath-3a';
@@ -230,7 +230,7 @@ function test_cyclic_link_protection(callback) {
 function test_cyclic_link_overprotection(callback) {
   console.log('test_cyclic_link_overprotection');
   if (skipSymlinks) {
-    console.log('skipping symlink test (no privs)');
+    console.log('1..0 # Skipped: symlink test (no privs)');
     return runNextTest();
   }
   var cycles = common.tmpDir + '/cycles';
@@ -251,7 +251,7 @@ function test_cyclic_link_overprotection(callback) {
 function test_relative_input_cwd(callback) {
   console.log('test_relative_input_cwd');
   if (skipSymlinks) {
-    console.log('skipping symlink test (no privs)');
+    console.log('1..0 # Skipped: symlink test (no privs)');
     return runNextTest();
   }
 
@@ -295,7 +295,7 @@ function test_deep_symlink_mix(callback) {
   if (isWindows) {
     // This one is a mix of files and directories, and it's quite tricky
     // to get the file/dir links sorted out correctly.
-    console.log('skipping symlink test (no way to work on windows)');
+    console.log('1..0 # Skipped: symlink test (no privs)');
     return runNextTest();
   }
 
@@ -391,7 +391,7 @@ assert.equal(upone, uponeActual,
 function test_up_multiple(cb) {
   console.error('test_up_multiple');
   if (skipSymlinks) {
-    console.log('skipping symlink test (no privs)');
+    console.log('1..0 # Skipped: symlink test (no privs)');
     return runNextTest();
   }
   function cleanup() {

test/parallel/test-http-curl-chunk-problem.js

@@ -2,7 +2,7 @@
 var common = require('../common');
 var assert = require('assert');
 if (!common.opensslCli) {
-  console.error('Skipping because node compiled without OpenSSL CLI.');
+  console.log('1..0 # Skipped: node compiled without OpenSSL CLI.');
   process.exit(0);
 }
 

test/parallel/test-http-full-response.js

@@ -27,7 +27,7 @@ function runAb(opts, callback) {
   exec(command, function(err, stdout, stderr) {
     if (err) {
       if (/ab|apr/mi.test(stderr)) {
-        console.log('problem spawning ab - skipping test.\n' + stderr);
+        console.log('1..0 # Skipped: problem spawning `ab`.\n' + stderr);
         process.reallyExit(0);
       }
       process.exit();

test/parallel/test-http-localaddress.js

@@ -4,7 +4,7 @@ var http = require('http'),
     assert = require('assert');
 
 if (!common.hasMultiLocalhost()) {
-  console.log('Skipping platform-specific test.');
+  console.log('1..0 # Skipped: platform-specific test.');
   process.exit();
 }
 

test/parallel/test-https-foafssl.js

@@ -2,7 +2,7 @@
 var common = require('../common');
 
 if (!common.opensslCli) {
-  console.error('Skipping because node compiled without OpenSSL CLI.');
+  console.log('1..0 # Skipped: node compiled without OpenSSL CLI.');
   process.exit(0);
 }
 

test/parallel/test-https-localaddress.js

@@ -10,7 +10,7 @@ if (!common.hasCrypto) {
 var https = require('https');
 
 if (!common.hasMultiLocalhost()) {
-  console.log('Skipping platform-specific test.');
+  console.log('1..0 # Skipped: platform-specific test.');
   process.exit();
 }
 

test/parallel/test-intl.js

@@ -24,7 +24,7 @@ if (!haveIntl) {
       '"Intl" object is NOT present but v8_enable_i18n_support is ' +
       enablei18n;
   assert.equal(enablei18n, false, erMsg);
-  console.log('Skipping Intl tests because Intl object not present.');
+  console.log('1..0 # Skipped: Intl tests because Intl object not present.');
 
 } else {
   var erMsg =
@@ -46,8 +46,8 @@ if (!haveIntl) {
 
   // If list is specified and doesn't contain 'en' then return.
   if (process.config.variables.icu_locales && !haveLocale('en')) {
-    console.log('Skipping detailed Intl tests because English is not listed ' +
-                'as supported.');
+    console.log('1..0 # Skipped: detailed Intl tests because English is not ' +
+                'listed as supported.');
     // Smoke test. Does it format anything, or fail?
     console.log('Date(0) formatted to: ' + dtf.format(date0));
     return;

test/parallel/test-listen-fd-cluster.js

@@ -10,7 +10,7 @@ var cluster = require('cluster');
 console.error('Cluster listen fd test', process.argv.slice(2));
 
 if (process.platform === 'win32') {
-  console.error('This test is disabled on windows.');
+  console.log('1..0 # Skipped: This test is disabled on windows.');
   return;
 }
 

test/parallel/test-listen-fd-detached-inherit.js

@@ -7,7 +7,7 @@ var PORT = common.PORT;
 var spawn = require('child_process').spawn;
 
 if (process.platform === 'win32') {
-  console.error('This test is disabled on windows.');
+  console.log('1..0 # Skipped: This test is disabled on windows.');
   return;
 }
 

test/parallel/test-listen-fd-detached.js

@@ -7,7 +7,7 @@ var PORT = common.PORT;
 var spawn = require('child_process').spawn;
 
 if (process.platform === 'win32') {
-  console.error('This test is disabled on windows.');
+  console.log('1..0 # Skipped: This test is disabled on windows.');
   return;
 }
 

test/parallel/test-listen-fd-server.js

@@ -7,7 +7,7 @@ var PORT = common.PORT;
 var spawn = require('child_process').spawn;
 
 if (process.platform === 'win32') {
-  console.error('This test is disabled on windows.');
+  console.log('1..0 # Skipped: This test is disabled on windows.');
   return;
 }
 

test/parallel/test-module-loading-error.js

@@ -13,7 +13,7 @@ var error_desc = {
 var dlerror_msg = error_desc[process.platform];
 
 if (!dlerror_msg) {
-  console.error('Skipping test, platform not supported.');
+  console.log('1..0 # Skipped: platform not supported.');
   process.exit();
 }
 

test/parallel/test-net-connect-options-ipv6.js

@@ -5,7 +5,7 @@ var net = require('net');
 var dns = require('dns');
 
 if (!common.hasIPv6) {
-  console.error('Skipping test, no IPv6 support');
+  console.log('1..0 # Skipped: no IPv6 support');
   return;
 }
 

test/parallel/test-process-getgroups.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 var exec = require('child_process').exec;
 
 if (process.platform === 'darwin') {
-  console.log('Skipping. Output of `id -G` is unreliable on Darwin.');
+  console.log('1..0 # Skipped: Output of `id -G` is unreliable on Darwin.');
   return;
 }
 

test/parallel/test-stdio-closed.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 var spawn = require('child_process').spawn;
 
 if (process.platform === 'win32') {
-  console.log('Skipping test, platform not supported.');
+  console.log('1..0 # Skipped: platform not supported.');
   return;
 }
 

test/parallel/test-tls-alert-handling.js

@@ -3,7 +3,7 @@ var common = require('../common');
 var assert = require('assert');
 
 if (!common.opensslCli) {
-  console.error('Skipping because node compiled without OpenSSL CLI.');
+  console.log('1..0 # Skipped: node compiled without OpenSSL CLI.');
   process.exit(0);
 }
 

test/parallel/test-tls-alert.js

@@ -3,7 +3,7 @@ var common = require('../common');
 var assert = require('assert');
 
 if (!common.opensslCli) {
-  console.error('Skipping because node compiled without OpenSSL CLI.');
+  console.log('1..0 # Skipped: node compiled without OpenSSL CLI.');
   process.exit(0);
 }
 

test/parallel/test-tls-no-sslv3.js

@@ -12,7 +12,7 @@ var fs = require('fs');
 var spawn = require('child_process').spawn;
 
 if (common.opensslCli === false) {
-  console.error('Skipping because openssl command cannot be executed');
+  console.log('1..0 # Skipped: node compiled without OpenSSL CLI.');
   process.exit(0);
 }
 

test/parallel/test-tls-npn-server-client.js

@@ -1,6 +1,6 @@
 'use strict';
 if (!process.features.tls_npn) {
-  console.error('Skipping because node compiled without OpenSSL or ' +
+  console.log('1..0 # Skipped: node compiled without OpenSSL or ' +
               'with old OpenSSL version.');
   process.exit(0);
 }

test/parallel/test-tls-ocsp-callback.js

@@ -2,12 +2,12 @@
 var common = require('../common');
 
 if (!process.features.tls_ocsp) {
-  console.error('Skipping because node compiled without OpenSSL or ' +
+  console.log('1..0 # Skipped: node compiled without OpenSSL or ' +
               'with old OpenSSL version.');
   process.exit(0);
 }
 if (!common.opensslCli) {
-  console.error('Skipping because node compiled without OpenSSL CLI.');
+  console.log('1..0 # Skipped: node compiled without OpenSSL CLI.');
   process.exit(0);
 }
 

test/parallel/test-tls-server-verify.js

@@ -2,7 +2,7 @@
 var common = require('../common');
 
 if (!common.opensslCli) {
-  console.error('Skipping because node compiled without OpenSSL CLI.');
+  console.log('1..0 # Skipped: node compiled without OpenSSL CLI.');
   process.exit(0);
 }
 

test/parallel/test-tls-session-cache.js

@@ -2,7 +2,7 @@
 var common = require('../common');
 
 if (!common.opensslCli) {
-  console.error('Skipping because node compiled without OpenSSL CLI.');
+  console.log('1..0 # Skipped: node compiled without OpenSSL CLI.');
   process.exit(0);
 }
 

test/parallel/test-tls-set-ciphers.js

@@ -2,7 +2,7 @@
 var common = require('../common');
 
 if (!common.opensslCli) {
-  console.error('Skipping because node compiled without OpenSSL CLI.');
+  console.log('1..0 # Skipped: node compiled without OpenSSL CLI.');
   process.exit(0);
 }
 

test/parallel/test-tls-sni-option.js

@@ -1,6 +1,6 @@
 'use strict';
 if (!process.features.tls_sni) {
-  console.error('Skipping because node compiled without OpenSSL or ' +
+  console.log('1..0 # Skipped: node compiled without OpenSSL or ' +
               'with old OpenSSL version.');
   process.exit(0);
 }

test/parallel/test-tls-sni-server-client.js

@@ -1,6 +1,6 @@
 'use strict';
 if (!process.features.tls_sni) {
-  console.error('Skipping because node compiled without OpenSSL or ' +
+  console.log('1..0 # Skipped: node compiled without OpenSSL or ' +
               'with old OpenSSL version.');
   process.exit(0);
 }

test/parallel/test-tty-wrap.js

@@ -6,7 +6,7 @@ var TTY = process.binding('tty_wrap').TTY;
 var isTTY = process.binding('tty_wrap').isTTY;
 
 if (isTTY(1) == false) {
-  console.error('fd 1 is not a tty. skipping test.');
+  console.log('1..0 # Skipped: fd 1 is not a tty.');
   process.exit(0);
 }
 

test/pummel/test-abort-fatal-error.js

@@ -3,7 +3,7 @@ var assert = require('assert');
 var common = require('../common');
 
 if (process.platform === 'win32') {
-  console.log('skipping test on windows');
+  console.log('1..0 # Skipped: no RLIMIT_NOFILE on Windows');
   process.exit(0);
 }
 

test/pummel/test-dtrace-jsstack.js

@@ -5,7 +5,7 @@ var os = require('os');
 var util = require('util');
 
 if (os.type() != 'SunOS') {
-  console.error('Skipping because DTrace not available.');
+  console.log('1..0 # Skipped: no DTRACE support');
   process.exit(0);
 }
 

test/pummel/test-https-ci-reneg-attack.js

@@ -13,7 +13,7 @@ var https = require('https');
 var fs = require('fs');
 
 if (!common.opensslCli) {
-  console.error('Skipping because node compiled without OpenSSL CLI.');
+  console.log('1..0 # Skipped: node compiled without OpenSSL CLI.');
   process.exit(0);
 }
 

test/pummel/test-keep-alive.js

@@ -1,6 +1,6 @@
 'use strict';
 if (process.platform === 'win32') {
-  console.log('skipping this test because there is no wrk on windows');
+  console.log('1..0 # Skipped: no `wrk` on windows');
   process.exit(0);
 }
 

test/pummel/test-tls-ci-reneg-attack.js

@@ -12,7 +12,7 @@ var tls = require('tls');
 var fs = require('fs');
 
 if (!common.opensslCli) {
-  console.error('Skipping because node compiled without OpenSSL CLI.');
+  console.log('1..0 # Skipped: node compiled without OpenSSL CLI.');
   process.exit(0);
 }
 

test/pummel/test-tls-securepair-client.js

@@ -4,7 +4,7 @@
 var common = require('../common');
 
 if (!common.opensslCli) {
-  console.error('Skipping because node compiled without OpenSSL CLI.');
+  console.log('1..0 # Skipped: node compiled without OpenSSL CLI.');
   process.exit(0);
 }
 

test/pummel/test-tls-session-timeout.js

@@ -2,7 +2,7 @@
 var common = require('../common');
 
 if (!common.opensslCli) {
-  console.error('Skipping because node compiled without OpenSSL CLI.');
+  console.log('1..0 # Skipped: node compiled without OpenSSL CLI.');
   process.exit(0);
 }
 

test/sequential/test-child-process-emfile.js

@@ -5,7 +5,7 @@ var spawn = require('child_process').spawn;
 var fs = require('fs');
 
 if (process.platform === 'win32') {
-  console.log('Skipping test, no RLIMIT_NOFILE on Windows.');
+  console.log('1..0 # Skipped: no RLIMIT_NOFILE on Windows');
   return;
 }
 

test/sequential/test-net-server-address.js

@@ -37,7 +37,7 @@ server_ipv6.listen(common.PORT, localhost_ipv6, function() {
 });
 
 if (!common.hasIPv6) {
-  console.error('Skipping ipv6 part of test, no IPv6 support');
+  console.log('1..0 # Skipped: ipv6 part of test, no IPv6 support');
   return;
 }
 

test/sequential/test-setproctitle.js

@@ -3,7 +3,7 @@
 
 // FIXME add sunos support
 if ('linux freebsd darwin'.indexOf(process.platform) === -1) {
-  console.error('Skipping test, platform not supported.');
+  console.log(`1..0 # Skipped: Unsupported platform [${process.platform}]`);
   process.exit();
 }