From 732d37d290a58454b79cd141ad076deb77e94dc6 Mon Sep 17 00:00:00 2001 From: dragonmacher <48328597+dragonmacher@users.noreply.github.com> Date: Tue, 21 Sep 2021 09:55:19 -0400 Subject: [PATCH] Added test debug to track down a timing issue in parallel test mode --- .../core/analysis/AnalysisOptionsTest.java | 26 ++++++++----------- 1 file changed, 11 insertions(+), 15 deletions(-) diff --git a/Ghidra/Features/Base/src/test.slow/java/ghidra/app/plugin/core/analysis/AnalysisOptionsTest.java b/Ghidra/Features/Base/src/test.slow/java/ghidra/app/plugin/core/analysis/AnalysisOptionsTest.java index 1672a3e94f..42df50ae91 100644 --- a/Ghidra/Features/Base/src/test.slow/java/ghidra/app/plugin/core/analysis/AnalysisOptionsTest.java +++ b/Ghidra/Features/Base/src/test.slow/java/ghidra/app/plugin/core/analysis/AnalysisOptionsTest.java @@ -18,7 +18,6 @@ package ghidra.app.plugin.core.analysis; import static org.junit.Assert.*; import java.io.File; -import java.util.Collection; import java.util.concurrent.atomic.AtomicBoolean; import javax.swing.ComboBoxModel; @@ -310,11 +309,18 @@ public class AnalysisOptionsTest extends AbstractGhidraHeadedIntegrationTest { private AnalysisOptionsDialog invokeAnalysisDialog() { DockingActionIf action = getAction(tool, "Auto Analyze"); performAction(action, false); - return waitForDialogComponent(AnalysisOptionsDialog.class); - } - private void apply() { - pressButtonByText(optionsDialog, "Analyze"); + // TODO temp debug to catch issue seen when running parallel tests + try { + return waitForDialogComponent(AnalysisOptionsDialog.class); + } + catch (Throwable t) { + + printOpenWindows(); + + failWithException("Unable to find analysis dialog", t); + return null; // can't get here + } } private boolean isAnalyzerEnabledInProgramOptions(String analyzerName) { @@ -322,16 +328,6 @@ public class AnalysisOptionsTest extends AbstractGhidraHeadedIntegrationTest { return options.getBoolean(analyzerName, false); } - private void verifyDefaultOptions(Collection programs) { - for (Program program : programs) { - Options options = program.getOptions(Program.ANALYSIS_PROPERTIES); - for (String name : options.getOptionNames()) { - assertTrue("Program options are unexpectedly changed: " + program, - options.isDefaultValue(name)); - } - } - } - private void setAnalyzerEnabled(String name, boolean enabled) { TableModel model = getAnalyzerTableModel(); int analyzerRow = getRowForAnalyzer(name, model);