Clean up tests in DebugServiceTests.cs

Author: andyleejordan

.editorconfig

@@ -12,6 +12,7 @@ indent_size = 4
 trim_trailing_whitespace = true
 csharp_space_before_open_square_brackets = true
 csharp_space_after_keywords_in_control_flow_statements = true
+csharp_space_before_open_square_brackets = false
 
 # CS0168: The variable 'var' is declared but never used
 dotnet_diagnostic.CS0168.severity = error

test/PowerShellEditorServices.Test.E2E/DebugAdapterProtocolMessageTests.cs

@@ -260,12 +260,7 @@ public async Task CanStepPastSystemWindowsForms()
                 new SetFunctionBreakpointsArguments
                 {
                     Breakpoints = new FunctionBreakpoint[]
-                    {
-                        new FunctionBreakpoint
-                        {
-                            Name = "Write-Host",
-                        }
-                    }
+                        { new FunctionBreakpoint { Name = "Write-Host", } }
                 }).ConfigureAwait(false);
 
             var breakpoint = setBreakpointsResponse.Breakpoints.First();

test/PowerShellEditorServices.Test/Debugging/DebugServiceTests.cs

@@ -114,7 +114,7 @@ await this.debugService.SetCommandBreakpointsAsync(
             await executeTask.ConfigureAwait(false);
 
             StackFrameDetails[] stackFrames = debugService.GetStackFrames();
-            Assert.Equal(StackFrameDetails.NoFileScriptPath, stackFrames [0].ScriptPath);
+            Assert.Equal(StackFrameDetails.NoFileScriptPath, stackFrames[0].ScriptPath);
 
             VariableDetailsBase[] variables =
                 debugService.GetVariables(stackFrames[0].LocalVariables.Id);
@@ -151,7 +151,7 @@ public async Task DebuggerAcceptsScriptArgs(string[] args)
 
             await this.debugService.SetLineBreakpointsAsync(
                 debugWithParamsFile,
-                new [] { BreakpointDetails.Create(debugWithParamsFile.FilePath, 3) }).ConfigureAwait(false);
+                new[] { BreakpointDetails.Create(debugWithParamsFile.FilePath, 3) }).ConfigureAwait(false);
 
             string arguments = string.Join(" ", args);
 
@@ -580,8 +580,6 @@ await this.AssertStateChange(
                 PowerShellContextState.Ready,
                 PowerShellExecutionResult.Stopped).ConfigureAwait(false);
 
-            // Abort script execution early and wait for completion
-            this.debugService.Abort();
             await executeTask.ConfigureAwait(false);
         }
 
@@ -872,7 +870,7 @@ await this.debugService.SetLineBreakpointsAsync(
             Assert.Equal("[1]", childVars[1].Name);
 
             var childVarStrs = new HashSet<string>(childVars.Select(v => v.ValueString));
-            var expectedVars = new [] {
+            var expectedVars = new[] {
                 "[firstChild, \"Child\"]",
                 "[secondChild, 42]"
             };
@@ -1026,15 +1024,15 @@ await this.debugService.SetLineBreakpointsAsync(
             await executeTask.ConfigureAwait(false);
         }
 
-        public async Task AssertDebuggerPaused()
+        private async Task AssertDebuggerPaused()
         {
             DebuggerStoppedEventArgs eventArgs =
                 await this.debuggerStoppedQueue.DequeueAsync(new CancellationTokenSource(10000).Token).ConfigureAwait(false);
 
             Assert.Empty(eventArgs.OriginalEvent.Breakpoints);
         }
 
-        public async Task AssertDebuggerStopped(
+        private async Task AssertDebuggerStopped(
             string scriptPath,
             int lineNumber = -1)
         {