diff --git a/Ghidra/Debug/Framework-TraceModeling/src/test/java/ghidra/trace/database/program/DBTraceProgramViewListingTest.java b/Ghidra/Debug/Framework-TraceModeling/src/test/java/ghidra/trace/database/program/DBTraceProgramViewListingTest.java index 2f3a61cddb6..f5f387bd504 100644 --- a/Ghidra/Debug/Framework-TraceModeling/src/test/java/ghidra/trace/database/program/DBTraceProgramViewListingTest.java +++ b/Ghidra/Debug/Framework-TraceModeling/src/test/java/ghidra/trace/database/program/DBTraceProgramViewListingTest.java @@ -15,7 +15,6 @@ */ package ghidra.trace.database.program; -import static ghidra.lifecycle.Unfinished.TODO; import static org.junit.Assert.*; import java.io.IOException; @@ -841,21 +840,6 @@ public void testGetFirstUndefinedData() throws InsufficientBytesException, assertEquals(b.addr(0x4007), cu.getAddress()); } - @Test - @Ignore("wip: Debug/TraceModeling cluster #189") - public void testGetUndefinedRanges() throws InsufficientBytesException, - UnknownInstructionException, CodeUnitInsertionException, CancelledException { - try (Transaction tx = b.startTransaction()) { - b.addData(0, b.addr(0x4000), Undefined4DataType.dataType, b.buf(1, 2, 3, 4)); - b.addInstruction(0, b.addr(0x4005), b.host, b.buf(0xf4, 0)); - } - - TODO(); // Should I expect OTHER ranges in the undefined set? - assertEquals(b.set(b.range(0, 0x3fff), b.range(0x4004), b.range(0x4007, -1)), - listing.getUndefinedRanges( - view.getAddressFactory().getAddressSet(), false, TaskMonitor.DUMMY)); - } - @Test public void testGetDefinedCodeUnitAfter() throws InsufficientBytesException, UnknownInstructionException, CodeUnitInsertionException { diff --git a/docs/testing/ignore-test-inventory.md b/docs/testing/ignore-test-inventory.md index a5b449307fb..78131964532 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` | 35 | +| `wip` | 34 | | `blocked-on` | 19 | | `manual-tool` | 1 | | `flaky` | 0 | -| **Total properly-categorized** | **55** | +| **Total properly-categorized** | **54** | 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.