From b226049a4a28ea3f7f32266269fb76340c324d42 Mon Sep 17 00:00:00 2001 From: Vitali Zaidman Date: Thu, 13 Mar 2025 04:22:14 -0700 Subject: [PATCH] Update tests to support new didOpen delegate fn (#50006) Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/50006 Changelog: [Internal] Reviewed By: robhogan, huntie Differential Revision: D71111802 fbshipit-source-id: 733a601007125b6aed4411ba95a41845823a26da --- .../tests/InspectorPackagerConnectionTest.cpp | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/packages/react-native/ReactCommon/jsinspector-modern/tests/InspectorPackagerConnectionTest.cpp b/packages/react-native/ReactCommon/jsinspector-modern/tests/InspectorPackagerConnectionTest.cpp index c9997a05bdd..e0ed7fcdacb 100644 --- a/packages/react-native/ReactCommon/jsinspector-modern/tests/InspectorPackagerConnectionTest.cpp +++ b/packages/react-native/ReactCommon/jsinspector-modern/tests/InspectorPackagerConnectionTest.cpp @@ -16,7 +16,6 @@ #include #include -#include #include #include "FollyDynamicMatchers.h" @@ -41,10 +40,15 @@ class InspectorPackagerConnectionTestBase : public testing::Test { "my-device", "my-app", packagerConnectionDelegates_.make_unique(asyncExecutor_)}) { + auto makeSocket = webSockets_.lazily_make_unique< + const std::string&, + std::weak_ptr>(); ON_CALL(*packagerConnectionDelegate(), connectWebSocket(_, _)) - .WillByDefault(webSockets_.lazily_make_unique< - const std::string&, - std::weak_ptr>()); + .WillByDefault([makeSocket](auto&&... args) { + auto socket = makeSocket(std::forward(args)...); + socket->getDelegate().didOpen(); + return std::move(socket); + }); } void TearDown() override {