Make response of workspace/_sourceKitOptions request non-optional

ahoppen · ahoppen · commit 112252632d3d · 2025-02-27T19:03:50.000-08:00
The response was not specified as being optional and shouldn’t be.
diff --git a/Sources/LanguageServerProtocol/Requests/SourceKitOptionsRequest.swift b/Sources/LanguageServerProtocol/Requests/SourceKitOptionsRequest.swift
@@ -19,7 +19,7 @@
 /// **(LSP Extension)**.
 public struct SourceKitOptionsRequest: RequestType, Hashable {
   public static let method: String = "workspace/_sourceKitOptions"
-  public typealias Response = SourceKitOptionsResponse?
+  public typealias Response = SourceKitOptionsResponse
 
   /// The document to get options for
   public var textDocument: TextDocumentIdentifier
diff --git a/Tests/BuildSystemIntegrationTests/BuildServerBuildSystemTests.swift b/Tests/BuildSystemIntegrationTests/BuildServerBuildSystemTests.swift
@@ -421,7 +421,7 @@ final class BuildServerBuildSystemTests: XCTestCase {
         allowFallbackSettings: false
       )
     )
-    XCTAssertEqual(options?.data, LSPAny.dictionary(["custom": .string("value")]))
+    XCTAssertEqual(options.data, LSPAny.dictionary(["custom": .string("value")]))
   }
 
   func testBuildSettingsForFilePartOfMultipleTargets() async throws {

Original file line number	Diff line number	Diff line change
`@@ -421,7 +421,7 @@ final class BuildServerBuildSystemTests: XCTestCase {`
`421`	`421`	`allowFallbackSettings: false`
`422`	`422`	`)`
`423`	`423`	`)`
`424`		`- XCTAssertEqual(options?.data, LSPAny.dictionary(["custom": .string("value")]))`
	`424`	`+ XCTAssertEqual(options.data, LSPAny.dictionary(["custom": .string("value")]))`
`425`	`425`	`}`
`426`	`426`
`427`	`427`	`func testBuildSettingsForFilePartOfMultipleTargets() async throws {`