Merge e59bff3 into d1d5c2d

getsentry · Mar 3, 2023 · 542210a · 542210a
2 parents d1d5c2d + e59bff3
commit 542210a
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 2 deletions.
diff --git a/Sources/Sentry/SentryClient.m b/Sources/Sentry/SentryClient.m
@@ -102,6 +102,15 @@ - (instancetype)initWithOptions:(SentryOptions *)options
     SentryTransportAdapter *transportAdapter =
         [[SentryTransportAdapter alloc] initWithTransport:transport options:options];
 
+    return [self initWithOptions:options fileManager:fileManager transportAdapter:transportAdapter];
+}
+
+/** Internal constructor for testing purposes. */
+- (instancetype)initWithOptions:(SentryOptions *)options
+                    fileManager:(SentryFileManager *)fileManager
+               transportAdapter:(SentryTransportAdapter *)transportAdapter
+
+{
     SentryInAppLogic *inAppLogic =
         [[SentryInAppLogic alloc] initWithInAppIncludes:options.inAppIncludes
                                           inAppExcludes:options.inAppExcludes];

diff --git a/Tests/SentryTests/SentryClient+TestInit.h b/Tests/SentryTests/SentryClient+TestInit.h
@@ -15,6 +15,10 @@ SentryClient ()
 - (_Nullable instancetype)initWithOptions:(SentryOptions *)options
                               fileManager:(SentryFileManager *)fileManager;
 
+- (instancetype)initWithOptions:(SentryOptions *)options
+                    fileManager:(SentryFileManager *)fileManager
+               transportAdapter:(SentryTransportAdapter *)transportAdapter;
+
 - (instancetype)initWithOptions:(SentryOptions *)options
                transportAdapter:(SentryTransportAdapter *)transportAdapter
                     fileManager:(SentryFileManager *)fileManager

diff --git a/Tests/SentryTests/TestClient.swift b/Tests/SentryTests/TestClient.swift
@@ -2,11 +2,15 @@ import Foundation
 
 class TestClient: SentryClient {
     override init?(options: Options) {
-        super.init(options: options, fileManager: try! TestFileManager(options: options))
+        super.init(options: options, fileManager: try! TestFileManager(options: options), transportAdapter: TestTransportAdapter(transport: TestTransport(), options: options))
     }
 
     override init?(options: Options, fileManager: SentryFileManager) {
-        super.init(options: options, fileManager: fileManager)
+        super.init(options: options, fileManager: fileManager, transportAdapter: TestTransportAdapter(transport: TestTransport(), options: options))
+    }
+
+    override init(options: Options, fileManager: SentryFileManager, transportAdapter: SentryTransportAdapter) {
+        super.init(options: options, fileManager: fileManager, transportAdapter: transportAdapter)
     }
 
     // Without this override we get a fatal error: use of unimplemented initializer