Repository: firebase/firebase-ios-sdk (language: cpp)

Issue:
[Firebase 11] Audit FIAM API tests for potential improvement opportunities
### Description

See todos in https://github.com/firebase/firebase-ios-sdk/blob/main/FirebaseInAppMessaging/Swift/Tests/Unit/FirebaseInAppMessaging_APIBuildTests.swift

### Reproducing the issue

_No response_

### Firebase SDK Version

10.24

### Xcode Version

15.2

### Installation Method

Swift Package Manager

### Firebase Product(s)

In-App Messaging

### Targeted Platforms

All

### Relevant Log Output

_No response_

### If using Swift Package Manager, the project's Package.resolved

<!--- Look below for instructions on how to share your Package.resolved. --->

<details>
<summary>Expand <code>Package.resolved</code> snippet</summary>
<br>

```json

Replace this line with the contents of your Package.resolved.

```

</details>


### If using CocoaPods, the project's Podfile.lock

<!--- Look below for instructions on how to share your Podfile.lock. --->

<details>
<summary>Expand <code>Podfile.lock</code> snippet</summary>
<br>

```yml

Replace this line with the contents of your Podfile.lock!

```

</details>



Discussion / hints:
Merged #13089

Fix patch:
diff --git a/FirebaseInAppMessaging/Swift/Tests/Unit/FirebaseInAppMessaging_APIBuildTests.swift b/FirebaseInAppMessaging/Swift/Tests/Unit/FirebaseInAppMessaging_APIBuildTests.swift
index f5a77be7c23..2d9faeb5cf3 100644
--- a/FirebaseInAppMessaging/Swift/Tests/Unit/FirebaseInAppMessaging_APIBuildTests.swift
+++ b/FirebaseInAppMessaging/Swift/Tests/Unit/FirebaseInAppMessaging_APIBuildTests.swift
@@ -27,13 +27,10 @@ final class FirebaseInAppMessagingSwift_APIBuildTests: XCTestCase {
     let _: Bool = inAppMessaging.messageDisplaySuppressed
     inAppMessaging.messageDisplaySuppressed = true
 
-    // TODO(ncooke3): This should probably be removed in favor of the one in
-    // FirebaseCore?
     let _: Bool = inAppMessaging.automaticDataCollectionEnabled
     inAppMessaging.automaticDataCollectionEnabled = true
 
     let _: FirebaseInAppMessaging.InAppMessagingDisplay = inAppMessaging.messageDisplayComponent
-    // FIXME(ncooke3): Below.
     let displayConformer: FirebaseInAppMessaging.InAppMessagingDisplay! = nil
     inAppMessaging.messageDisplayComponent = displayConformer
 
@@ -42,7 +39,6 @@ final class FirebaseInAppMessagingSwift_APIBuildTests: XCTestCase {
     let delegate: FirebaseInAppMessaging.InAppMessagingDisplayDelegate? = inAppMessaging.delegate
     inAppMessaging.delegate = nil
 
-    // TODO(ncooke3): Does it make sense for these params to be nullable?
     let nullableText: String? = nil
     let nullableURL: URL? = nil
     let action = FirebaseInAppMessaging.InAppMessagingAction(


Test patch: