Ensure we don’t start another sync on reentry to POS

Author: joshheald

Modules/Sources/Yosemite/Tools/POS/POSCatalogSyncCoordinator.swift

@@ -94,9 +94,12 @@ public actor POSCatalogSyncCoordinator: POSCatalogSyncCoordinatorProtocol {
             throw POSCatalogSyncError.shouldNotSync
         }
 
-        if case .syncStarted = fullSyncStateModel.state[siteID] {
+        switch fullSyncStateModel.state[siteID] {
+        case .syncStarted, .initialSyncStarted:
             DDLogInfo("⚠️ POSCatalogSyncCoordinator: Sync already in progress for site \(siteID)")
             throw POSCatalogSyncError.syncAlreadyInProgress(siteID: siteID)
+        default:
+            break
         }
 
         let isFirstSync = await lastFullSyncDate(for: siteID) == nil

Modules/Tests/YosemiteTests/Tools/POS/POSCatalogSyncCoordinatorTests.swift

@@ -204,7 +204,10 @@ struct POSCatalogSyncCoordinatorTests {
         }
 
         let currentState = await sut.loadLastFullSyncState(for: sampleSiteID)
-        let isSyncStarted: Bool = if case .syncStarted = currentState { true } else { false }
+        let isSyncStarted: Bool = switch currentState {
+        case .syncStarted, .initialSyncStarted: true
+        default: false
+        }
         #expect(isSyncStarted)
 
         // Cleanup - resume the first sync and wait for it to complete