debug interop

Author: chloe-yeo

Sources/FoundationEssentials/ProgressManager/ProgressManager+Interop.swift

@@ -123,18 +123,19 @@ internal final class SubprogressBridge: Sendable {
     internal let manager: ProgressManager
 
     init(parent: Progress, portion: Int64, manager: ProgressManager) {
-        self.manager = manager
         self.progressBridge = Progress(totalUnitCount: 1, parent: parent, pendingUnitCount: portion)
-                
+        self.manager = manager
+
         manager.addObserver { [weak self] observerState in
             guard let self else {
                 return
             }
 
             switch observerState {
             case .fractionUpdated(let totalCount, let completedCount):
-                self.progressBridge.completedUnitCount = Int64(completedCount)
+                // This needs to change totalUnitCount before completedUnitCount otherwise progressBridge will finish and mess up the math
                 self.progressBridge.totalUnitCount = Int64(totalCount)
+                self.progressBridge.completedUnitCount = Int64(completedCount)
             }
         }
     }

Tests/FoundationEssentialsTests/ProgressManager/ProgressManagerInteropTests.swift

@@ -41,15 +41,15 @@ import Testing
             p1.completedUnitCount = 2
         }.value
 
-        // Check if ProgressManager values propagate to Progress parent
+        // Check if Progress values propagate to Progress parent
         #expect(overall.fractionCompleted == 0.5)
         #expect(overall.completedUnitCount == 5)
 
         // Add ProgressManager as Child
         let p2 = overall.makeChild(withPendingUnitCount: 5)
         await doSomethingWithReporter(subprogress: p2)
 
-        // Check if Progress values propagate to Progress parent
+        // Check if ProgressManager values propagate to Progress parent
         #expect(overall.fractionCompleted == 1.0)
         #expect(overall.completedUnitCount == 10)
     }