Avoid null-dereference of _languageServer in OnDebuggerStop handler

andyleejordan · andyleejordan · commit b6571a57ec32 · 2021-08-11T17:27:06.000-07:00
This primarily occured during unit testing, where the language server is
not instantiated. Note that elsewhere in this file it is treated
similarly.
diff --git a/src/PowerShellEditorServices/Services/PowerShellContext/PowerShellContextService.cs b/src/PowerShellEditorServices/Services/PowerShellContext/PowerShellContextService.cs
@@ -2416,7 +2416,7 @@ private void OnDebuggerStop(object sender, DebuggerStopEventArgs e)
 
             if (!IsDebugServerActive)
             {
-                _languageServer.SendNotification("powerShell/startDebugger");
+                _languageServer?.SendNotification("powerShell/startDebugger");
             }
 
             // We've hit a breakpoint so go to a new line so that the prompt can be rendered.

Original file line number	Diff line number	Diff line change
`@@ -2416,7 +2416,7 @@ private void OnDebuggerStop(object sender, DebuggerStopEventArgs e)`
`2416`	`2416`
`2417`	`2417`	`if (!IsDebugServerActive)`
`2418`	`2418`	`{`
`2419`		`- _languageServer.SendNotification("powerShell/startDebugger");`
	`2419`	`+ _languageServer?.SendNotification("powerShell/startDebugger");`
`2420`	`2420`	`}`
`2421`	`2421`
`2422`	`2422`	`// We've hit a breakpoint so go to a new line so that the prompt can be rendered.`