test: move dgram invalid host test to internet tests

This moves a dgram test from `parallel` to `internet` because it relies
on a DNS request.
In certain cases, ISPs hijack invalid IETF-reserved invalid names which
causes a false negative failure.

Fixes: #27341

PR-URL: #27565
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>

Author: Benjamin Ki

test/internet/test-dgram-connect.js

@@ -0,0 +1,21 @@
+'use strict';
+
+const common = require('../common');
+const { addresses } = require('../common/internet');
+const assert = require('assert');
+const dgram = require('dgram');
+
+const PORT = 12345;
+
+const client = dgram.createSocket('udp4');
+client.connect(PORT, addresses.INVALID_HOST, common.mustCall((err) => {
+  assert.ok(err.code === 'ENOTFOUND' || err.code === 'EAI_AGAIN');
+
+  client.once('error', common.mustCall((err) => {
+    assert.ok(err.code === 'ENOTFOUND' || err.code === 'EAI_AGAIN');
+    client.once('connect', common.mustCall(() => client.close()));
+    client.connect(PORT);
+  }));
+
+  client.connect(PORT, addresses.INVALID_HOST);
+}));

test/parallel/test-dgram-connect.js

@@ -1,7 +1,6 @@
 'use strict';
 
 const common = require('../common');
-const { addresses } = require('../common/internet');
 const assert = require('assert');
 const dgram = require('dgram');
 
@@ -36,17 +35,8 @@ client.connect(PORT, common.mustCall(() => {
     code: 'ERR_SOCKET_DGRAM_NOT_CONNECTED'
   });
 
-  client.connect(PORT, addresses.INVALID_HOST, common.mustCall((err) => {
-    assert.ok(err.code === 'ENOTFOUND' || err.code === 'EAI_AGAIN');
-
-    client.once('error', common.mustCall((err) => {
-      assert.ok(err.code === 'ENOTFOUND' || err.code === 'EAI_AGAIN');
-      client.once('connect', common.mustCall(() => client.close()));
-      client.connect(PORT);
-    }));
-
-    client.connect(PORT, addresses.INVALID_HOST);
-  }));
+  client.once('connect', common.mustCall(() => client.close()));
+  client.connect(PORT);
 }));
 
 assert.throws(() => {