test: update test-aborted-util to use node:test

PR-URL: #54578
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

Author: jasnell

test/parallel/test-aborted-util.js

@@ -1,59 +1,94 @@
 // Flags: --expose-gc
 'use strict';
 
-constcommon=require('../common');
+require('../common');
 const{ aborted }=require('util');
-constassert=require('assert');
+const{
+ match,
+ rejects,
+ strictEqual,
+}=require('assert');
 const{ getEventListeners }=require('events');
-const{spawn}=require('child_process');
+const{inspect}=require('util');
 
-{
-// Test aborted works when provided a resource
+const{
+ test,
+}=require('node:test');
+
+test('Aborted works when provided a resource',async()=>{
 constac=newAbortController();
-aborted(ac.signal,{}).then(common.mustCall());
+constpromise=aborted(ac.signal,{});
 ac.abort();
-assert.strictEqual(ac.signal.aborted,true);
-assert.strictEqual(getEventListeners(ac.signal,'abort').length,0);
-}
+awaitpromise;
+strictEqual(ac.signal.aborted,true);
+strictEqual(getEventListeners(ac.signal,'abort').length,0);
+});
 
-{
+test('Aborted with gc cleanup',async()=>{
 // Test aborted with gc cleanup
 constac=newAbortController();
-aborted(ac.signal,{}).then(common.mustNotCall());
+
+constabortedPromise=aborted(ac.signal,{});
+const{ promise, resolve }=Promise.withResolvers();
+
 setImmediate(()=>{
 global.gc();
 ac.abort();
-assert.strictEqual(ac.signal.aborted,true);
-assert.strictEqual(getEventListeners(ac.signal,'abort').length,0);
+strictEqual(ac.signal.aborted,true);
+strictEqual(getEventListeners(ac.signal,'abort').length,0);
+resolve();
 });
-}
 
-{
-// Fails with error if not provided abort signal
-Promise.all([{},null,undefined,Symbol(),[],1,0,1n,true,false,'a',()=>{}].map((sig)=>
-assert.rejects(aborted(sig,{}),{
+awaitpromise;
+
+// Ensure that the promise is still pending
+match(inspect(abortedPromise),/<pending>/);
+});
+
+test('Fails with error if not provided AbortSignal',async()=>{
+awaitPromise.all([{},null,undefined,Symbol(),[],1,0,1n,true,false,'a',()=>{}].map((sig)=>
+rejects(aborted(sig,{}),{
 code: 'ERR_INVALID_ARG_TYPE',
 })
-)).then(common.mustCall());
-}
+));
+});
 
-{
+test('Fails if not provided a resource',async()=>{
 // Fails if not provided a resource
 constac=newAbortController();
-Promise.all([null,undefined,0,1,0n,1n,Symbol(),'','a'].map((resource)=>
-assert.rejects(aborted(ac.signal,resource),{
+awaitPromise.all([null,undefined,0,1,0n,1n,Symbol(),'','a'].map((resource)=>
+rejects(aborted(ac.signal,resource),{
 code: 'ERR_INVALID_ARG_TYPE',
 })
-)).then(common.mustCall());
+));
+});
+
+// To allow this case to be more flexibly tested on runtimes that do not have
+// child_process.spawn, we lazily require it and skip the test if it is not
+// present.
+
+letspawn;
+functionlazySpawn(){
+if(spawn===undefined){
+try{
+spawn=require('child_process').spawn;
+}catch{
+// Ignore if spawn does not exist.
+}
+}
+returnspawn;
 }
 
-{
+test('Does not hang forever',{skip: !lazySpawn()},async()=>{
+const{ promise, resolve }=Promise.withResolvers();
 constchildProcess=spawn(process.execPath,['--input-type=module']);
-childProcess.on('exit',common.mustCall((code)=>{
-assert.strictEqual(code,13);
-}));
+childProcess.on('exit',(code)=>{
+strictEqual(code,13);
+resolve();
+});
 childProcess.stdin.end(`
  import { aborted } from 'node:util';
  await aborted(new AbortController().signal, {});
  `);
-}
+awaitpromise;
+});