fix: Native wrapper methods don't throw disabled error after re-initi…

…alizing (#3093)

Co-authored-by: Kryštof Woldřich <[email protected]>

CHANGELOG.md

@@ -5,6 +5,7 @@
 ### Fixes
 
 - Dynamically resolve `collectModulesScript` path to support monorepos ([#3092])(https://github.com/getsentry/sentry-react-native/pull/3092)
+- Native wrapper methods don't throw disabled error after re-initializing [#3093](https://github.com/getsentry/sentry-react-native/pull/3093)
 
 ### Dependencies
 

src/js/wrapper.ts

@@ -180,13 +180,15 @@ export const NATIVE: SentryNativeWrapper = {
           'Note: Native Sentry SDK was not initialized automatically, you will need to initialize it manually. If you wish to disable the native SDK and get rid of this warning, pass enableNative: false',
         );
       }
+      this.enableNative = true;
       return false;
     }
 
     if (!options.dsn) {
       logger.warn(
         'Warning: No DSN was provided. The Sentry SDK will be disabled. Native SDK will also not be initalized.',
       );
+      this.enableNative = false;
       return false;
     }
 
@@ -201,6 +203,7 @@ export const NATIVE: SentryNativeWrapper = {
     const nativeIsReady = await RNSentry.initNativeSdk(filteredOptions);
 
     this.nativeIsReady = nativeIsReady;
+    this.enableNative = true;
 
     return nativeIsReady;
   },

test/client.test.ts

@@ -216,6 +216,7 @@ describe('Tests ReactNativeClient', () => {
 
       const client = new ReactNativeClient({
         ...DEFAULT_OPTIONS,
+        dsn: EXAMPLE_DSN,
         enableNative: true,
         transport: () => new NativeTransport(),
       });

test/wrapper.test.ts

@@ -131,6 +131,7 @@ describe('Tests Native Wrapper', () => {
     });
 
     test('does not initialize with autoInitializeNativeSdk: false', async () => {
+      NATIVE.enableNative = false;
       logger.warn = jest.fn();
 
       await NATIVE.initNativeSdk({
@@ -487,6 +488,7 @@ describe('Tests Native Wrapper', () => {
   });
 
   describe('closeNativeSdk', () => {
+    NATIVE.enableNative = true;
     test('closeNativeSdk calls native bridge', async () => {
       await NATIVE.closeNativeSdk();