From aa363c49eadabe45344986476b8388d66cfced37 Mon Sep 17 00:00:00 2001
From: cjihrig <cjihrig@gmail.com>
Date: Wed, 4 Dec 2019 00:17:10 -0500
Subject: test: remove common.busyLoop()

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

PR-URL: https://github.com/nodejs/node/pull/30787
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
---
 test/sequential/test-performance-eventloopdelay.js                  | 5 +++--
 test/sequential/test-timers-block-eventloop.js                      | 4 +++-
 test/sequential/test-timers-blocking-callback.js                    | 4 +++-
 .../test-timers-set-interval-excludes-callback-duration.js          | 6 ++++--
 4 files changed, 13 insertions(+), 6 deletions(-)

(limited to 'test/sequential')

diff --git a/test/sequential/test-performance-eventloopdelay.js b/test/sequential/test-performance-eventloopdelay.js
index 7b9527b386..8e7ee4f0ad 100644
--- a/test/sequential/test-performance-eventloopdelay.js
+++ b/test/sequential/test-performance-eventloopdelay.js
@@ -1,4 +1,4 @@
-// Flags: --expose-gc
+// Flags: --expose-gc --expose-internals
 'use strict';
 
 const common = require('../common');
@@ -6,6 +6,7 @@ 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 {
diff --git a/test/sequential/test-timers-block-eventloop.js b/test/sequential/test-timers-block-eventloop.js
index 811216fcb2..6118695c92 100644
--- a/test/sequential/test-timers-block-eventloop.js
+++ b/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);
diff --git a/test/sequential/test-timers-blocking-callback.js b/test/sequential/test-timers-blocking-callback.js
index 053bc767b8..a5e0f596a3 100644
--- a/test/sequential/test-timers-blocking-callback.js
+++ b/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);
diff --git a/test/sequential/test-timers-set-interval-excludes-callback-duration.js b/test/sequential/test-timers-set-interval-excludes-callback-duration.js
index 90eb16b0e2..b32c7e7e3c 100644
--- a/test/sequential/test-timers-set-interval-excludes-callback-duration.js
+++ b/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();
-- 
cgit v1.2.3