test: increase usage of assert.ifError()

PR-URL: #10543
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

Author: cjihrig

test/disabled/test-dgram-send-error.js

@@ -34,7 +34,7 @@ function onMessage(message, info) {
 }
 
 function afterSend(err) {
-  if (err) throw err;
+  assert.ifError(err);
   packetsSent++;
 }
 

test/gc/node_modules/nan/tools/1to2.js

@@ -179,7 +179,7 @@ if (process.argv[2] !== 'child') {
       common.PORT,
       LOCAL_BROADCAST_HOST,
       function(err) {
-        if (err) throw err;
+        assert.ifError(err);
         console.error('[PARENT] sent %s to %s:%s',
                       util.inspect(buf.toString()),
                       LOCAL_BROADCAST_HOST, common.PORT);

test/internet/test-dgram-broadcast-multi-process.js

@@ -160,7 +160,7 @@ if (process.argv[2] !== 'child') {
       common.PORT,
       LOCAL_BROADCAST_HOST,
       function(err) {
-        if (err) throw err;
+        assert.ifError(err);
         console.error('[PARENT] sent "%s" to %s:%s',
                       buf.toString(),
                       LOCAL_BROADCAST_HOST, common.PORT);

test/internet/test-dgram-multicast-multi-process.js

@@ -87,7 +87,7 @@ TEST(function test_lookup_ipv6_explicit(done) {
 /* This ends up just being too problematic to test
 TEST(function test_lookup_ipv6_implicit(done) {
   var req = dns.lookup('ipv6.google.com', function(err, ip, family) {
-    if (err) throw err;
+    assert.ifError(err);
     assert.ok(net.isIPv6(ip));
     assert.strictEqual(family, 6);
 
@@ -196,7 +196,7 @@ TEST(function test_lookupservice_ip_ipv6(done) {
 /* Disabled because it appears to be not working on linux. */
 /* TEST(function test_lookup_localhost_ipv6(done) {
   var req = dns.lookup('localhost', 6, function(err, ip, family) {
-    if (err) throw err;
+    assert.ifError(err);
     assert.ok(net.isIPv6(ip));
     assert.strictEqual(family, 6);
 

test/internet/test-dns-ipv6.js

@@ -104,8 +104,7 @@ TEST(function test_resolve6_ttl(done) {
 
 TEST(function test_resolveMx(done) {
   var req = dns.resolveMx('gmail.com', function(err, result) {
-    if (err) throw err;
-
+    assert.ifError(err);
     assert.ok(result.length > 0);
 
     for (var i = 0; i < result.length; i++) {
@@ -140,8 +139,7 @@ TEST(function test_resolveMx_failure(done) {
 
 TEST(function test_resolveNs(done) {
   var req = dns.resolveNs('rackspace.com', function(err, names) {
-    if (err) throw err;
-
+    assert.ifError(err);
     assert.ok(names.length > 0);
 
     for (var i = 0; i < names.length; i++) {
@@ -171,8 +169,7 @@ TEST(function test_resolveNs_failure(done) {
 
 TEST(function test_resolveSrv(done) {
   var req = dns.resolveSrv('_jabber._tcp.google.com', function(err, result) {
-    if (err) throw err;
-
+    assert.ifError(err);
     assert.ok(result.length > 0);
 
     for (var i = 0; i < result.length; i++) {
@@ -209,8 +206,7 @@ TEST(function test_resolveSrv_failure(done) {
 
 TEST(function test_resolvePtr(done) {
   var req = dns.resolvePtr('8.8.8.8.in-addr.arpa', function(err, result) {
-    if (err) throw err;
-
+    assert.ifError(err);
     assert.ok(result.length > 0);
 
     for (var i = 0; i < result.length; i++) {
@@ -240,8 +236,7 @@ TEST(function test_resolvePtr_failure(done) {
 
 TEST(function test_resolveNaptr(done) {
   var req = dns.resolveNaptr('sip2sip.info', function(err, result) {
-    if (err) throw err;
-
+    assert.ifError(err);
     assert.ok(result.length > 0);
 
     for (var i = 0; i < result.length; i++) {
@@ -278,8 +273,7 @@ TEST(function test_resolveNaptr_failure(done) {
 
 TEST(function test_resolveSoa(done) {
   var req = dns.resolveSoa('nodejs.org', function(err, result) {
-    if (err) throw err;
-
+    assert.ifError(err);
     assert.ok(result);
     assert.strictEqual(typeof result, 'object');
 
@@ -325,8 +319,7 @@ TEST(function test_resolveSoa_failure(done) {
 
 TEST(function test_resolveCname(done) {
   var req = dns.resolveCname('www.microsoft.com', function(err, names) {
-    if (err) throw err;
-
+    assert.ifError(err);
     assert.ok(names.length > 0);
 
     for (var i = 0; i < names.length; i++) {
@@ -357,7 +350,7 @@ TEST(function test_resolveCname_failure(done) {
 
 TEST(function test_resolveTxt(done) {
   var req = dns.resolveTxt('google.com', function(err, records) {
-    if (err) throw err;
+    assert.ifError(err);
     assert.strictEqual(records.length, 1);
     assert.ok(util.isArray(records[0]));
     assert.strictEqual(records[0][0].indexOf('v=spf1'), 0);
@@ -398,7 +391,7 @@ TEST(function test_lookup_failure(done) {
 
 TEST(function test_lookup_null(done) {
   var req = dns.lookup(null, function(err, ip, family) {
-    if (err) throw err;
+    assert.ifError(err);
     assert.strictEqual(ip, null);
     assert.strictEqual(family, 4);
 
@@ -411,7 +404,7 @@ TEST(function test_lookup_null(done) {
 
 TEST(function test_lookup_ip_all(done) {
   var req = dns.lookup('127.0.0.1', {all: true}, function(err, ips, family) {
-    if (err) throw err;
+    assert.ifError(err);
     assert.ok(Array.isArray(ips));
     assert.ok(ips.length > 0);
     assert.strictEqual(ips[0].address, '127.0.0.1');
@@ -426,7 +419,7 @@ TEST(function test_lookup_ip_all(done) {
 
 TEST(function test_lookup_null_all(done) {
   var req = dns.lookup(null, {all: true}, function(err, ips, family) {
-    if (err) throw err;
+    assert.ifError(err);
     assert.ok(Array.isArray(ips));
     assert.strictEqual(ips.length, 0);
 
@@ -439,7 +432,7 @@ TEST(function test_lookup_null_all(done) {
 
 TEST(function test_lookup_all_mixed(done) {
   var req = dns.lookup('www.google.com', {all: true}, function(err, ips) {
-    if (err) throw err;
+    assert.ifError(err);
     assert.ok(Array.isArray(ips));
     assert.ok(ips.length > 0);
 

test/internet/test-dns.js

@@ -76,7 +76,7 @@ if (process.argv[2] === 'child') {
           serverPort,
           '127.0.0.1',
           function(err) {
-            if (err) throw err;
+            assert.ifError(err);
           }
         );
       }

test/parallel/test-child-process-fork-dgram.js

@@ -31,7 +31,7 @@ d.run(function() {
     var fs = require('fs');
     fs.readdir(__dirname, function() {
       fs.open('this file does not exist', 'r', function(er) {
-        if (er) throw er;
+        assert.ifError(er);
         throw new Error('should not get here!');
       });
     });

test/parallel/test-domain-implicit-fs.js

@@ -9,14 +9,14 @@ common.refreshTmpDir();
 
 assert.doesNotThrow(() => {
   fs.access(Buffer.from(common.tmpDir), common.mustCall((err) => {
-    if (err) throw err;
+    assert.ifError(err);
   }));
 });
 
 assert.doesNotThrow(() => {
   const buf = Buffer.from(path.join(common.tmpDir, 'a.txt'));
   fs.open(buf, 'w+', common.mustCall((err, fd) => {
-    if (err) throw err;
+    assert.ifError(err);
     assert(fd);
     fs.close(fd, common.mustCall(() => {
       fs.unlinkSync(buf);
@@ -30,12 +30,12 @@ assert.throws(() => {
 
 const dir = Buffer.from(common.fixturesDir);
 fs.readdir(dir, 'hex', common.mustCall((err, list) => {
-  if (err) throw err;
+  assert.ifError(err);
   list = list.map((i) => {
     return Buffer.from(i, 'hex').toString();
   });
   fs.readdir(dir, common.mustCall((err, list2) => {
-    if (err) throw err;
+    assert.ifError(err);
     assert.deepStrictEqual(list, list2);
   }));
 }));

test/parallel/test-fs-buffer.js

@@ -12,7 +12,7 @@ const dstPath = path.join(common.tmpDir, 'link1.js');
 fs.writeFileSync(srcPath, 'hello world');
 
 const callback = function(err) {
-  if (err) throw err;
+  assert.ifError(err);
   const dstContent = fs.readFileSync(dstPath, 'utf8');
   assert.strictEqual('hello world', dstContent);
 };