Merge pull request #2105 from ahoppen/test-timeout-cancellation

ahoppen · web-flow · commit 6a8ea4cd97ea · 2025-04-09T15:23:28.000-07:00
Fix issue that caused request cancellation after a timeout in tests to not work
diff --git a/Sources/SKTestSupport/TestSourceKitLSPClient.swift b/Sources/SKTestSupport/TestSourceKitLSPClient.swift
@@ -261,7 +261,7 @@ package final class TestSourceKitLSPClient: MessageHandler, Sendable {
           .failure(ResponseError.unknown("\(R.method) request timed out after \(defaultTimeoutDuration)"))
         )
       }
-      await server.handle(notification: CancelRequestNotification(id: requestID))
+      server.handle(CancelRequestNotification(id: requestID))
     }
     server.handle(request, id: requestID) { result in
       if replyOutstanding.takeValue() ?? false {

Original file line number	Diff line number	Diff line change
`@@ -261,7 +261,7 @@ package final class TestSourceKitLSPClient: MessageHandler, Sendable {`
`261`	`261`	`.failure(ResponseError.unknown("\(R.method) request timed out after \(defaultTimeoutDuration)"))`
`262`	`262`	`)`
`263`	`263`	`}`
`264`		`- await server.handle(notification: CancelRequestNotification(id: requestID))`
	`264`	`+ server.handle(CancelRequestNotification(id: requestID))`
`265`	`265`	`}`
`266`	`266`	`server.handle(request, id: requestID) { result in`
`267`	`267`	`if replyOutstanding.takeValue() ?? false {`