Remove unused AppError.metadataRequestFailed

Author: finestructure

Sources/App/Commands/Common.swift

@@ -123,8 +123,6 @@ func recordError(database: Database,
             try await setStatus(id: id, status: .cacheDirectoryDoesNotExist)
         case let .invalidRevision(id, _):
             try await setStatus(id: id, status: .analysisFailed)
-        case let .metadataRequestFailed(id, _, _):
-            try await setStatus(id: id, status: .metadataRequestFailed)
         case let .noValidVersions(id, _):
             try await setStatus(id: id, status: .noValidVersions)
     }

Sources/App/Core/AppError.swift

@@ -21,7 +21,6 @@ enum AppError: LocalizedError {
     case invalidPackageCachePath(Package.Id?, _ path: String)
     case cacheDirectoryDoesNotExist(Package.Id?, _ path: String)
     case invalidRevision(Version.Id?, _ revision: String?)
-    case metadataRequestFailed(Package.Id?, HTTPStatus, URI)
     case noValidVersions(Package.Id?, _ url: String)
     case shellCommandFailed(_ command: String, _ path: String, _ message: String)
 
@@ -39,8 +38,6 @@ enum AppError: LocalizedError {
                 return "Package cache directory, \(value), does not exist: (id: \(id)"
             case let .invalidRevision(id, value):
                 return "Invalid revision: \(value ?? "nil") (id: \(id))"
-            case let .metadataRequestFailed(id, status, uri):
-                return "Metadata request for URI '\(uri.description)' failed with status '\(status)'  (id: \(id))"
             case let .noValidVersions(id, value):
                 return "No valid version found for package '\(value)' (id: \(id))"
             case let .shellCommandFailed(command, path, message):

Tests/AppTests/IngestorTests.swift

@@ -240,7 +240,7 @@ class IngestorTests: AppTestCase {
                                                             "https://github.com/foo/2"])
             .map(Joined<Package, Repository>.init(model:))
         let results: [Result<Joined<Package, Repository>, Error>] = [
-            .failure(AppError.metadataRequestFailed(try pkgs[0].model.requireID(), .badRequest, "1")),
+            .failure(AppError.genericError(try pkgs[0].model.requireID(), "error 1")),
             .success(pkgs[1])
         ]
 
@@ -253,7 +253,7 @@ class IngestorTests: AppTestCase {
         // validate
         do {
             let pkgs = try await Package.query(on: app.db).sort(\.$url).all()
-            XCTAssertEqual(pkgs.map(\.status), [.metadataRequestFailed, .new])
+            XCTAssertEqual(pkgs.map(\.status), [.ingestionFailed, .new])
             XCTAssertEqual(pkgs.map(\.processingStage), [.ingestion, .ingestion])
         }
     }
@@ -308,7 +308,7 @@ class IngestorTests: AppTestCase {
                                                    processingStage: .reconciliation)
         Current.fetchMetadata = { _, owner, repository in
             if owner == "foo" && repository == "2" {
-                throw AppError.metadataRequestFailed(packages[1].id, .badRequest, URI("2"))
+                throw AppError.genericError(packages[1].id, "error 2")
             }
             return .mock(owner: owner, repository: repository)
         }
@@ -326,7 +326,7 @@ class IngestorTests: AppTestCase {
         (try await Package.query(on: app.db).all()).forEach { pkg in
             switch pkg.url {
                 case "https://github.com/foo/2":
-                    XCTAssertEqual(pkg.status, .metadataRequestFailed)
+                    XCTAssertEqual(pkg.status, .ingestionFailed)
                 default:
                     XCTAssertEqual(pkg.status, .new)
             }

Tests/AppTests/Util+ext.swift

@@ -96,8 +96,6 @@ extension App.AppError: Swift.Equatable {
                 return (id1, v1) == (id2, v2)
             case let (.invalidRevision(id1, v1), .invalidRevision(id2, v2)):
                 return (id1, v1) == (id2, v2)
-            case let (.metadataRequestFailed(id1, s1, u1), .metadataRequestFailed(id2, s2, u2)):
-                return (id1, s1.code, u1.description) == (id2, s2.code, u2.description)
             default:
                 return false
         }