diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactContext.java b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactContext.java index f1f1c83dffc..d247714c51e 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactContext.java +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/ReactContext.java @@ -547,14 +547,6 @@ public class ReactContext extends ContextWrapper { return null; } - public @Nullable JSIModule getJSIModule(JSIModuleType moduleType) { - if (!hasActiveReactInstance()) { - throw new IllegalStateException( - "Unable to retrieve a JSIModule if CatalystInstance is not active."); - } - return mCatalystInstance.getJSIModule(moduleType); - } - @DeprecatedInNewArchitecture( message = "This method will be deprecated later as part of Stable APIs with bridge removal and not encouraged usage.") diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/BridgelessReactContext.java b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/BridgelessReactContext.java index 6e1994a3804..edaebd1cd01 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/BridgelessReactContext.java +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/runtime/BridgelessReactContext.java @@ -12,8 +12,6 @@ import com.facebook.infer.annotation.Nullsafe; import com.facebook.react.bridge.Arguments; import com.facebook.react.bridge.Callback; import com.facebook.react.bridge.CatalystInstance; -import com.facebook.react.bridge.JSIModule; -import com.facebook.react.bridge.JSIModuleType; import com.facebook.react.bridge.JavaScriptModule; import com.facebook.react.bridge.JavaScriptModuleRegistry; import com.facebook.react.bridge.NativeArray; @@ -75,16 +73,6 @@ class BridgelessReactContext extends ReactApplicationContext implements EventDis return mSourceURL.get(); } - @Override - public @Nullable JSIModule getJSIModule(JSIModuleType moduleType) { - if (moduleType == JSIModuleType.UIManager) { - return mReactHost.getUIManager(); - } - throw new UnsupportedOperationException( - "getJSIModule is not implemented for bridgeless mode. Trying to get module: " - + moduleType.name()); - } - @Override public @Nullable UIManager getFabricUIManager() { return mReactHost.getUIManager(); diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/ThemedReactContext.java b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/ThemedReactContext.java index eded794637a..50c22442af0 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/ThemedReactContext.java +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/ThemedReactContext.java @@ -10,8 +10,6 @@ package com.facebook.react.uimanager; import android.app.Activity; import android.content.Context; import androidx.annotation.Nullable; -import com.facebook.react.bridge.JSIModule; -import com.facebook.react.bridge.JSIModuleType; import com.facebook.react.bridge.LifecycleEventListener; import com.facebook.react.bridge.ReactApplicationContext; import com.facebook.react.bridge.ReactContext; @@ -110,14 +108,6 @@ public class ThemedReactContext extends ReactContext { return mReactApplicationContext.isBridgeless(); } - @Override - public JSIModule getJSIModule(JSIModuleType moduleType) { - if (isBridgeless()) { - return mReactApplicationContext.getJSIModule(moduleType); - } - return super.getJSIModule(moduleType); - } - @Override public UIManager getFabricUIManager() { if (isBridgeless()) { diff --git a/packages/react-native/ReactAndroid/src/test/java/com/facebook/react/runtime/BridgelessReactContextTest.kt b/packages/react-native/ReactAndroid/src/test/java/com/facebook/react/runtime/BridgelessReactContextTest.kt index 7d19faeddc9..d8a8c3b4730 100644 --- a/packages/react-native/ReactAndroid/src/test/java/com/facebook/react/runtime/BridgelessReactContextTest.kt +++ b/packages/react-native/ReactAndroid/src/test/java/com/facebook/react/runtime/BridgelessReactContextTest.kt @@ -9,7 +9,6 @@ package com.facebook.react.runtime import android.app.Activity import android.content.Context -import com.facebook.react.bridge.JSIModuleType import com.facebook.react.fabric.FabricUIManager import com.facebook.react.uimanager.UIManagerModule import com.facebook.testutils.shadows.ShadowSoLoader @@ -49,17 +48,11 @@ class BridgelessReactContextTest { Assertions.assertThat(uiManagerModule).isEqualTo(mUiManagerModule) } - @Test(expected = UnsupportedOperationException::class) - fun getJSIModule_throwsException() { - bridgelessReactContext.getJSIModule(JSIModuleType.TurboModuleManager) - } - @Test - fun getJSIModuleTest() { + fun getFabricUIManagerTest() { val fabricUiManager = Mockito.mock(FabricUIManager::class.java) doReturn(fabricUiManager).`when`(reactHost).uiManager - Assertions.assertThat(bridgelessReactContext.getJSIModule(JSIModuleType.UIManager)) - .isEqualTo(fabricUiManager) + Assertions.assertThat(bridgelessReactContext.getFabricUIManager()).isEqualTo(fabricUiManager) } @Test(expected = UnsupportedOperationException::class)