diff --git a/Ghidra/Features/PDB/src/test/java/ghidra/app/util/pdb/pdbapplicator/CppCompositeTypeTest.java b/Ghidra/Features/PDB/src/test/java/ghidra/app/util/pdb/pdbapplicator/CppCompositeTypeTest.java index 3b79822a332..b75ff915fc7 100644 --- a/Ghidra/Features/PDB/src/test/java/ghidra/app/util/pdb/pdbapplicator/CppCompositeTypeTest.java +++ b/Ghidra/Features/PDB/src/test/java/ghidra/app/util/pdb/pdbapplicator/CppCompositeTypeTest.java @@ -2772,25 +2772,4 @@ else if (e.getValue() instanceof ProgramVirtualFunctionTable vft) { } - //============================================================================================== - //============================================================================================== - - /** - * Test struct J5 - 32 - syntactics - * @throws Exception upon error - */ -// @Test - @Ignore("wip: misc cluster — undocumented bare-ignore #192") - public void testJ5_32_syntactic_layout() throws Exception { -// SyntacticClass structJ5 = createSyntacticStructJ5(MsftVxtManager32); - // TODO: determine if we need a builder that inputs a SyntacticClass along with a - // data type manager (w/ data organization), along with layout options for creating - // data type (/class). - //structJ5.createLayoutFromSyntacticDescription(MsftVxtManager32, TaskMonitor.DUMMY); - - // - //Composite composite = structJ5.getComposite(); - //CompositeTestUtils.assertExpectedComposite(this, getExpectedJ5_32(), composite, true); - } - } diff --git a/docs/testing/ignore-test-inventory.md b/docs/testing/ignore-test-inventory.md index 212ff409de3..a89a798ecd9 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` | 37 | +| `wip` | 36 | | `blocked-on` | 19 | | `manual-tool` | 1 | | `flaky` | 0 | -| **Total properly-categorized** | **57** | +| **Total properly-categorized** | **56** | 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.