use Language instead of String for DoccDocumentationError.unsupportedLanguage

Author: matthewbastien

Sources/DocCDocumentation/DoccDocumentationError.swift

@@ -14,15 +14,15 @@ import Foundation
 package import LanguageServerProtocol
 
 package enum DocCDocumentationError: LocalizedError {
-  case unsupportedLanguage(String)
+  case unsupportedLanguage(Language)
   case noDocumentableSymbols
   case indexNotAvailable
   case symbolNotFound(String)
 
   var errorDescription: String? {
     switch self {
     case .unsupportedLanguage(let language):
-      return "Documentation preview is not available for \(language) files"
+      return "Documentation preview is not available for \(language.description) files"
     case .noDocumentableSymbols:
       return "No documentable symbols were found in this Swift file"
     case .indexNotAvailable:

Sources/SourceKitLSP/Clang/ClangLanguageService.swift

@@ -500,7 +500,7 @@ extension ClangLanguageService {
     }
 
     let snapshot = try sourceKitLSPServer.documentManager.latestSnapshot(req.textDocument.uri)
-    throw ResponseError.requestFailed(doccDocumentationError: .unsupportedLanguage(snapshot.language.description))
+    throw ResponseError.requestFailed(doccDocumentationError: .unsupportedLanguage(snapshot.language))
   }
   #endif
 

Sources/SourceKitLSP/Documentation/DoccDocumentationHandler.swift

@@ -105,7 +105,7 @@ extension DocumentationLanguageService {
         catalogURL: catalogURL
       )
     default:
-      throw ResponseError.requestFailed(doccDocumentationError: .unsupportedLanguage(snapshot.language.description))
+      throw ResponseError.requestFailed(doccDocumentationError: .unsupportedLanguage(snapshot.language))
     }
   }
 }

Tests/SourceKitLSPTests/DoccDocumentationTests.swift

@@ -26,17 +26,17 @@ final class DoccDocumentationTests: XCTestCase {
     try await renderDocumentation(
       markedText: "1️⃣",
       language: .c,
-      expectedResponses: ["1️⃣": .error(.unsupportedLanguage("C"))]
+      expectedResponses: ["1️⃣": .error(.unsupportedLanguage(.c))]
     )
     try await renderDocumentation(
       markedText: "2️⃣",
       language: .cpp,
-      expectedResponses: ["2️⃣": .error(.unsupportedLanguage("C++"))]
+      expectedResponses: ["2️⃣": .error(.unsupportedLanguage(.cpp))]
     )
     try await renderDocumentation(
       markedText: "3️⃣",
       language: .objective_c,
-      expectedResponses: ["3️⃣": .error(.unsupportedLanguage("Objective-C"))]
+      expectedResponses: ["3️⃣": .error(.unsupportedLanguage(.objective_c))]
     )
   }