test: changing process.exit to return while skipping tests

This patch uses `return` statement to skip the test instead of using
`process.exit` call.

PR-URL: #2109
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>

Author: thefourtheye

test/internet/test-http-https-default-ports.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var https = require('https');
 

test/internet/test-tls-connnect-cnnic.js

@@ -11,7 +11,7 @@
 var common = require('../common');
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 
 var tls = require('tls');

test/internet/test-tls-connnect-melissadata.js

@@ -5,7 +5,7 @@
 var common = require('../common');
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 
 var tls = require('tls');

test/internet/test-tls-reuse-host-from-socket.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var tls = require('tls');
 

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

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

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

@@ -1,7 +1,7 @@
 'use strict';
 if (process.platform === 'win32') {
   console.log('1..0 # Skipped: on windows, because clustered dgram is ENOTSUP');
-  process.exit(0);
+  return;
 }
 
 var cluster = require('cluster');

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

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

test/parallel/test-crypto-authenticated.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 

test/parallel/test-crypto-binary-default.js

@@ -9,7 +9,7 @@ var constants = require('constants');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 var tls = require('tls');

test/parallel/test-crypto-certificate.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 

test/parallel/test-crypto-cipher-decipher.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 

test/parallel/test-crypto-dh-odd-key.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 

test/parallel/test-crypto-dh.js

@@ -5,7 +5,7 @@ var constants = require('constants');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 

test/parallel/test-crypto-domain.js

@@ -5,7 +5,7 @@ var domain = require('domain');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 

test/parallel/test-crypto-domains.js

@@ -8,7 +8,7 @@ var errors = 0;
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 

test/parallel/test-crypto-ecb.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 

test/parallel/test-crypto-from-binary.js

@@ -8,7 +8,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 

test/parallel/test-crypto-hash-stream-pipe.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 

test/parallel/test-crypto-hash.js

@@ -6,7 +6,7 @@ var path = require('path');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 

test/parallel/test-crypto-hmac.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 

test/parallel/test-crypto-padding-aes256.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 

test/parallel/test-crypto-padding.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 

test/parallel/test-crypto-pbkdf2.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 

test/parallel/test-crypto-random.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 

test/parallel/test-crypto-rsa-dsa.js

@@ -6,7 +6,7 @@ var constants = require('constants');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 

test/parallel/test-crypto-sign-verify.js

@@ -5,7 +5,7 @@ var fs = require('fs');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 

test/parallel/test-crypto-stream.js

@@ -6,7 +6,7 @@ var util = require('util');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 

test/parallel/test-crypto-verify-failure.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 var tls = require('tls');

test/parallel/test-crypto.js

@@ -5,7 +5,7 @@ var util = require('util');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var crypto = require('crypto');
 

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

@@ -6,7 +6,7 @@ var dgram = require('dgram');
 // skip test in FreeBSD jails since 0.0.0.0 will resolve to default interface
 if (common.inFreeBSDJail) {
   console.log('1..0 # Skipped: In a FreeBSD jail');
-  process.exit();
+  return;
 }
 
 dgram.createSocket('udp4').bind(common.PORT + 0, common.mustCall(function() {

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

@@ -8,7 +8,7 @@ var path = require('path');
 // the directory there.
 if (process.platform === 'freebsd') {
   console.log('1..0 # Skipped: platform not supported.');
-  process.exit();
+  return;
 }
 
 var callbacks = 0;

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

@@ -7,7 +7,7 @@ var path = require('path');
 
 if (process.platform === 'win32') {
   console.log('1..0 # Skipped: No /dev/stdin on windows.');
-  process.exit();
+  return;
 }
 
 var fs = require('fs');

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

@@ -6,7 +6,7 @@ var assert = require('assert');
 
 if (process.platform === 'win32') {
   console.log('1..0 # Skipped: No /dev/stdin on windows.');
-  process.exit();
+  return;
 }
 
 var fs = require('fs');

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

@@ -7,7 +7,7 @@ var path = require('path');
 
 if (process.platform === 'win32') {
   console.log('1..0 # Skipped: No /dev/stdin on windows.');
-  process.exit();
+  return;
 }
 
 var fs = require('fs');

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

@@ -3,7 +3,7 @@ var common = require('../common');
 var assert = require('assert');
 if (!common.opensslCli) {
   console.log('1..0 # Skipped: node compiled without OpenSSL CLI.');
-  process.exit(0);
+  return;
 }
 
 // http://groups.google.com/group/nodejs/browse_thread/thread/f66cd3c960406919

test/parallel/test-http-localaddress.js

@@ -5,7 +5,7 @@ var http = require('http'),
 
 if (!common.hasMultiLocalhost()) {
   console.log('1..0 # Skipped: platform-specific test.');
-  process.exit();
+  return;
 }
 
 var server = http.createServer(function(req, res) {

test/parallel/test-http-url.parse-https.request.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var https = require('https');
 

test/parallel/test-https-agent-servername.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 
 var https = require('https');

test/parallel/test-https-agent.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var https = require('https');
 

test/parallel/test-https-byteswritten.js

@@ -6,7 +6,7 @@ var http = require('http');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var https = require('https');
 

test/parallel/test-https-client-checkServerIdentity.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var https = require('https');
 

test/parallel/test-https-client-get-url.js

@@ -7,7 +7,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var https = require('https');
 

test/parallel/test-https-client-reject.js

@@ -4,7 +4,7 @@ var assert = require('assert');
 
 if (!common.hasCrypto) {
   console.log('1..0 # Skipped: missing crypto');
-  process.exit();
+  return;
 }
 var https = require('https');