[Swift 6] Add Swift 6 testing for Sessions

.github/workflows/sessions.yml

@@ -28,10 +28,17 @@ jobs:
           - os: macos-14
             xcode: Xcode_15.3
             tests:
+            swift_version: 5.9
           # Flaky tests on CI
           - os: macos-15
             xcode: Xcode_16.2
             tests: --skip-tests
+            swift_version: 5.9
+          # Flaky tests on CI
+          - os: macos-15
+            xcode: Xcode_16.2
+            tests: --skip-tests
+            swift_version: 6.0
     runs-on: ${{ matrix.build-env.os }}
     steps:
     - uses: actions/checkout@v4
@@ -40,6 +47,8 @@ jobs:
       run: scripts/setup_bundler.sh
     - name: Xcode
       run: sudo xcode-select -s /Applications/${{ matrix.build-env.xcode }}.app/Contents/Developer
+    - name: Set Swift swift_version
+      run: sed -i "" "s/s.swift_version[[:space:]]*=[[:space:]]*'5.9'/s.swift_version = '${{ matrix.build-env.swift_version }}'/" FirebaseSessions.podspec
     - uses: nick-fields/retry@v3
       with:
         timeout_minutes: 120

FirebaseSessions/Tests/Unit/FirebaseSessionsTests+BaseBehaviors.swift

@@ -87,7 +87,7 @@
     // We wanted to make sure that since we've introduced promises,
     // once the promise has been fulfilled, that .then'ing on the promise
     // in future initiations still results in a log
-    func test_multipleInitiations_logsSessionEventEachInitiation() {
+    @MainActor func test_multipleInitiations_logsSessionEventEachInitiation() {
       var loggedCount = 0
       var lastLoggedSessionID = ""
       let loggedTwiceExpectation = expectation(description: "Sessions SDK logged events twice")
@@ -138,10 +138,10 @@
         }
       )
 
       wait(for: [loggedTwiceExpectation], timeout: 3)
 
       // Make sure we logged 2 events
       XCTAssertEqual(loggedCount, 2)
     }
 
   #endif // !os(watchOS)

FirebaseSessions/Tests/Unit/InitiatorTests.swift

@@ -58,7 +58,7 @@ class InitiatorTests: XCTestCase {
     XCTAssert(initiateCalled)
   }
 
-  func test_appForegrounded_initiatesNewSession() throws {
+  @MainActor func test_appForegrounded_initiatesNewSession() throws {
     // Given
     var pausedClock = date
     let initiator = SessionInitiator(

FirebaseSessions/Tests/Unit/Library/LifecycleNotifications.swift

@@ -37,18 +37,7 @@ import Dispatch
 #endif // swift(>=5.9)
 
 extension XCTestCase {
-  func postBackgroundedNotification() {
-    // On Catalyst, the notifications can only be called on a the main thread
-    if Thread.isMainThread {
-      postBackgroundedNotificationInternal()
-    } else {
-      DispatchQueue.main.sync {
-        self.postBackgroundedNotificationInternal()
-      }
-    }
-  }
-
-  private func postBackgroundedNotificationInternal() {
+  @MainActor func postBackgroundedNotification() {
     let notificationCenter = NotificationCenter.default
     #if os(iOS) || os(tvOS)
       notificationCenter.post(name: UIApplication.didEnterBackgroundNotification, object: nil)
@@ -74,18 +63,7 @@ extension XCTestCase {
     #endif // swift(>=5.9)
   }
 
-  func postForegroundedNotification() {
-    // On Catalyst, the notifications can only be called on a the main thread
-    if Thread.isMainThread {
-      postForegroundedNotificationInternal()
-    } else {
-      DispatchQueue.main.sync {
-        self.postForegroundedNotificationInternal()
-      }
-    }
-  }
-
-  private func postForegroundedNotificationInternal() {
+  @MainActor func postForegroundedNotification() {
     let notificationCenter = NotificationCenter.default
     #if os(iOS) || os(tvOS)
       notificationCenter.post(name: UIApplication.didBecomeActiveNotification, object: nil)