react-native-tvos 0.82.1-0 → 0.83.0-0

package/ReactCommon/jsinspector-modern/tests/InspectorPackagerConnectionTest.cpp

@@ -35,11 +35,12 @@ template <typename Executor>
 class InspectorPackagerConnectionTestBase : public testing::Test {
  protected:
   InspectorPackagerConnectionTestBase()
-      : packagerConnection_(InspectorPackagerConnection{
-            "ws://mock-host:12345",
-            "my-device",
-            "my-app",
-            packagerConnectionDelegates_.make_unique(asyncExecutor_)}) {
+      : packagerConnection_(
+            InspectorPackagerConnection{
+                "ws://mock-host:12345",
+                "my-device",
+                "my-app",
+                packagerConnectionDelegates_.make_unique(asyncExecutor_)}) {
     auto makeSocket = webSockets_.lazily_make_unique<
         const std::string&,
         std::weak_ptr<IWebSocketDelegate>>();
@@ -49,6 +50,8 @@ class InspectorPackagerConnectionTestBase : public testing::Test {
           socket->getDelegate().didOpen();
           return std::move(socket);
         });
+    EXPECT_CALL(*packagerConnectionDelegate(), connectWebSocket(_, _))
+        .Times(AnyNumber());
   }
 
   void TearDown() override {
@@ -59,7 +62,7 @@ class InspectorPackagerConnectionTestBase : public testing::Test {
     auto pages = getInspectorInstance().getPages();
     int liveConnectionCount = 0;
     for (size_t i = 0; i != localConnections_.objectsVended(); ++i) {
-      if (localConnections_[i]) {
+      if (localConnections_[i] != nullptr) {
         liveConnectionCount++;
         // localConnections_[i] is a strict mock and will complain when we
         // removePage if the call is unexpected.
@@ -69,7 +72,7 @@ class InspectorPackagerConnectionTestBase : public testing::Test {
     for (auto& page : pages) {
       getInspectorInstance().removePage(page.id);
     }
-    if (!pages.empty() && liveConnectionCount) {
+    if (!pages.empty() && (liveConnectionCount != 0)) {
       if (!::testing::Test::HasFailure()) {
         FAIL()
             << "Test case ended with " << liveConnectionCount
@@ -281,14 +284,15 @@ TEST_F(InspectorPackagerConnectionTest, TestSendReceiveEvents) {
           .lazily_make_unique<std::unique_ptr<IRemoteConnection>>());
 
   // Connect to the page.
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "connect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   ASSERT_TRUE(localConnections_[0]);
 
   // Send an event from the mocked backend (local) to the frontend (remote)
@@ -321,16 +325,17 @@ TEST_F(InspectorPackagerConnectionTest, TestSendReceiveEvents) {
           AtJsonPtr("/params", ElementsAre("arg1", "arg2"))))))
       .RetiresOnSaturation();
 
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "wrappedEvent",
           "payload": {{
             "pageId": {0},
             "wrappedEvent": {1}
           }}
         }})",
-      toJson(std::to_string(pageId)),
-      toJson(R"({
+          toJson(std::to_string(pageId)),
+          toJson(R"({
                 "method": "FakeDomain.fakeMethod",
                 "id": 1234,
                 "params": ["arg1", "arg2"]
@@ -360,11 +365,12 @@ TEST_F(InspectorPackagerConnectionTest, TestSendReceiveEventsToMultiplePages) {
 
   const int kNumPages = 2;
   for (int i = 0; i < kNumPages; ++i) {
-    pageIds.push_back(getInspectorInstance().addPage(
-        "mock-description",
-        "mock-vm",
-        localConnections_
-            .lazily_make_unique<std::unique_ptr<IRemoteConnection>>()));
+    pageIds.push_back(
+        getInspectorInstance().addPage(
+            "mock-description",
+            "mock-vm",
+            localConnections_
+                .lazily_make_unique<std::unique_ptr<IRemoteConnection>>()));
     if (i > 0) {
       ASSERT_NE(pageIds[i], pageIds[i - 1])
           << "Received duplicate page IDs from inspector.";
@@ -373,14 +379,15 @@ TEST_F(InspectorPackagerConnectionTest, TestSendReceiveEventsToMultiplePages) {
 
   for (int i = 0; i < kNumPages; ++i) {
     // Connect to the i-th page.
-    webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-        R"({{
+    webSockets_[0]->getDelegate().didReceiveMessage(
+        fmt::format(
+            R"({{
         "event": "connect",
         "payload": {{
           "pageId": {0}
         }}
       }})",
-        toJson(std::to_string(pageIds[i]))));
+            toJson(std::to_string(pageIds[i]))));
     ASSERT_TRUE(localConnections_[i]);
   }
 
@@ -415,16 +422,17 @@ TEST_F(InspectorPackagerConnectionTest, TestSendReceiveEventsToMultiplePages) {
         *localConnections_[i],
         sendMessage(JsonParsed(AtJsonPtr("/method", Eq(method)))))
         .RetiresOnSaturation();
-    webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-        R"({{
+    webSockets_[0]->getDelegate().didReceiveMessage(
+        fmt::format(
+            R"({{
           "event": "wrappedEvent",
           "payload": {{
             "pageId": {0},
             "wrappedEvent": {1}
           }}
         }})",
-        toJson(std::to_string(pageIds[i])),
-        toJson(toJson(dynamic::object("method", method)))));
+            toJson(std::to_string(pageIds[i])),
+            toJson(toJson(dynamic::object("method", method)))));
   }
 
   for (int i = 0; i < kNumPages; ++i) {
@@ -445,14 +453,15 @@ TEST_F(InspectorPackagerConnectionTest, TestSendEventToAllConnections) {
           .lazily_make_unique<std::unique_ptr<IRemoteConnection>>());
 
   // Connect to the page.
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "connect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   ASSERT_TRUE(localConnections_[0]);
 
   // Impersonate the frontend (remote) to send a message to all (local)
@@ -486,26 +495,28 @@ TEST_F(InspectorPackagerConnectionTest, TestConnectThenDisconnect) {
           .lazily_make_unique<std::unique_ptr<IRemoteConnection>>());
 
   // Connect to the page.
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "connect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   ASSERT_TRUE(localConnections_[0]);
 
   // Disconnect from the page.
   EXPECT_CALL(*localConnections_[0], disconnect()).RetiresOnSaturation();
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "disconnect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   EXPECT_FALSE(localConnections_[0]);
 }
 
