Fix memory leak in TestCallInvoker (#53287)

Summary:
Pull Request resolved: #53287

Changelog: [General][Fixed] Fix memory leak in TestCallInvoker

This fixes leaks in TestCallInvoker holding onto the jsi::Runtime

Reviewed By: lenaic

Differential Revision: D80295420

fbshipit-source-id: b14368ccfa86b3bf24b1f84613ec07931bd71a43

Author: christophpurrer

packages/react-native/ReactCommon/callinvoker/ReactCommon/tests/TestCallInvoker.h

@@ -10,28 +10,27 @@
 #include <ReactCommon/CallInvoker.h>
 #include <jsi/jsi.h>
 #include <list>
-#include <memory>
 
 namespace facebook::react {
 
 class TestCallInvoker : public CallInvoker {
  public:
-  explicit TestCallInvoker(std::shared_ptr<facebook::jsi::Runtime> runtime)
+  explicit TestCallInvoker(facebook::jsi::Runtime& runtime)
       : runtime_(runtime) {}
 
   void invokeAsync(CallFunc&& func) noexcept override {
     queue_.push_back(std::move(func));
   }
 
   void invokeSync(CallFunc&& func) override {
-    func(*runtime_);
+    func(runtime_);
   }
 
   void flushQueue() {
     while (!queue_.empty()) {
-      queue_.front()(*runtime_);
+      queue_.front()(runtime_);
       queue_.pop_front();
-      runtime_->drainMicrotasks(); // Run microtasks every cycle.
+      runtime_.drainMicrotasks(); // Run microtasks every cycle.
     }
   }
 
@@ -40,8 +39,8 @@ class TestCallInvoker : public CallInvoker {
   }
 
  private:
+  facebook::jsi::Runtime& runtime_;
   std::list<CallFunc> queue_{};
-  std::shared_ptr<facebook::jsi::Runtime> runtime_{};
 };
 
 } // namespace facebook::react

packages/react-native/ReactCommon/react/bridging/tests/BridgingTest.h

@@ -31,7 +31,7 @@ class BridgingTest : public ::testing::Test {
                 .withMicrotaskQueue(true)
                 .build())),
         rt(*runtime),
-        invoker(std::make_shared<TestCallInvoker>(runtime)) {}
+        invoker(std::make_shared<TestCallInvoker>(*runtime)) {}
 
   ~BridgingTest() override {
     LongLivedObjectCollection::get(rt).clear();

packages/react-native/ReactCommon/react/nativemodule/core/tests/TurboModuleTestFixture.h

@@ -27,7 +27,7 @@ class TurboModuleTestFixture : public ::testing::Test {
  public:
   explicit TurboModuleTestFixture(Args... args)
       : runtime_(hermes::makeHermesRuntime()),
-        jsInvoker_(std::make_shared<TestCallInvoker>(runtime_)),
+        jsInvoker_(std::make_shared<TestCallInvoker>(*runtime_)),
         module_(std::make_shared<T>(jsInvoker_, std::forward<Args>(args)...)) {}
 
   void SetUp() override {

packages/react-native/ReactCxxPlatform/react/io/tests/NetworkingModuleTests.cpp

@@ -25,7 +25,7 @@ class NetworkingModuleTests : public testing::Test {
  protected:
   void SetUp() override {
     rt_ = facebook::hermes::makeHermesRuntime();
-    jsInvoker_ = std::make_shared<TestCallInvoker>(rt_);
+    jsInvoker_ = std::make_shared<TestCallInvoker>(*rt_);
   }
 
   static void verifyFormData(

packages/rn-tester/NativeCxxModuleExample/tests/NativeCxxModuleExampleTests.cpp

@@ -6,7 +6,6 @@
  */
 
 #include <NativeCxxModuleExample/NativeCxxModuleExample.h>
-#include <ReactCommon/TestCallInvoker.h>
 #include <ReactCommon/TurboModuleTestFixture.h>
 #include <gtest/gtest.h>
 #include <list>