diff --git a/Ghidra/Debug/Framework-TraceModeling/src/test/java/ghidra/trace/database/listing/DBTraceCodeUnitTest.java b/Ghidra/Debug/Framework-TraceModeling/src/test/java/ghidra/trace/database/listing/DBTraceCodeUnitTest.java index aa029b35e0f..ad15be991fd 100644 --- a/Ghidra/Debug/Framework-TraceModeling/src/test/java/ghidra/trace/database/listing/DBTraceCodeUnitTest.java +++ b/Ghidra/Debug/Framework-TraceModeling/src/test/java/ghidra/trace/database/listing/DBTraceCodeUnitTest.java @@ -24,7 +24,6 @@ import org.junit.*; import db.*; -import ghidra.app.plugin.assembler.*; import ghidra.docking.settings.Settings; import ghidra.program.model.address.*; import ghidra.program.model.data.*; @@ -848,14 +847,6 @@ public void testGetBytes() throws Exception { assertEquals(new BigInteger("-115", 16), lil.getBigInteger(0, 2, true)); } - @Test - @Ignore("wip: Debug/TraceModeling cluster #189") - public void testFigureOutAssembly() throws AssemblySyntaxException, AssemblySemanticException { - Assembler asm = Assemblers.getAssembler(b.language); - System.out.println( - NumericUtilities.convertBytesToString(asm.assembleLine(b.addr(0x4024), "call 0x4004"))); - } - @Test public void testDataValueGetters() throws TraceOverlappedRegionException, CodeUnitInsertionException { diff --git a/docs/testing/ignore-test-inventory.md b/docs/testing/ignore-test-inventory.md index a89a798ecd9..a5b449307fb 100644 --- a/docs/testing/ignore-test-inventory.md +++ b/docs/testing/ignore-test-inventory.md @@ -23,11 +23,11 @@ Live count from `grep -rn '@Ignore("' Ghidra --include='*.java' | grep -v Repeat | Category | Count | |---|---| -| `wip` | 36 | +| `wip` | 35 | | `blocked-on` | 19 | | `manual-tool` | 1 | | `flaky` | 0 | -| **Total properly-categorized** | **56** | +| **Total properly-categorized** | **55** | The `manual-tool` count dropped from 10 → 8 and `flaky` from 3 → 0 in [PR #28-6a](#sequencing): five method-level `@Ignore` lines were removed because their enclosing classes are already `@Ignore`'d at class level (two in `JdiExperimentsTest`/`ProjectExperimentsTest`, three in `JavaMethodsTest`). The method annotations were dead — the class-level annotation skipped them first. Issue references (#178, #190, #193) remain valid; if the class-level ignores are ever lifted, those issues can be re-attached at the method level.