Avoid null-dereference of _languageServer in OnDebuggerStop handler

andyleejordan · andyleejordan · commit b4a2ab5a5d36 · 2021-08-11T13:29:41.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
@@ -2392,7 +2392,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
`@@ -2392,7 +2392,7 @@ private void OnDebuggerStop(object sender, DebuggerStopEventArgs e)`
`2392`	`2392`
`2393`	`2393`	`if (!IsDebugServerActive)`
`2394`	`2394`	`{`
`2395`		`- _languageServer.SendNotification("powerShell/startDebugger");`
	`2395`	`+ _languageServer?.SendNotification("powerShell/startDebugger");`
`2396`	`2396`	`}`
`2397`	`2397`
`2398`	`2398`	`// We've hit a breakpoint so go to a new line so that the prompt can be rendered.`