should not ignore options.transport function provided in Sentry.init(...)

CHANGELOG.md

@@ -5,6 +5,7 @@
 ### Fixes
 
 - SENTRY_DIST accepts non-number values on Android ([#2395](https://github.com/getsentry/sentry-react-native/pull/2395))
+- should not ignore `options.transport` function provided in `Sentry.init(...)` ([#2398](https://github.com/getsentry/sentry-react-native/pull/2398))
 
 ### Features
 

src/js/client.ts

@@ -25,14 +25,14 @@ export class ReactNativeClient extends BaseClient<ReactNativeClientOptions> {
    * @param options Configuration options for this SDK.
    */
    public constructor(options: ReactNativeClientOptions) {
-    const transport = (options: BrowserTransportOptions, nativeFetch?: FetchImpl): Transport => {
-      if (NATIVE.isNativeTransportAvailable()) {
-        return new NativeTransport();
-      }
-      return makeFetchTransport(options, nativeFetch);
+     if (!options.transport) {
+       options.transport = (options: BrowserTransportOptions, nativeFetch?: FetchImpl): Transport => {
+         if (NATIVE.isNativeTransportAvailable()) {
+           return new NativeTransport();
+         }
+         return makeFetchTransport(options, nativeFetch);
+       };
      }
-
-     options.transport = transport;
      super(options);
 
     // This is a workaround for now using fetch on RN, this is a known issue in react-native and only generates a warning

test/client.test.ts

@@ -1,3 +1,4 @@
+import { Envelope, Transport } from '@sentry/types';
 import * as RN from 'react-native';
 
 import { ReactNativeClient } from '../src/js/client';
@@ -99,6 +100,25 @@ describe('Tests ReactNativeClient', () => {
       // eslint-disable-next-line deprecation/deprecation
       await expect(RN.YellowBox.ignoreWarnings).toBeCalled();
     });
+
+    test('use custom transport function', async () => {
+      // eslint-disable-next-line @typescript-eslint/no-unused-vars
+      const mySend = (request: Envelope) => Promise.resolve();
+      const myFlush = (timeout?: number) => Promise.resolve(Boolean(timeout));
+      const myCustomTransportFn = (): Transport => ({
+        send: mySend,
+        flush: myFlush
+      });
+      const client = new ReactNativeClient({
+        ...DEFAULT_OPTIONS,
+        dsn: EXAMPLE_DSN,
+        transport: myCustomTransportFn
+      } as ReactNativeClientOptions);
+      // eslint-disable-next-line @typescript-eslint/unbound-method
+      expect(client.getTransport()?.flush).toBe(myFlush);
+      // eslint-disable-next-line @typescript-eslint/unbound-method
+      expect(client.getTransport()?.send).toBe(mySend);
+    });
   });
 
   describe('onReady', () => {