test: add tests for dnsPromises.lookup

Added tests for dnsPromises.lookup to increase coverage and test `onlookup()` and `onlookupall()` methods. PR-URL: #21559 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
nodejs · Sep 3, 2018 · ae016c8 · ae016c8
1 parent 7794d4e
commit ae016c8
Showing 1 changed file with 25 additions and 0 deletions.
diff --git a/test/parallel/test-dns-lookup.js b/test/parallel/test-dns-lookup.js
@@ -1,5 +1,6 @@
 'use strict';
 const common = require('../common');
+const { addresses } = require('../common/internet');
 const assert = require('assert');
 const cares = process.binding('cares_wrap');
 const dns = require('dns');
@@ -92,6 +93,30 @@ common.expectsError(() => {
     all: false
   });
   assert.deepStrictEqual(res, { address: '127.0.0.1', family: 4 });
+
+  assert.rejects(
+    dnsPromises.lookup(addresses.INVALID_HOST, {
+      hints: 0,
+      family: 0,
+      all: false
+    }),
+    {
+      code: 'ENOTFOUND',
+      message: `getaddrinfo ENOTFOUND ${addresses.INVALID_HOST}`
+    }
+  );
+
+  assert.rejects(
+    dnsPromises.lookup(addresses.INVALID_HOST, {
+      hints: 0,
+      family: 0,
+      all: true
+    }),
+    {
+      code: 'ENOTFOUND',
+      message: `getaddrinfo ENOTFOUND ${addresses.INVALID_HOST}`
+    }
+  );
 })();
 
 dns.lookup(false, {