Log updatePackage errors (and don't abort)

SwiftPackageIndex · Sep 25, 2023 · 132f8d6 · 132f8d6
1 parent d95b7a9
commit 132f8d6
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/Sources/App/Commands/Common.swift b/Sources/App/Commands/Common.swift
@@ -43,7 +43,11 @@ func updatePackages(client: Client,
         }
     }
     for result in results {
-        try await updatePackage(client: client, database: database, logger: logger, result: result, stage: stage)
+        do {
+            try await updatePackage(client: client, database: database, logger: logger, result: result, stage: stage)
+        } catch {
+            logger.critical("updatePackage failed: \(error)")
+        }
     }
 
     logger.debug("updateStatus ops: \(results.count)")

diff --git a/Tests/AppTests/AnalyzerTests.swift b/Tests/AppTests/AnalyzerTests.swift
@@ -788,7 +788,7 @@ class AnalyzerTests: AppTestCase {
         XCTAssertEqual(targets.map(\.type), [.regular, .executable])
     }
 
-    func test_updatePackage() async throws {
+    func test_updatePackages() async throws {
         // setup
         let packages = try savePackages(on: app.db, ["1", "2"].asURLs)
             .map(Joined<Package, Repository>.init(model:))