From d191accd13508648cc61261db250ab88e1f61c8b Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Mon, 25 Nov 2019 23:02:24 -0800
Subject: [PATCH] test: add logging in case of infinite loop

test-inspector-contexts may be entering an infinite loop (or very
long-running loop) in CI, resulting in flakiness. Or maybe not. Add
logging to find out.

PR-URL: https://github.com/nodejs/node/pull/30649
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
---
 test/sequential/test-inspector-contexts.js | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/test/sequential/test-inspector-contexts.js b/test/sequential/test-inspector-contexts.js
index f27f95da99333f..0918d22e90b931 100644
--- a/test/sequential/test-inspector-contexts.js
+++ b/test/sequential/test-inspector-contexts.js
@@ -65,8 +65,10 @@ async function testContextCreatedAndDestroyed() {
                        JSON.stringify(contextCreated));
 
     // GC is unpredictable...
+    console.log('Checking/waiting for GC.');
     while (!contextDestroyed)
       global.gc();
+    console.log('Context destroyed.');
 
     assert.strictEqual(contextDestroyed.params.executionContextId, id,
                        JSON.stringify(contextDestroyed));
@@ -95,8 +97,10 @@ async function testContextCreatedAndDestroyed() {
                        JSON.stringify(contextCreated));
 
     // GC is unpredictable...
+    console.log('Checking/waiting for GC again.');
     while (!contextDestroyed)
       global.gc();
+    console.log('Other context destroyed.');
   }
 
   {
@@ -119,8 +123,10 @@ async function testContextCreatedAndDestroyed() {
                        JSON.stringify(contextCreated));
 
     // GC is unpredictable...
+    console.log('Checking/waiting for GC a third time.');
     while (!contextDestroyed)
       global.gc();
+    console.log('Context destroyed once again.');
   }
 
   {
@@ -141,8 +147,10 @@ async function testContextCreatedAndDestroyed() {
                        JSON.stringify(contextCreated));
 
     // GC is unpredictable...
+    console.log('Checking/waiting for GC a fourth time.');
     while (!contextDestroyed)
       global.gc();
+    console.log('Context destroyed a fourth time.');
   }
 }