test: remove common.busyLoop()

This commit replaces common.busyLoop() with sleep().

PR-URL: #30787
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Gus Caplan <[email protected]>
Reviewed-By: Richard Lau <[email protected]>

Author: cjihrig

test/common/README.md

@@ -45,11 +45,6 @@ tasks.
 
 Takes `whitelist` and concats that with predefined `knownGlobals`.
 
-### busyLoop(time)
-* `time` [<number>][]
-
-Blocks for `time` amount of time.
-
 ### canCreateSymLink()
 * return [<boolean>][]
 

test/common/index.js

@@ -464,12 +464,6 @@ function nodeProcessAborted(exitCode, signal) {
   }
 }
 
-function busyLoop(time) {
-  const startTime = Date.now();
-  const stopTime = startTime + time;
-  while (Date.now() < stopTime) {}
-}
-
 function isAlive(pid) {
   try {
     process.kill(pid, 'SIGCONT');
@@ -727,7 +721,6 @@ function runWithInvalidFD(func) {
 module.exports = {
   allowGlobals,
   buildType,
-  busyLoop,
   canCreateSymLink,
   childShouldThrowAndAbort,
   createZeroFilledFile,

test/common/index.mjs

@@ -40,7 +40,6 @@ const {
   skip,
   ArrayStream,
   nodeProcessAborted,
-  busyLoop,
   isAlive,
   expectWarning,
   expectsError,
@@ -87,7 +86,6 @@ export {
   skip,
   ArrayStream,
   nodeProcessAborted,
-  busyLoop,
   isAlive,
   expectWarning,
   expectsError,

test/parallel/test-timers-nested.js

@@ -1,7 +1,9 @@
+// Flags: --expose-internals
 'use strict';
 
-const common = require('../common');
+require('../common');
 const assert = require('assert');
+const { sleep } = require('internal/util');
 
 // Make sure we test 0ms timers, since they would had always wanted to run on
 // the current tick, and greater than 0ms timers, for scenarios where the
@@ -23,7 +25,7 @@ scenarios.forEach(function(delay) {
 
     // Busy loop for the same timeout used for the nested timer to ensure that
     // we are in fact expiring the nested timer.
-    common.busyLoop(delay);
+    sleep(delay);
 
     // The purpose of running this assert in nextTick is to make sure it runs
     // after A but before the next iteration of the libuv event loop.

test/parallel/test-timers-next-tick.js

@@ -1,6 +1,8 @@
+// Flags: --expose-internals
 'use strict';
 
 const common = require('../common');
+const { sleep } = require('internal/util');
 
 // This test verifies that the next tick queue runs after each
 // individual Timeout, as well as each individual Immediate.
@@ -16,7 +18,7 @@ const t2 = setTimeout(common.mustNotCall(), 1);
 const t3 = setTimeout(common.mustNotCall(), 1);
 setTimeout(common.mustCall(), 1);
 
-common.busyLoop(5);
+sleep(5);
 
 setImmediate(common.mustCall(() => {
   process.nextTick(() => {

test/sequential/test-performance-eventloopdelay.js

@@ -1,11 +1,12 @@
-// Flags: --expose-gc
+// Flags: --expose-gc --expose-internals
 'use strict';
 
 const common = require('../common');
 const assert = require('assert');
 const {
   monitorEventLoopDelay
 } = require('perf_hooks');
+const { sleep } = require('internal/util');
 
 {
   const histogram = monitorEventLoopDelay();
@@ -54,7 +55,7 @@ const {
   histogram.enable();
   let m = 5;
   function spinAWhile() {
-    common.busyLoop(1000);
+    sleep(1000);
     if (--m > 0) {
       setTimeout(spinAWhile, common.platformTimeout(500));
     } else {

test/sequential/test-timers-block-eventloop.js

@@ -1,7 +1,9 @@
+// Flags: --expose-internals
 'use strict';
 
 const common = require('../common');
 const assert = require('assert');
+const { sleep } = require('internal/util');
 
 let called = false;
 const t1 = setInterval(() => {
@@ -14,5 +16,5 @@ const t1 = setInterval(() => {
 }, 10);
 
 const t2 = setInterval(() => {
-  common.busyLoop(20);
+  sleep(20);
 }, 10);

test/sequential/test-timers-blocking-callback.js

@@ -1,3 +1,4 @@
+// Flags: --expose-internals
 'use strict';
 
 /*
@@ -25,6 +26,7 @@
 
 const common = require('../common');
 const assert = require('assert');
+const { sleep } = require('internal/util');
 
 const TIMEOUT = 100;
 
@@ -65,7 +67,7 @@ function blockingCallback(retry, callback) {
       return callback();
   } else {
     // Block by busy-looping to trigger the issue
-    common.busyLoop(TIMEOUT);
+    sleep(TIMEOUT);
 
     timeCallbackScheduled = Date.now();
     setTimeout(blockingCallback.bind(null, retry, callback), TIMEOUT);

test/sequential/test-timers-set-interval-excludes-callback-duration.js

@@ -1,13 +1,15 @@
+// Flags: --expose-internals
 'use strict';
-const common = require('../common');
+require('../common');
 const assert = require('assert');
+const { sleep } = require('internal/util');
 
 let cntr = 0;
 let first;
 const t = setInterval(() => {
   cntr++;
   if (cntr === 1) {
-    common.busyLoop(100);
+    sleep(100);
     // Ensure that the event loop passes before the second interval
     setImmediate(() => assert.strictEqual(cntr, 1));
     first = Date.now();