@@ -521,14 +532,15 @@ TEST_F(InspectorPackagerConnectionTest, TestConnectThenCloseSocket) {
           .lazily_make_unique<std::unique_ptr<IRemoteConnection>>());
 
   // Connect to the page.
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "connect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   ASSERT_TRUE(localConnections_[0]);
 
   // Notify that the socket was closed.
@@ -549,14 +561,15 @@ TEST_F(InspectorPackagerConnectionTest, TestConnectThenSocketFailure) {
           .lazily_make_unique<std::unique_ptr<IRemoteConnection>>());
 
   // Connect to the page.
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "connect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   ASSERT_TRUE(localConnections_[0]);
 
   // Notify that the socket was closed (implicitly, as the result of an error).
@@ -579,14 +592,15 @@ TEST_F(
           .lazily_make_unique<std::unique_ptr<IRemoteConnection>>());
 
   // Connect to the page.
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "connect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   ASSERT_TRUE(localConnections_[0]);
 
   // Notify that the socket was closed (implicitly, as the result of an error).
@@ -625,26 +639,28 @@ TEST_F(
           .lazily_make_unique<std::unique_ptr<IRemoteConnection>>());
 
   // Connect to the page.
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "connect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   ASSERT_TRUE(localConnections_[0]);
 
   // Try connecting to the same page again. This results in a disconnection.
   EXPECT_CALL(*localConnections_[0], disconnect()).RetiresOnSaturation();
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "connect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   EXPECT_FALSE(localConnections_[0]);
 }
 
