mirror/ghidra
1
0
Fork 0
mirror of https://github.com/NationalSecurityAgency/ghidra.git synced 2026-09-28 17:11:11 -09:00
Code Issues Packages Projects Releases Wiki Activity

GP-0: Fix breakpoint-marker tests (adjust waitOn's)

Browse Source
This commit is contained in:
Dan
2024-08-28 08:36:25 -04:00
parent 33323f11f2
commit d973740105
1 changed files with 16 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
Ghidra/Test/DebuggerIntegrationTest/src/test/java/ghidra/app/plugin/core/debug/gui/breakpoint/AbstractDebuggerBreakpointMarkerPluginTest.java
View File

@@ -483,9 +483,13 @@ public abstract class AbstractDebuggerBreakpointMarkerPluginTest<T>
addMappedBreakpointOpenAndWait(); // wasteful, but whatever addMappedBreakpointOpenAndWait(); // wasteful, but whatever
for (LogicalBreakpoint lb : List.copyOf(breakpointService.getAllBreakpoints())) { for (LogicalBreakpoint lb : List.copyOf(breakpointService.getAllBreakpoints())) {
TraceBreakpoint brk = Unique.assertOne(lb.getTraceBreakpoints(tb.trace)); TraceBreakpoint brk = Unique.assertOne(lb.getTraceBreakpoints(tb.trace));
waitOn(lb.delete()); CompletableFuture<Void> delete = lb.delete();
handleDeleteBreakpointInvocation(brk); handleDeleteBreakpointInvocation(brk);
waitOn(delete);
} }
waitForDomainObject(program);
waitForDomainObject(tb.trace);
waitOn(breakpointService.changesSettled());
waitForPass(() -> assertEquals(0, breakpointService.getAllBreakpoints().size())); waitForPass(() -> assertEquals(0, breakpointService.getAllBreakpoints().size()));
try (Transaction tx = program.openTransaction("Disassemble")) { try (Transaction tx = program.openTransaction("Disassemble")) {
@@ -501,6 +505,9 @@ public abstract class AbstractDebuggerBreakpointMarkerPluginTest<T>
waitForDialogComponent(DebuggerPlaceBreakpointDialog.class); waitForDialogComponent(DebuggerPlaceBreakpointDialog.class);
runSwing(() -> dialog.okCallback()); runSwing(() -> dialog.okCallback());
handleSetBreakpointInvocation(TraceBreakpointKindSet.SW_EXECUTE, 0x55550123); handleSetBreakpointInvocation(TraceBreakpointKindSet.SW_EXECUTE, 0x55550123);
waitForDomainObject(program);
waitForDomainObject(tb.trace);
waitOn(breakpointService.changesSettled());
waitForPass(() -> { waitForPass(() -> {
LogicalBreakpoint lb = Unique.assertOne(breakpointService.getAllBreakpoints()); LogicalBreakpoint lb = Unique.assertOne(breakpointService.getAllBreakpoints());
@@ -515,9 +522,13 @@ public abstract class AbstractDebuggerBreakpointMarkerPluginTest<T>
addMappedBreakpointOpenAndWait(); // wasteful, but whatever addMappedBreakpointOpenAndWait(); // wasteful, but whatever
for (LogicalBreakpoint lb : List.copyOf(breakpointService.getAllBreakpoints())) { for (LogicalBreakpoint lb : List.copyOf(breakpointService.getAllBreakpoints())) {
TraceBreakpoint brk = Unique.assertOne(lb.getTraceBreakpoints(tb.trace)); TraceBreakpoint brk = Unique.assertOne(lb.getTraceBreakpoints(tb.trace));
waitOn(lb.delete()); CompletableFuture<Void> delete = lb.delete();
handleDeleteBreakpointInvocation(brk); handleDeleteBreakpointInvocation(brk);
waitOn(delete);
} }
waitForDomainObject(program);
waitForDomainObject(tb.trace);
waitOn(breakpointService.changesSettled());
waitForPass(() -> assertEquals(0, breakpointService.getAllBreakpoints().size())); waitForPass(() -> assertEquals(0, breakpointService.getAllBreakpoints().size()));
try (Transaction tx = program.openTransaction("Disassemble")) { try (Transaction tx = program.openTransaction("Disassemble")) {
@@ -532,6 +543,9 @@ public abstract class AbstractDebuggerBreakpointMarkerPluginTest<T>
waitForDialogComponent(DebuggerPlaceBreakpointDialog.class); waitForDialogComponent(DebuggerPlaceBreakpointDialog.class);
runSwing(() -> dialog.okCallback()); runSwing(() -> dialog.okCallback());
handleSetBreakpointInvocation(TraceBreakpointKindSet.ACCESS, 0x55550123); handleSetBreakpointInvocation(TraceBreakpointKindSet.ACCESS, 0x55550123);
waitForDomainObject(program);
waitForDomainObject(tb.trace);
waitOn(breakpointService.changesSettled());
waitForPass(() -> { waitForPass(() -> {
LogicalBreakpoint lb = Unique.assertOne(breakpointService.getAllBreakpoints()); LogicalBreakpoint lb = Unique.assertOne(breakpointService.getAllBreakpoints());