test: use .then(common.mustCall()) for all async IIFEs

This makes sure that all async functions finish as expected. PR-URL: #34363 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

Author: addaleax

test/es-module/test-esm-error-cache.js

@@ -1,6 +1,6 @@
 'use strict';
 
-require('../common');
+const common = require('../common');
 const assert = require('assert');
 
 const file = '../fixtures/syntax/bad_syntax.mjs';
@@ -23,4 +23,4 @@ let error;
  return true;
  }
  );
-})();
+})().then(common.mustCall());

test/es-module/test-esm-import-meta-resolve.mjs

@@ -1,5 +1,5 @@
 // Flags: --experimental-import-meta-resolve
-import '../common/index.mjs';
+import { mustCall } from '../common/index.mjs';
 import assert from 'assert';
 
 const dirname = import.meta.url.slice(0, import.meta.url.lastIndexOf('/') + 1);
@@ -21,4 +21,4 @@ const fixtures = dirname.slice(0, dirname.lastIndexOf('/', dirname.length - 2) +
  assert.strictEqual(await import.meta.resolve('../fixtures/'), fixtures);
  assert.strictEqual(await import.meta.resolve('baz/', fixtures),
  fixtures + 'node_modules/baz/');
-})();
+})().then(mustCall());

test/internet/test-dns-promises-resolve.js

@@ -1,5 +1,5 @@
 'use strict';
-require('../common');
+const common = require('../common');
 const assert = require('assert');
 
 const dnsPromises = require('dns').promises;
@@ -38,5 +38,5 @@ const dnsPromises = require('dns').promises;
  const result = await dnsPromises.resolve('example.org', rrtype);
  assert.ok(result !== undefined);
  assert.ok(result.length > 0);
- })();
+ })().then(common.mustCall());
 }

test/internet/test-dns-txt-sigsegv.js

@@ -1,13 +1,13 @@
 'use strict';
-require('../common');
+const common = require('../common');
 const assert = require('assert');
 const dns = require('dns');
 const dnsPromises = dns.promises;
 
 (async function() {
  const result = await dnsPromises.resolveTxt('www.microsoft.com');
  assert.strictEqual(result.length, 0);
-})();
+})().then(common.mustCall());
 
 dns.resolveTxt('www.microsoft.com', function(err, records) {
  assert.strictEqual(err, null);

test/internet/test-dns.js

@@ -748,4 +748,4 @@ dns.lookupService('0.0.0.0', 0, common.mustCall());
 (async function() {
  await dnsPromises.lookup(addresses.INET6_HOST, 6);
  await dnsPromises.lookup(addresses.INET_HOST, {});
-})();
+})().then(common.mustCall());

test/parallel/test-c-ares.js

@@ -40,7 +40,7 @@ const dnsPromises = dns.promises;
  res = await dnsPromises.lookup('::1');
  assert.strictEqual(res.address, '::1');
  assert.strictEqual(res.family, 6);
-})();
+})().then(common.mustCall());
 
 // Try resolution without hostname.
 dns.lookup(null, common.mustSucceed((result, addressType) => {
@@ -89,5 +89,5 @@ if (!common.isWindows && !common.isIBMi) {
 
  (async function() {
  assert.ok(Array.isArray(await dnsPromises.reverse('127.0.0.1')));
- })();
+ })().then(common.mustCall());
 }

test/parallel/test-child-process-spawn-args.js

@@ -52,4 +52,4 @@ const expectedResult = tmpdir.path.trim().toLowerCase();
  );
 
  assert.deepStrictEqual([...new Set(results)], [expectedResult]);
-})();
+})().then(common.mustCall());

test/parallel/test-dns-lookup-promises.js

@@ -135,5 +135,5 @@ async function lookupallNegative() {
  lookupNegative(),
  lookupallPositive(),
  lookupallNegative()
- ]).then(common.mustCall());
-})();
+ ]);
+})().then(common.mustCall());

test/parallel/test-dns-lookup.js

@@ -110,7 +110,7 @@ assert.throws(() => {
  all: false
  });
  assert.deepStrictEqual(res, { address: '127.0.0.1', family: 4 });
-})();
+})().then(common.mustCall());
 
 dns.lookup(false, {
  hints: 0,

test/parallel/test-dns.js

@@ -274,7 +274,7 @@ dns.lookup('', {
  await dnsPromises.lookup('', {
  hints: dns.ADDRCONFIG | dns.V4MAPPED | dns.ALL
  });
-})();
+})().then(common.mustCall());
 
 {
  const err = {
@@ -447,7 +447,7 @@ assert.throws(() => {
  cases.shift();
  nextCase();
  }));
- })();
+ })().then(common.mustCall());
 
  }));
 }