Skip to content

Commit 26fcf04

Browse files
authored
[NFC] Remove dated TODOs (#13190)
1 parent 09920d1 commit 26fcf04

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

FirebaseInAppMessaging/Swift/Tests/Unit/FirebaseInAppMessaging_APIBuildTests.swift

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,13 +27,10 @@ final class FirebaseInAppMessagingSwift_APIBuildTests: XCTestCase {
2727
let _: Bool = inAppMessaging.messageDisplaySuppressed
2828
inAppMessaging.messageDisplaySuppressed = true
2929

30-
// TODO(ncooke3): This should probably be removed in favor of the one in
31-
// FirebaseCore?
3230
let _: Bool = inAppMessaging.automaticDataCollectionEnabled
3331
inAppMessaging.automaticDataCollectionEnabled = true
3432

3533
let _: FirebaseInAppMessaging.InAppMessagingDisplay = inAppMessaging.messageDisplayComponent
36-
// FIXME(ncooke3): Below.
3734
let displayConformer: FirebaseInAppMessaging.InAppMessagingDisplay! = nil
3835
inAppMessaging.messageDisplayComponent = displayConformer
3936

@@ -42,7 +39,6 @@ final class FirebaseInAppMessagingSwift_APIBuildTests: XCTestCase {
4239
let delegate: FirebaseInAppMessaging.InAppMessagingDisplayDelegate? = inAppMessaging.delegate
4340
inAppMessaging.delegate = nil
4441

45-
// TODO(ncooke3): Does it make sense for these params to be nullable?
4642
let nullableText: String? = nil
4743
let nullableURL: URL? = nil
4844
let action = FirebaseInAppMessaging.InAppMessagingAction(

0 commit comments

Comments
 (0)