Merge remote-tracking branch 'origin/GP-0_Dan_testFixes-2023-01-30-1'

This commit is contained in:
Ryan Kurtz
2023-01-30 12:51:22 -05:00
2 changed files with 7 additions and 4 deletions
@@ -400,7 +400,7 @@ public class DebuggerCoordinates {
* @return exactly these same coordinates with the snap/time changed * @return exactly these same coordinates with the snap/time changed
*/ */
public DebuggerCoordinates snapNoResolve(long snap) { public DebuggerCoordinates snapNoResolve(long snap) {
if (time.isSnapOnly() && time.getSnap() == snap) { if (time != null && time.isSnapOnly() && time.getSnap() == snap) {
return this; return this;
} }
TraceSchedule newTime = TraceSchedule.snap(snap); TraceSchedule newTime = TraceSchedule.snap(snap);
@@ -361,15 +361,18 @@ public class DebuggerModelServiceTest extends AbstractGhidraHeadedDebuggerGUITes
} }
@Test @Test
public void testGetTraceThreadWithTarget() throws Exception { public void testGetTraceThreadWithTarget() throws Throwable {
createTestModel(); createTestModel();
mb.createTestProcessesAndThreads(); mb.createTestProcessesAndThreads();
modelService.recordTarget(mb.testProcess1, createTargetTraceMapper(mb.testProcess1), TraceRecorder recorder =
ActionSource.AUTOMATIC); modelService.recordTarget(mb.testProcess1, createTargetTraceMapper(mb.testProcess1),
ActionSource.AUTOMATIC);
waitRecorder(recorder);
// The most complicated case, lest I want another dimension in a cross product // The most complicated case, lest I want another dimension in a cross product
mb.createTestThreadStacksAndFramesHaveRegisterBanks(); mb.createTestThreadStacksAndFramesHaveRegisterBanks();
waitRecorder(recorder);
waitForPass(() -> { waitForPass(() -> {
TraceThread traceThread = modelService.getTraceThread(mb.testProcess1, mb.testThread1); TraceThread traceThread = modelService.getTraceThread(mb.testProcess1, mb.testThread1);