Use the test’s default timeout duration for sourcekitd timeouts

ahoppen · ahoppen · commit 2623b4949dde · 2024-10-09T13:55:18.000-07:00
Otherwise, we are stuck with the timeout default of 120s and the test runner can’t increase it.
diff --git a/Sources/SKTestSupport/TestSourceKitLSPClient.swift b/Sources/SKTestSupport/TestSourceKitLSPClient.swift
@@ -130,6 +130,9 @@ package final class TestSourceKitLSPClient: MessageHandler, Sendable {
         (options.swiftPMOrDefault.swiftCompilerFlags ?? []) + ["-module-cache-path", globalModuleCache.path]
     }
     options.backgroundIndexing = enableBackgroundIndexing
+    if options.sourcekitdRequestTimeout == nil {
+      options.sourcekitdRequestTimeout = defaultTimeout
+    }
 
     var notificationYielder: AsyncStream<any NotificationType>.Continuation!
     self.notifications = AsyncStream { continuation in

Original file line number	Diff line number	Diff line change
`@@ -130,6 +130,9 @@ package final class TestSourceKitLSPClient: MessageHandler, Sendable {`
`130`	`130`	`(options.swiftPMOrDefault.swiftCompilerFlags ?? []) + ["-module-cache-path", globalModuleCache.path]`
`131`	`131`	`}`
`132`	`132`	`options.backgroundIndexing = enableBackgroundIndexing`
	`133`	`+ if options.sourcekitdRequestTimeout == nil {`
	`134`	`+ options.sourcekitdRequestTimeout = defaultTimeout`
	`135`	`+ }`
`133`	`136`
`134`	`137`	`var notificationYielder: AsyncStream<any NotificationType>.Continuation!`
`135`	`138`	`self.notifications = AsyncStream { continuation in`