From 514dc4a8ded804cdaad2827f11fb9e1cab4b8058 Mon Sep 17 00:00:00 2001
From: Anna Henningsen <anna@addaleax.net>
Date: Mon, 30 Mar 2020 11:01:07 +0200
Subject: [PATCH] test: wait for message from parent in embedding cctest
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

I’ve seen this fail a few times in CI, presumably because the
inspector commmand did not reach the child thread in time.
Explicitly waiting seems to solve that.

Refs: https://github.com/nodejs/node/pull/30467
---
 test/cctest/test_environment.cc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/test/cctest/test_environment.cc b/test/cctest/test_environment.cc
index 23b37ee3c60ae4..e53fe4ed5e923d 100644
--- a/test/cctest/test_environment.cc
+++ b/test/cctest/test_environment.cc
@@ -406,6 +406,7 @@ TEST_F(EnvironmentTest, InspectorMultipleEmbeddedEnvironments) {
 
       v8::Local<v8::Value> extracted_value = LoadEnvironment(
           environment,
+          "while (!global.variableFromParent) {}\n"
           "return global.variableFromParent;",
           std::move(data->inspector_parent_handle)).ToLocalChecked();