From ead04cf6448b0ce47a2b6c1fc67ad7a61eb139e3 Mon Sep 17 00:00:00 2001 From: dragonmacher <48328597+dragonmacher@users.noreply.github.com> Date: Tue, 23 Jul 2019 11:58:15 -0400 Subject: [PATCH] Tests - fixed test failing due to new provider action --- .../plugin/core/memory/MemoryMapProvider1Test.java | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/Ghidra/Features/Base/src/test.slow/java/ghidra/app/plugin/core/memory/MemoryMapProvider1Test.java b/Ghidra/Features/Base/src/test.slow/java/ghidra/app/plugin/core/memory/MemoryMapProvider1Test.java index bf4c4ac0a0..de72b1987f 100644 --- a/Ghidra/Features/Base/src/test.slow/java/ghidra/app/plugin/core/memory/MemoryMapProvider1Test.java +++ b/Ghidra/Features/Base/src/test.slow/java/ghidra/app/plugin/core/memory/MemoryMapProvider1Test.java @@ -119,7 +119,7 @@ public class MemoryMapProvider1Test extends AbstractGhidraHeadedIntegrationTest Set actions = getActionsByOwner(tool, plugin.getName()); for (DockingActionIf action : actions) { if (action.getName().equals("Merge Blocks")) { - assertTrue(!action.isEnabled()); + assertFalse(action.isEnabled()); } else { assertTrue(action.isEnabled()); @@ -137,11 +137,14 @@ public class MemoryMapProvider1Test extends AbstractGhidraHeadedIntegrationTest String name = action.getName(); if (name.equals("Add Block") || name.equals("Merge Blocks") || name.equals("Delete Block") || name.equals("Set Image Base") || - name.equals("Memory Map")) { - assertTrue(action.isEnabled()); + name.equals("Memory Map") || name.equals("Close Window")) { + assertTrue("Action should be enabled for a multi-row selection - '" + name + "'", + action.isEnabled()); } else { - assertTrue(!action.isEnabled()); + assertFalse( + "Action should not be enabled for a multi-row selection - '" + name + "'", + action.isEnabled()); } } }