Revert TestStyle enum changes

Author: plemarquand

Sources/LanguageServerProtocol/SupportTypes/TestItem.swift

@@ -19,15 +19,6 @@ public struct TestTag: Codable, Equatable, Sendable {
   }
 }
 
-/// An enum representing the different styles of tests that can be found in a document.
-public enum TestStyle: String, Codable, Sendable {
-  /// XCTest: https://developer.apple.com/documentation/xctest
-  case xcTest = "XCTest"
-
-  /// Swift Testing: https://swiftpackageindex.com/apple/swift-testing/main/documentation/testing
-  case swiftTesting = "swift-testing"
-}
-
 /// A test item that can be shown an a client's test explorer or used to identify tests alongside a source file.
 ///
 /// A `TestItem` can represent either a test suite or a test itself, since they both have similar capabilities.
@@ -52,7 +43,7 @@ public struct TestItem: ResponseType, Equatable {
   public var disabled: Bool
 
   /// The type of test, eg. the testing framework that was used to declare the test.
-  public var style: TestStyle
+  public var style: String
 
   /// The location of the test item in the source code.
   public var location: Location
@@ -71,7 +62,7 @@ public struct TestItem: ResponseType, Equatable {
     description: String? = nil,
     sortText: String? = nil,
     disabled: Bool,
-    style: TestStyle,
+    style: String,
     location: Location,
     children: [TestItem],
     tags: [TestTag]

Sources/SourceKitLSP/Swift/SwiftTestingScanner.swift

@@ -257,7 +257,7 @@ final class SyntacticSwiftTestingTestScanner: SyntaxVisitor {
         id: (parentTypeNames + typeNames).joined(separator: "/"),
         label: attributeData?.displayName ?? typeNames.last!,
         disabled: (attributeData?.isDisabled ?? false) || allTestsDisabled,
-        style: .swiftTesting,
+        style: TestStyle.swiftTesting,
         location: Location(uri: snapshot.uri, range: range),
         children: memberScanner.result.map(\.testItem),
         tags: attributeData?.tags.map(TestTag.init(id:)) ?? []
@@ -330,7 +330,7 @@ final class SyntacticSwiftTestingTestScanner: SyntaxVisitor {
         id: (parentTypeNames + [name]).joined(separator: "/"),
         label: attributeData.displayName ?? name,
         disabled: attributeData.isDisabled || allTestsDisabled,
-        style: .swiftTesting,
+        style: TestStyle.swiftTesting,
         location: Location(uri: snapshot.uri, range: range),
         children: [],
         tags: attributeData.tags.map(TestTag.init(id:))

Sources/SourceKitLSP/Swift/SyntacticSwiftXCTestScanner.swift

@@ -72,7 +72,7 @@ final class SyntacticSwiftXCTestScanner: SyntaxVisitor {
         id: "\(containerName)/\(function.name.text)()",
         label: "\(function.name.text)()",
         disabled: false,
-        style: .xcTest,
+        style: TestStyle.xcTest,
         location: Location(uri: snapshot.uri, range: range),
         children: [],
         tags: []
@@ -106,7 +106,7 @@ final class SyntacticSwiftXCTestScanner: SyntaxVisitor {
         id: node.name.text,
         label: node.name.text,
         disabled: false,
-        style: .xcTest,
+        style: TestStyle.xcTest,
         location: Location(uri: snapshot.uri, range: range),
         children: testMethods,
         tags: []

Sources/SourceKitLSP/TestDiscovery.swift

@@ -16,6 +16,11 @@ import LanguageServerProtocol
 import SemanticIndex
 import SwiftSyntax
 
+public enum TestStyle {
+  public static let xcTest = "XCTest"
+  public static let swiftTesting = "swift-testing"
+}
+
 fileprivate extension SymbolOccurrence {
   /// Assuming that this is a symbol occurrence returned by the index, return whether it can constitute the definition
   /// of a test case.
@@ -150,7 +155,7 @@ extension SourceKitLSPServer {
           id: id,
           label: testSymbolOccurrence.symbol.name,
           disabled: false,
-          style: .xcTest,
+          style: TestStyle.xcTest,
           location: location,
           children: children.map(\.testItem),
           tags: []
@@ -216,7 +221,7 @@ extension SourceKitLSPServer {
       testsFromSyntacticIndex
       .compactMap { (item) -> AnnotatedTestItem? in
         let testItem = item.testItem
-        if testItem.style == .swiftTesting {
+        if testItem.style == TestStyle.swiftTesting {
           // Swift-testing tests aren't part of the semantic index. Always include them.
           return item
         }
@@ -292,7 +297,7 @@ extension SourceKitLSPServer {
 
     if let index = workspace.index(checkedFor: indexCheckLevel) {
       var syntacticSwiftTestingTests: [AnnotatedTestItem] {
-        syntacticTests?.filter { $0.testItem.style == .swiftTesting } ?? []
+        syntacticTests?.filter { $0.testItem.style == TestStyle.swiftTesting } ?? []
       }
 
       let testSymbols =