Skip to content

Commit

Permalink
ref: Expose DispatchQueueWrapper to Swift (#3745)
Browse files Browse the repository at this point in the history
  • Loading branch information
philipphofmann committed Mar 15, 2024
1 parent da2f43e commit 973d574
Show file tree
Hide file tree
Showing 13 changed files with 12 additions and 6 deletions.
1 change: 0 additions & 1 deletion SentryTestUtils/SentryTestUtils-ObjC-BridgingHeader.h
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@
#import "SentryCrashWrapper.h"
#import "SentryDependencyContainer.h"
#import "SentryDispatchFactory.h"
#import "SentryDispatchQueueWrapper.h"
#import "SentryDispatchSourceWrapper.h"
#import "SentryEnvelope.h"
#import "SentryFileManager+Test.h"
Expand Down
2 changes: 1 addition & 1 deletion SentryTestUtils/TestDispatchFactory.swift
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import _SentryPrivate
import Foundation
import Sentry

public class TestDispatchFactory: SentryDispatchFactory {
public var vendedSourceHandler: ((TestDispatchSourceWrapper) -> Void)?
Expand Down
1 change: 1 addition & 0 deletions SentryTestUtils/TestSentryDispatchQueueWrapper.swift
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
import _SentryPrivate
import Foundation

/// A wrapper around `SentryDispatchQueueWrapper` that memoized invocations to its methods and allows customization of async logic, specifically: dispatch-after calls can be made to run immediately, or not at all.
Expand Down
1 change: 1 addition & 0 deletions Sources/Sentry/include/SentryPrivate.h
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

#import "SentryBaggage.h"
#import "SentryBaseIntegration.h"
#import "SentryDispatchQueueWrapper.h"
#import "SentryRandom.h"
#import "SentrySdkInfo.h"
#import "SentryTime.h"
1 change: 1 addition & 0 deletions Tests/SentryProfilerTests/SentryProfilerSwiftTests.swift
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
import _SentryPrivate
@testable import Sentry
import SentryTestUtils
import XCTest
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ class SentryAppStartTrackerTests: NotificationCenterTestCase {
notificationCenterWrapper: SentryNSNotificationCenterWrapper()
)

framesTracker = SentryFramesTracker(displayLinkWrapper: displayLinkWrapper, dateProvider: currentDate, dispatchQueueWrapper: SentryDispatchQueueWrapper(), keepDelayedFramesDuration: 0)
framesTracker = SentryFramesTracker(displayLinkWrapper: displayLinkWrapper, dateProvider: currentDate, dispatchQueueWrapper: TestSentryDispatchQueueWrapper(), keepDelayedFramesDuration: 0)
framesTracker.start()

runtimeInitTimestamp = SentryDependencyContainer.sharedInstance().dateProvider.date().addingTimeInterval(0.2)
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
import _SentryPrivate
import Nimble
import SentryTestUtils
import XCTest
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
import _SentryPrivate
import Nimble
import SentryTestUtils
import XCTest
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ class SentrySubClassFinderTests: XCTestCase {
}

var sut: SentrySubClassFinder {
return SentrySubClassFinder(dispatchQueue: SentryDispatchQueueWrapper(), objcRuntimeWrapper: runtimeWrapper)
return SentrySubClassFinder(dispatchQueue: TestSentryDispatchQueueWrapper(), objcRuntimeWrapper: runtimeWrapper)
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ class SentryTimeToDisplayTrackerTest: XCTestCase {
var framesTracker: SentryFramesTracker

init() {
framesTracker = SentryFramesTracker(displayLinkWrapper: displayLinkWrapper, dateProvider: dateProvider, dispatchQueueWrapper: SentryDispatchQueueWrapper(), keepDelayedFramesDuration: 0)
framesTracker = SentryFramesTracker(displayLinkWrapper: displayLinkWrapper, dateProvider: dateProvider, dispatchQueueWrapper: TestSentryDispatchQueueWrapper(), keepDelayedFramesDuration: 0)
SentryDependencyContainer.sharedInstance().framesTracker = framesTracker
framesTracker.start()
}
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
import _SentryPrivate
import XCTest

class SentryDispatchQueueWrapperTests: XCTestCase {
Expand Down
2 changes: 1 addition & 1 deletion Tests/SentryTests/Transaction/SentrySpanTests.swift
Original file line number Diff line number Diff line change
Expand Up @@ -541,7 +541,7 @@ class SentrySpanTests: XCTestCase {

private func givenFramesTracker() -> (TestDisplayLinkWrapper, SentryFramesTracker) {
let displayLinkWrapper = TestDisplayLinkWrapper(dateProvider: self.fixture.currentDateProvider)
let framesTracker = SentryFramesTracker(displayLinkWrapper: displayLinkWrapper, dateProvider: self.fixture.currentDateProvider, dispatchQueueWrapper: SentryDispatchQueueWrapper(), keepDelayedFramesDuration: 10)
let framesTracker = SentryFramesTracker(displayLinkWrapper: displayLinkWrapper, dateProvider: self.fixture.currentDateProvider, dispatchQueueWrapper: TestSentryDispatchQueueWrapper(), keepDelayedFramesDuration: 10)
framesTracker.start()
displayLinkWrapper.call()

Expand Down
1 change: 1 addition & 0 deletions Tests/SentryTests/Transaction/SentryTracerTests.swift
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
import _SentryPrivate
import Nimble
@testable import Sentry
import SentryTestUtils
Expand Down

0 comments on commit 973d574

Please sign in to comment.