test: use gcUntil() in test-v8-serialize-leak

Previously this can be flaky because the there could be a delay of the deallocation after gc() is invoked. Use gcUntil() to run the GC multiple times to make the test more robust. PR-URL: #49168 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>

Author: joyeecheung

test/parallel/test-v8-serialize-leak.js

@@ -8,22 +8,26 @@ if (common.isIBMi)
 common.skip('On IBMi, the rss memory always returns zero');
 
 constv8=require('v8');
-constassert=require('assert');
 
 constbefore=process.memoryUsage.rss();
 
 for(leti=0;i<1000000;i++){
 v8.serialize('');
 }
 
-global.gc();
-
-constafter=process.memoryUsage.rss();
-
-if(process.config.variables.asan){
-assert(after<before*10,`asan: before=${before} after=${after}`);
-}elseif(process.config.variables.node_builtin_modules_path){
-assert(after<before*4,`node_builtin_modules_path: before=${before} after=${after}`);
-}else{
-assert(after<before*2,`before=${before} after=${after}`);
+asyncfunctionmain(){
+awaitcommon.gcUntil('RSS should go down',()=>{
+constafter=process.memoryUsage.rss();
+if(process.config.variables.asan){
+console.log(`asan: before=${before} after=${after}`);
+returnafter<before*10;
+}elseif(process.config.variables.node_builtin_modules_path){
+console.log(`node_builtin_modules_path: before=${before} after=${after}`);
+returnafter<before*10;
+}
+console.log(`before=${before} after=${after}`);
+returnafter<before*10;
+});
 }
+
+main();