From 7cfd686214c378349e64cbd10521d7e1a4abbed3 Mon Sep 17 00:00:00 2001 From: Moti Zilberman Date: Wed, 28 Feb 2024 10:03:15 -0800 Subject: [PATCH] Use standard (lowercase) `assert` in ReactInstanceIntegrationTest Summary: Changelog: [Internal] TSIA; `ASSERT` isn't universally defined so let's switch to `assert`. bypass-github-export-checks Reviewed By: huntie Differential Revision: D54305583 fbshipit-source-id: ee80b462f1372e53301f1bb66eb589e4f7da89c6 --- .../tests/ReactInstanceIntegrationTest.h | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/packages/react-native/ReactCommon/jsinspector-modern/tests/ReactInstanceIntegrationTest.h b/packages/react-native/ReactCommon/jsinspector-modern/tests/ReactInstanceIntegrationTest.h index b95500f609c..34710c6d5c9 100644 --- a/packages/react-native/ReactCommon/jsinspector-modern/tests/ReactInstanceIntegrationTest.h +++ b/packages/react-native/ReactCommon/jsinspector-modern/tests/ReactInstanceIntegrationTest.h @@ -7,15 +7,17 @@ #pragma once +#include "InspectorMocks.h" +#include "ReactNativeMocks.h" +#include "UniquePtrFactory.h" + #include #include #include #include -#include -#include "InspectorMocks.h" -#include "ReactNativeMocks.h" -#include "UniquePtrFactory.h" +#include +#include namespace facebook::react::jsinspector_modern { @@ -49,7 +51,7 @@ class ReactInstanceIntegrationTest : public Test { MockRemoteConnection& getRemoteConnection() { EXPECT_EQ(mockRemoteConnections_.objectsVended(), 1); auto rawPtr = mockRemoteConnections_[0]; - ASSERT(rawPtr != nullptr); + assert(rawPtr); return *rawPtr; }