Skip to content

Commit 11d9a2f

Browse files
committed
Minor cleanup
1 parent 1a248a4 commit 11d9a2f

File tree

1 file changed

+12
-5
lines changed

1 file changed

+12
-5
lines changed

test/PowerShellEditorServices.Test.E2E/DebugAdapterProtocolMessageTests.cs

+12-5
Original file line numberDiff line numberDiff line change
@@ -237,6 +237,7 @@ public async Task UsesDotSourceOperatorAndQuotesAsync()
237237
await client.LaunchScript(filePath);
238238
ConfigurationDoneResponse configDoneResponse = await client.RequestConfigurationDone(new ConfigurationDoneArguments());
239239
Assert.NotNull(configDoneResponse);
240+
240241
string actual = await ReadScriptLogLineAsync();
241242
Assert.StartsWith(". '", actual);
242243
}
@@ -248,6 +249,7 @@ public async Task UsesCallOperatorWithSettingAsync()
248249
await client.LaunchScript(filePath, executeMode: "Call");
249250
ConfigurationDoneResponse configDoneResponse = await client.RequestConfigurationDone(new ConfigurationDoneArguments());
250251
Assert.NotNull(configDoneResponse);
252+
251253
string actual = await ReadScriptLogLineAsync();
252254
Assert.StartsWith("& '", actual);
253255
}
@@ -261,7 +263,9 @@ public async Task CanLaunchScriptWithNoBreakpointsAsync()
261263

262264
ConfigurationDoneResponse configDoneResponse = await client.RequestConfigurationDone(new ConfigurationDoneArguments());
263265
Assert.NotNull(configDoneResponse);
264-
Assert.Equal("works", await ReadScriptLogLineAsync());
266+
267+
string actual = await ReadScriptLogLineAsync();
268+
Assert.Equal("works", actual);
265269
}
266270

267271
[SkippableFact]
@@ -308,8 +312,11 @@ public async Task CanSetBreakpointsAsync()
308312

309313
_ = await client.RequestContinue(new ContinueArguments { ThreadId = 1 });
310314

311-
Assert.Equal("at breakpoint", await ReadScriptLogLineAsync());
312-
Assert.Equal("after breakpoint", await ReadScriptLogLineAsync());
315+
string atBreakpointActual = await ReadScriptLogLineAsync();
316+
Assert.Equal("at breakpoint", atBreakpointActual);
317+
318+
string afterBreakpointActual = await ReadScriptLogLineAsync();
319+
Assert.Equal("after breakpoint", afterBreakpointActual);
313320
}
314321

315322
[SkippableFact]
@@ -337,7 +344,7 @@ await client.SetBreakpoints(
337344
await client.RequestConfigurationDone(new ConfigurationDoneArguments());
338345
await client.LaunchScript(filePath);
339346

340-
// Try to get the stacktrace. If we are not at a breakpoint, this should fail.
347+
// Try to get the stacktrace, which should throw as we are not currently at a breakpoint.
341348
await Assert.ThrowsAsync<JsonRpcException>(() => client.RequestStackTrace(
342349
new StackTraceArguments { }
343350
));
@@ -353,7 +360,7 @@ public async Task SendsInitialLabelBreakpointForPerformanceReasons()
353360
"label breakpoint"
354361
));
355362

356-
// Trigger a launch. Note that per DAP spec, launch doesn't actually begin until ConfigDone finishes.
363+
// Request a launch. Note that per DAP spec, launch doesn't actually begin until ConfigDone finishes.
357364
await client.LaunchScript(filePath);
358365

359366
SetBreakpointsResponse setBreakpointsResponse = await client.SetBreakpoints(new SetBreakpointsArguments

0 commit comments

Comments
 (0)