Merge pull request #5965 from dibarbet/debug_tests

Implement client side of test debugging

Author: dibarbet

package.json

@@ -36,7 +36,7 @@
     }
   },
   "defaults": {
-    "roslyn": "4.8.0-1.23372.1",
+    "roslyn": "4.8.0-1.23381.8",
     "omniSharp": "1.39.7",
     "razor": "7.0.0-preview.23328.2",
     "razorOmnisharp": "7.0.0-preview.23363.1"

src/lsptoolshost/roslynLanguageServer.ts

@@ -207,6 +207,8 @@ export class RoslynLanguageServer {
 
         // Register Razor dynamic file info handling
         this.registerDynamicFileInfo(this._languageClient);
+
+        this.registerDebuggerAttach(this._languageClient);
     }
 
     public async stop(): Promise<void> {
@@ -488,6 +490,25 @@ export class RoslynLanguageServer {
         );
     }
 
+    private registerDebuggerAttach(client: RoslynLanguageClient) {
+        client.onRequest<RoslynProtocol.DebugAttachParams, RoslynProtocol.DebugAttachResult, void>(
+            RoslynProtocol.DebugAttachRequest.type,
+            async (request) => {
+                const debugConfiguration: vscode.DebugConfiguration = {
+                    name: '.NET Core Attach',
+                    type: 'coreclr',
+                    request: 'attach',
+                    processId: request.processId,
+                };
+
+                const result = await vscode.debug.startDebugging(undefined, debugConfiguration, undefined);
+                return {
+                    didAttach: result,
+                };
+            }
+        );
+    }
+
     private registerExtensionsChanged(languageClient: RoslynLanguageClient) {
         // subscribe to extension change events so that we can get notified if C# Dev Kit is added/removed later.
         languageClient.addDisposable(

src/lsptoolshost/roslynProtocol.ts

@@ -76,6 +76,11 @@ export interface RunTestsParams extends lsp.WorkDoneProgressParams, lsp.PartialR
      * Note that this does not have to only include tests, for example this could be a range representing a class.
      */
     range: lsp.Range;
+
+    /**
+     * Whether the request should attempt to call back to the client to attach a debugger before running the tests.
+     */
+    attachDebugger: boolean;
 }
 
 export interface TestProgress {
@@ -103,6 +108,14 @@ export interface RunTestsPartialResult {
     progress?: TestProgress;
 }
 
+export interface DebugAttachParams {
+    processId: number;
+}
+
+export interface DebugAttachResult {
+    didAttach: boolean;
+}
+
 export namespace WorkspaceDebugConfigurationRequest {
     export const method = 'workspace/debugConfiguration';
     export const messageDirection: lsp.MessageDirection = lsp.MessageDirection.clientToServer;
@@ -140,3 +153,9 @@ export namespace RunTestsRequest {
         void
     >(method);
 }
+
+export namespace DebugAttachRequest {
+    export const method = 'workspace/attachDebugger';
+    export const messageDirection: lsp.MessageDirection = lsp.MessageDirection.serverToClient;
+    export const type = new lsp.RequestType<DebugAttachParams, DebugAttachResult, void>(method);
+}

src/lsptoolshost/unitTesting.ts

@@ -21,19 +21,31 @@ export function registerUnitTestingCommands(
     context.subscriptions.push(
         vscode.commands.registerTextEditorCommand(
             'dotnet.test.runTestsInContext',
-            async (textEditor: vscode.TextEditor) => runTestsInContext(textEditor, languageServer, dotnetTestChannel)
+            async (textEditor: vscode.TextEditor) => {
+                return runTestsInContext(false, textEditor, languageServer, dotnetTestChannel);
+            }
+        )
+    );
+
+    context.subscriptions.push(
+        vscode.commands.registerTextEditorCommand(
+            'dotnet.test.debugTestsInContext',
+            async (textEditor: vscode.TextEditor) => {
+                return runTestsInContext(true, textEditor, languageServer, dotnetTestChannel);
+            }
         )
     );
 }
 
 async function runTestsInContext(
+    debug: boolean,
     textEditor: vscode.TextEditor,
     languageServer: RoslynLanguageServer,
     dotnetTestChannel: vscode.OutputChannel
 ) {
     const contextRange: languageClient.Range = { start: textEditor.selection.active, end: textEditor.selection.active };
     const textDocument: languageClient.TextDocumentIdentifier = { uri: textEditor.document.fileName };
-    const request: RunTestsParams = { textDocument: textDocument, range: contextRange };
+    const request: RunTestsParams = { textDocument: textDocument, range: contextRange, attachDebugger: debug };
     await runTests(request, languageServer, dotnetTestChannel);
 }