[Infra] Fix-up import tests (#11804)

.github/workflows/client_app.yml

@@ -9,7 +9,7 @@ on:
       - "*.podspec"
       - "scripts/install_prereqs.sh"
       - "scripts/build.sh"
-      - "ClientApp/*"
+      - "ClientApp/**"
       - "Gemfile*"
   schedule:
     # Run every day at 12am (PST) - cron uses UTC times

ClientApp/Shared/objc-header-import-test.m

@@ -42,17 +42,15 @@
 #import "FirebaseDynamicLinks/FirebaseDynamicLinks.h"
 #endif
 #import <FirebaseFirestore/FirebaseFirestore.h>
-#import "FirebaseFirestore/FirebaseFirestore.h"
-#if (TARGET_OS_IOS || TARGET_OS_TV) && !TARGET_OS_MACCATALYST
-#import <FirebaseInAppMessaging/FirebaseInAppMessaging.h>
-#import "FirebaseInAppMessaging/FirebaseInAppMessaging.h"
-#endif
 #import <FirebaseInstallations/FirebaseInstallations.h>
 #import <FirebaseMessaging/FirebaseMessaging.h>
+#import "FirebaseFirestore/FirebaseFirestore.h"
 #import "FirebaseInstallations/FirebaseInstallations.h"
 #import "FirebaseMessaging/FirebaseMessaging.h"
 #if (TARGET_OS_IOS && !TARGET_OS_MACCATALYST) || TARGET_OS_TV
+#import <FirebaseInAppMessaging/FirebaseInAppMessaging.h>
 #import <FirebasePerformance/FirebasePerformance.h>
+#import "FirebaseInAppMessaging/FirebaseInAppMessaging.h"
 #import "FirebasePerformance/FirebasePerformance.h"
 #endif
 #import <FirebaseRemoteConfig/FirebaseRemoteConfig.h>

ClientApp/Shared/objc-module-import-test.m

@@ -33,13 +33,11 @@
 #endif
 @import FirebaseFirestore;
 @import FirebaseFunctions;
-#if (TARGET_OS_IOS || TARGET_OS_TV) && !TARGET_OS_MACCATALYST
-@import FirebaseInAppMessaging;
-#endif
 @import FirebaseInstallations;
 @import FirebaseMessaging;
 #if (TARGET_OS_IOS && !TARGET_OS_MACCATALYST) || TARGET_OS_TV
 @import FirebasePerformance;
+@import FirebaseInAppMessaging;
 #endif
 @import FirebaseRemoteConfig;
 @import FirebaseStorage;

ClientApp/Shared/objcxx-header-import-test.mm

@@ -44,10 +44,6 @@
 // #endif
 // #import <FirebaseFirestore/FirebaseFirestore.h>
 // #import "FirebaseFirestore/FirebaseFirestore.h"
-// #if (TARGET_OS_IOS || TARGET_OS_TV) && !TARGET_OS_MACCATALYST
-// #import <FirebaseInAppMessaging/FirebaseInAppMessaging.h>
-// #import "FirebaseInAppMessaging/FirebaseInAppMessaging.h"
-// #endif
 // #import <FirebaseInstallations/FirebaseInstallations.h>
 // #import <FirebaseMessaging/FirebaseMessaging.h>
 // #import "FirebaseInstallations/FirebaseInstallations.h"
@@ -55,6 +51,8 @@
 // #if (TARGET_OS_IOS && !TARGET_OS_MACCATALYST) || TARGET_OS_TV
 // #import <FirebasePerformance/FirebasePerformance.h>
 // #import "FirebasePerformance/FirebasePerformance.h"
+// #import <FirebaseInAppMessaging/FirebaseInAppMessaging.h>
+// #import "FirebaseInAppMessaging/FirebaseInAppMessaging.h"
 // #endif
 // #import <FirebaseRemoteConfig/FirebaseRemoteConfig.h>
 // #import "FirebaseRemoteConfig/FirebaseRemoteConfig.h"

ClientApp/Shared/objcxx-module-import-test.mm

@@ -31,13 +31,11 @@
 // #endif
 // @import FirebaseFirestore;
 // @import FirebaseFunctions;
-// #if (TARGET_OS_IOS || TARGET_OS_TV) && !TARGET_OS_MACCATALYST
-// @import FirebaseInAppMessaging;
-// #endif
 // @import FirebaseInstallations;
 // @import FirebaseMessaging;
 // #if (TARGET_OS_IOS && !TARGET_OS_MACCATALYST) || TARGET_OS_TV
 // @import FirebasePerformance;
+// @import FirebaseInAppMessaging;
 // #endif
 // @import FirebaseRemoteConfig;
 // @import FirebaseStorage;

ClientApp/Shared/swift-import-test.swift

@@ -46,15 +46,13 @@ import FirebaseFunctions
 #if SWIFT_PACKAGE
   import FirebaseFunctionsCombineSwift
 #endif // SWIFT_PACKAGE
-#if (os(iOS) || os(tvOS)) && !targetEnvironment(macCatalyst)
-  import FirebaseInAppMessaging
-  import FirebaseInAppMessagingSwift
-#endif
 import FirebaseInstallations
 import FirebaseMessaging
 import FirebaseMLModelDownloader
 #if (os(iOS) && !targetEnvironment(macCatalyst)) || os(tvOS)
   import FirebasePerformance
+  import FirebaseInAppMessaging
+  import FirebaseInAppMessagingSwift
 #endif
 import FirebaseRemoteConfig
 import FirebaseRemoteConfigSwift

SwiftPMTests/objc-import-test/objc-header.m

@@ -24,12 +24,10 @@
 #import "FirebaseDatabase/FirebaseDatabase.h"
 #import "FirebaseDynamicLinks/FirebaseDynamicLinks.h"
 #import "FirebaseFirestore/FirebaseFirestore.h"
-#if TARGET_OS_IOS || TARGET_OS_TV
-#import "FirebaseInAppMessaging/FirebaseInAppMessaging.h"
-#endif
 #import "FirebaseInstallations/FirebaseInstallations.h"
 #import "FirebaseMessaging/FirebaseMessaging.h"
 #if (TARGET_OS_IOS && !TARGET_OS_MACCATALYST) || TARGET_OS_TV
+#import "FirebaseInAppMessaging/FirebaseInAppMessaging.h"
 #import "FirebasePerformance/FirebasePerformance.h"
 #endif
 #import "FirebaseRemoteConfig/FirebaseRemoteConfig.h"

SwiftPMTests/objc-import-test/objc-module.m

@@ -25,12 +25,10 @@
 @import FirebaseDynamicLinks;
 @import FirebaseFirestore;
 @import FirebaseFunctions;
-#if TARGET_OS_IOS || TARGET_OS_TV
-@import FirebaseInAppMessaging;
-#endif
 @import FirebaseInstallations;
 @import FirebaseMessaging;
 #if (TARGET_OS_IOS && !TARGET_OS_MACCATALYST) || TARGET_OS_TV
+@import FirebaseInAppMessaging;
 @import FirebasePerformance;
 #endif
 @import FirebaseRemoteConfig;

SwiftPMTests/swift-test/all-imports.swift

@@ -27,15 +27,14 @@ import FirebaseDynamicLinks
 import FirebaseFirestore
 import FirebaseFirestoreSwift
 import FirebaseFunctions
-#if (os(iOS) || os(tvOS)) && !targetEnvironment(macCatalyst)
-  import FirebaseInAppMessaging
-  @testable import FirebaseInAppMessagingSwift
-  import SwiftUI
-#endif
 import FirebaseInstallations
 import FirebaseMessaging
 #if (os(iOS) && !targetEnvironment(macCatalyst)) || os(tvOS)
   import FirebasePerformance
+
+  import FirebaseInAppMessaging
+  @testable import FirebaseInAppMessagingSwift
+  import SwiftUI
 #endif
 import FirebaseRemoteConfig
 import FirebaseSessions