test: add dns.onlookupall() to increase coverage

Added test that callback should be called when error occurs in dns.lookupall(). PR-URL: #22985 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
nodejs · Sep 27, 2018 · 3e4af49 · 3e4af49
1 parent d68dfa9
commit 3e4af49
Showing 1 changed file with 17 additions and 1 deletion.
diff --git a/test/internet/test-dns-lookup.js b/test/internet/test-dns-lookup.js
@@ -1,7 +1,9 @@
 'use strict';
 
 require('../common');
-const dnsPromises = require('dns').promises;
+const common = require('../common');
+const dns = require('dns');
+const dnsPromises = dns.promises;
 const { addresses } = require('../common/internet');
 const assert = require('assert');
 
@@ -28,3 +30,17 @@ assert.rejects(
     message: `getaddrinfo ENOTFOUND ${addresses.INVALID_HOST}`
   }
 );
+
+dns.lookup(addresses.INVALID_HOST, {
+  hints: 0,
+  family: 0,
+  all: true
+}, common.mustCall((error) => {
+  assert.strictEqual(error.code, 'ENOTFOUND');
+  assert.strictEqual(
+    error.message,
+    `getaddrinfo ENOTFOUND ${addresses.INVALID_HOST}`
+  );
+  assert.strictEqual(error.syscall, 'getaddrinfo');
+  assert.strictEqual(error.hostname, addresses.INVALID_HOST);
+}));