Make SentryCrashBridge non null

Author: itaybre

Sources/Sentry/include/SentryCrash.h

@@ -66,8 +66,9 @@ SENTRY_NO_INIT
 /** Cache directory base path. */
 @property (nonatomic, readwrite, retain) NSString *basePath;
 
-/** Bridge to SDK services (notification center, date provider, crash reporter). */
-@property (nonatomic, strong, nullable) SentryCrashBridge *bridge;
+/** Bridge to SDK services (notification center, date provider, crash reporter).
+ * Must be set before calling install. */
+@property (nonatomic, strong) SentryCrashBridge *bridge;
 
 /** A dictionary containing any info you'd like to appear in crash reports. Must
  * contain only JSON-safe data: NSString for keys, and NSDictionary, NSArray,

Sources/Sentry/include/SentryCrashInstallation.h

@@ -63,8 +63,8 @@ NS_ASSUME_NONNULL_BEGIN
 - (void)sendAllReportsWithCompletion:(nullable SentryCrashReportFilterCompletion)onCompletion;
 
 /** Bridge for accessing SDK services without dependency container.
- * Set by SentryCrashIntegration before installation. */
-@property (nonatomic, strong, nullable) SentryCrashBridge *bridge;
+ * Must be set by SentryCrashIntegration before calling install. */
+@property (nonatomic, strong) SentryCrashBridge *bridge;
 
 @end
 

Tests/SentryTests/SentryCrash/SentryCrashInstallationTests.swift

@@ -73,6 +73,30 @@ class SentryCrashInstallationTests: XCTestCase {
         #endif // os(iOS) || os(tvOS)
     }
 
+    func testInstall_SetsUncaughtExceptionHandler() {
+        // Verifies the bridge is set before sentrycrash_install so that when
+        // the NSException monitor calls setEnabled(true), g_bridge is non-nil
+        // and uncaughtExceptionHandler gets assigned on the crash reporter.
+        // Previously, setBridge was called after install, so the assignment
+        // was a no-op via ObjC nil messaging.
+        let installation = getSut()
+        let crashReporter = SentryDependencyContainer.sharedInstance().crashReporter
+
+        // Uninstall first to reset g_installed and disable all monitors.
+        // g_installed is a static that persists across tests; if it's
+        // already 1, sentrycrash_install short-circuits and monitors are
+        // never re-enabled through the full setEnabled(true) path.
+        crashReporter.uninstall()
+
+        installation.install("/private/tmp")
+        defer { installation.uninstall() }
+
+        XCTAssertNotNil(
+            crashReporter.uncaughtExceptionHandler,
+            "uncaughtExceptionHandler should be set after install because the bridge must be available when the NSException monitor is enabled"
+        )
+    }
+
     // MARK: - Private
 
     private func getSut() -> SentryCrashTestInstallation {