Merge pull request #4033 from SwiftPackageIndex/update-CHECK_MON_001-window

Update checkMon001TimePeriod to reflect increased amount of time it takes to visit all packages

Author: finestructure

Sources/App/Commands/Alerting.swift

@@ -22,9 +22,9 @@ import Vapor
 enum Alerting {
     static let defaultLimit = 2000
     static let defaultTimePeriod = 2
-    // CHECK_MON_001 has to run on a longer time period, because it currently takes ~4h to visit every package
+    // CHECK_MON_001 has to run on a longer time period, because it currently takes ~8h to visit every package
     // during analysis. With a window shorter than that, they query will always report errors.
-    static let checkMon001TimePeriod: TimeAmount = .hours(6)
+    static let checkMon001TimePeriod: TimeAmount = .hours(9)
 
     struct Command: AsyncCommand {
         var help: String { "Application level alerting" }

Tests/AppTests/QueryPerformanceTests.swift

@@ -180,7 +180,7 @@ struct QueryPerformanceTests {
                   JOIN versions v ON v.package_id = p.id
                 WHERE v.reference ->> 'branch' = r.default_branch
                 """)
-            try await assertQueryPerformance(query, expectedCost: 220_000, variation: 10000)
+            try await assertQueryPerformance(query, expectedCost: 240_000, variation: 10000)
         }
     }