@@ -660,37 +676,40 @@ TEST_F(InspectorPackagerConnectionTest, TestMultipleDisconnect) {
           .lazily_make_unique<std::unique_ptr<IRemoteConnection>>());
 
   // Connect to the page.
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "connect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   ASSERT_TRUE(localConnections_[0]);
 
   // Disconnect from the page.
   EXPECT_CALL(*localConnections_[0], disconnect()).RetiresOnSaturation();
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "disconnect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   EXPECT_FALSE(localConnections_[0]);
 
   // Disconnect again. This is a noop.
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "disconnect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   EXPECT_FALSE(localConnections_[0]);
 }
 
@@ -706,40 +725,43 @@ TEST_F(InspectorPackagerConnectionTest, TestDisconnectThenSendEvent) {
           .lazily_make_unique<std::unique_ptr<IRemoteConnection>>());
 
   // Connect to the page.
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "connect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   ASSERT_TRUE(localConnections_[0]);
 
   // Disconnect from the page.
   EXPECT_CALL(*localConnections_[0], disconnect()).RetiresOnSaturation();
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "disconnect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   EXPECT_FALSE(localConnections_[0]);
 
   // Send an event from the frontend (remote) to the backend (local). This
   // is a noop.
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "wrappedEvent",
           "payload": {{
             "pageId": {0},
             "wrappedEvent": {1}
           }}
         }})",
-      toJson(std::to_string(pageId)),
-      toJson(R"({
+          toJson(std::to_string(pageId)),
+          toJson(R"({
                 "method": "FakeDomain.fakeMethod",
                 "id": 1234,
                 "params": ["arg1", "arg2"]
@@ -754,15 +776,16 @@ TEST_F(InspectorPackagerConnectionTest, TestSendEventToUnknownPage) {
 
   // Send an event from the frontend (remote) to the backend (local). This
   // is a noop (except for logging).
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "wrappedEvent",
           "payload": {{
             "pageId": "1234",
             "wrappedEvent": {0}
           }}
         }})",
-      toJson(R"({
+          toJson(R"({
                 "method": "FakeDomain.fakeMethod",
                 "id": 1234,
                 "params": ["arg1", "arg2"]
@@ -921,14 +944,15 @@ TEST_F(
           .lazily_make_unique<std::unique_ptr<IRemoteConnection>>());
 
   // Connect to the page.
-  retainedWebSocketDelegate->didReceiveMessage(fmt::format(
-      R"({{
+  retainedWebSocketDelegate->didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "connect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   ASSERT_TRUE(localConnections_[0]);
 
   // Send an event from the frontend (remote) to the backend (local) and
@@ -941,16 +965,17 @@ TEST_F(
           AtJsonPtr("/params", ElementsAre("arg1", "arg2"))))))
       .RetiresOnSaturation();
 
-  retainedWebSocketDelegate->didReceiveMessage(fmt::format(
-      R"({{
+  retainedWebSocketDelegate->didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "wrappedEvent",
           "payload": {{
             "pageId": {0},
             "wrappedEvent": {1}
           }}
         }})",
