test: use checkIfCollectableByCounting in SourceTextModule leak test

...which may be more reliable than than checkIfCollectable(). PR-URL: #51512 Refs: #51362 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Michaël Zasso <[email protected]>

Author: joyeecheung

test/es-module/test-vm-source-text-module-leak.js

@@ -1,21 +1,24 @@
-// Flags: --experimental-vm-modules --max-old-space-size=16 --trace-gc
+// Flags: --expose-internals --experimental-vm-modules --max-old-space-size=16 --trace-gc
 'use strict';
 
 // This tests that vm.SourceTextModule() does not leak.
 // See: https://github.com/nodejs/node/issues/33439
-require('../common');
-const{checkIfCollectable}=require('../common/gc');
+constcommon=require('../common');
+const{checkIfCollectableByCounting}=require('../common/gc');
 constvm=require('vm');
 
-asyncfunctioncreateSourceTextModule(){
-// Try to reach the maximum old space size.
-constm=newvm.SourceTextModule(`
- const bar = new Array(512).fill("----");
- export{bar };
- `);
-awaitm.link(()=>{});
-awaitm.evaluate();
-returnm;
-}
+constouter=32;
+constinner=128;
 
-checkIfCollectable(createSourceTextModule,4096,1024);
+checkIfCollectableByCounting(async(i)=>{
+for(letj=0;j<inner;j++){
+// Try to reach the maximum old space size.
+constm=newvm.SourceTextModule(`
+ const bar = new Array(512).fill("----");
+ export{bar };
+ `);
+awaitm.link(()=>{});
+awaitm.evaluate();
+}
+returninner;
+},vm.SourceTextModule,outer).then(common.mustCall());