test: remove unused config

process.maxTickDepth was removed in v0.12 a whole while ago and was mostly removed from our code base. There are still some places it was left in old benchmarks and tests. This PR removes setting the value in those tests and benchmarks. PR-URL: #21985 Reviewed-By: Anatoli Papirovski <[email protected]> Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Jon Moss <[email protected]>

Author: Benjamin Gruenbaum

benchmark/process/next-tick-depth-args.js

@@ -5,8 +5,6 @@ const bench = common.createBenchmark(main,{
 n: [12e6]
 });
 
-process.maxTickDepth=Infinity;
-
 functionmain({ n }){
 letcounter=n;
 functioncb4(arg1,arg2,arg3,arg4){

benchmark/process/next-tick-depth.js

@@ -4,8 +4,6 @@ const bench = common.createBenchmark(main,{
 n: [12e6]
 });
 
-process.maxTickDepth=Infinity;
-
 functionmain({ n }){
 letcounter=n;
 bench.start();

test/parallel/test-next-tick-intentional-starvation.js

@@ -23,12 +23,8 @@
 require('../common');
 constassert=require('assert');
 
-// this is the inverse of test-next-tick-starvation.
-// it verifies that process.nextTick will *always* come before other
-// events, up to the limit of the process.maxTickDepth value.
-
-// WARNING: unsafe!
-process.maxTickDepth=Infinity;
+// this is the inverse of test-next-tick-starvation. it verifies
+// that process.nextTick will *always* come before other events
 
 letran=false;
 letstarved=false;

test/parallel/test-stream2-read-sync-stack.js

@@ -22,13 +22,13 @@
 'use strict';
 constcommon=require('../common');
 constReadable=require('stream').Readable;
+
+// This tests synchronous read callbacks and verifies that even if they nest
+// heavily the process handles it without an error
+
 constr=newReadable();
 constN=256*1024;
 
-// Go ahead and allow the pathological case for this test.
-// Yes, it's an infinite loop, that's the point.
-process.maxTickDepth=N+2;
-
 letreads=0;
 r._read=function(n){
 constchunk=reads++===N ? null : Buffer.allocUnsafe(1);

test/sequential/test-next-tick-error-spin.js

@@ -39,7 +39,6 @@ if (process.argv[2] !== 'child'){
 
 constdomain=require('domain');
 constd=domain.create();
-process.maxTickDepth=10;
 
 // in the error handler, we trigger several MakeCallback events
 d.on('error',function(){