-      toJson(std::to_string(pageId)),
-      toJson(R"({
+          toJson(std::to_string(pageId)),
+          toJson(R"({
                 "method": "FakeDomain.fakeMethod",
                 "id": 1234,
                 "params": ["arg1", "arg2"]
@@ -975,14 +1000,15 @@ TEST_F(InspectorPackagerConnectionTest, TestDestroyConnectionOnPageRemoved) {
           .lazily_make_unique<std::unique_ptr<IRemoteConnection>>());
 
   // Connect to the page.
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "connect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   EXPECT_TRUE(localConnections_[0]);
 
   // Remove the page.
@@ -1005,14 +1031,15 @@ TEST_F(
           .lazily_make_unique<std::unique_ptr<IRemoteConnection>>());
 
   // Connect to the page.
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "connect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   ASSERT_TRUE(localConnections_[0]);
 
   // Send an event from the mocked backend (local) to the frontend (remote)
@@ -1045,14 +1072,15 @@ TEST_F(
           .lazily_make_unique<std::unique_ptr<IRemoteConnection>>());
 
   // Connect to the page.
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "connect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   ASSERT_TRUE(localConnections_[0]);
 
   // Send an event from the mocked backend (local) to the frontend (remote)
@@ -1064,25 +1092,27 @@ TEST_F(
 
   // Disconnect from the page.
   EXPECT_CALL(*localConnections_[0], disconnect()).RetiresOnSaturation();
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "disconnect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   EXPECT_FALSE(localConnections_[0]);
 
   // Connect to the same page again.
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "connect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
 
   EXPECT_TRUE(localConnections_[1]);
 
@@ -1127,14 +1157,15 @@ TEST_F(
           .lazily_make_unique<std::unique_ptr<IRemoteConnection>>());
 
   // Connect to the page.
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "connect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   ASSERT_TRUE(localConnections_[0]);
 
   // Send an event from the mocked backend (local) to the frontend (remote)
@@ -1151,25 +1182,27 @@ TEST_F(
 
   // Disconnect from the page.
   EXPECT_CALL(*localConnections_[0], disconnect()).RetiresOnSaturation();
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "disconnect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
   EXPECT_FALSE(localConnections_[0]);
 
   // Connect to the same page again.
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "connect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
 
   EXPECT_TRUE(localConnections_[1]);
 
@@ -1256,25 +1289,27 @@ TEST_F(InspectorPackagerConnectionTest, TestRejectedPageConnection) {
           AtJsonPtr("/payload/pageId", Eq(std::to_string(pageId)))))))
       .RetiresOnSaturation();
 
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "connect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
 
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "wrappedEvent",
           "payload": {{
             "pageId": {0},
             "wrappedEvent": {1}
           }}
         }})",
-      toJson(std::to_string(pageId)),
-      toJson(R"({
+          toJson(std::to_string(pageId)),
+          toJson(R"({
                 "method": "FakeDomain.fakeMethod",
                 "id": 1,
                 "params": ["arg1", "arg2"]
@@ -1291,25 +1326,27 @@ TEST_F(InspectorPackagerConnectionTest, TestRejectedPageConnection) {
           AtJsonPtr("/payload/pageId", Eq(std::to_string(pageId)))))))
       .RetiresOnSaturation();
 
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "connect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
 
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "wrappedEvent",
           "payload": {{
             "pageId": {0},
             "wrappedEvent": {1}
           }}
         }})",
-      toJson(std::to_string(pageId)),
-      toJson(R"({
+          toJson(std::to_string(pageId)),
+          toJson(R"({
                 "method": "FakeDomain.fakeMethod",
                 "id": 2,
                 "params": ["arg1", "arg2"]
@@ -1319,14 +1356,15 @@ TEST_F(InspectorPackagerConnectionTest, TestRejectedPageConnection) {
   // page.
   mockNextConnectionBehavior = Accept;
 
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "connect",
           "payload": {{
             "pageId": {0}
           }}
         }})",
-      toJson(std::to_string(pageId))));
+          toJson(std::to_string(pageId))));
 
   EXPECT_CALL(
       *localConnections_[0],
@@ -1336,16 +1374,17 @@ TEST_F(InspectorPackagerConnectionTest, TestRejectedPageConnection) {
           AtJsonPtr("/params", ElementsAre("arg1", "arg2"))))))
       .RetiresOnSaturation();
 
-  webSockets_[0]->getDelegate().didReceiveMessage(fmt::format(
-      R"({{
+  webSockets_[0]->getDelegate().didReceiveMessage(
+      fmt::format(
+          R"({{
           "event": "wrappedEvent",
           "payload": {{
             "pageId": {0},
             "wrappedEvent": {1}
           }}
         }})",
-      toJson(std::to_string(pageId)),
-      toJson(R"({
+          toJson(std::to_string(pageId)),
+          toJson(R"({
                 "method": "FakeDomain.fakeMethod",
                 "id": 3,
                 "params": ["arg1", "arg2"]