Automatically teardown settings for diagnotics tests

Moved these tests over to the fixture to make teardown less error

Author: award999

assets/test/.vscode/settings.json

@@ -7,5 +7,7 @@
         "-DTEST_ARGUMENT_SET_VIA_TEST_BUILD_ARGUMENTS_SETTING"
     ],
     "lldb.verboseLogging": true,
-    "swift.backgroundCompilation": false
+    "swift.backgroundCompilation": false,
+    "swift.diagnosticsStyle": "llvm",
+    "swift.diagnosticsCollection": "onlySwiftc"
 }

src/configuration.ts

@@ -29,6 +29,7 @@ export type DiagnosticCollectionOptions =
     | "keepSwiftc"
     | "keepSourceKit"
     | "keepAll";
+export type DiagnosticStyle = "default" | "llvm" | "swift";
 
 /** sourcekit-lsp configuration */
 export interface LSPConfiguration {
@@ -288,8 +289,10 @@ const configuration = {
             .get<DiagnosticCollectionOptions>("diagnosticsCollection", "keepSourceKit");
     },
     /** set the -diagnostic-style option when running `swift` tasks */
-    get diagnosticsStyle(): "default" | "llvm" | "swift" {
-        return vscode.workspace.getConfiguration("swift").get("diagnosticsStyle", "llvm");
+    get diagnosticsStyle(): DiagnosticStyle {
+        return vscode.workspace
+            .getConfiguration("swift")
+            .get<DiagnosticStyle>("diagnosticsStyle", "llvm");
     },
     /** where to show the build progress for the running task */
     get showBuildStatus(): ShowBuildStatusOptions {