Description
Line 41 of TestVmClass.java has a bare // FIXME comment immediately above the @Ignore annotation:
// FIXME
@Ignore
public class TestVmClass {The FIXME is functionally dead — it was the justification for ignoring the test, but the FIXME has been stale for years. The comment provides no useful information to a future developer (it doesn't say what to fix).
What to do
Delete line 41 (// FIXME). The @Ignore annotation speaks for itself.
Scope
This ticket is for the FIXME line only. The leftover System.out.println calls in testAssignableFrom (lines 73, 79, 81) are a separate issue.
Why
Dead comment. @Ignore + the static block in setUp are enough context. A bare FIXME without context is just noise.
Description
Line 41 of
TestVmClass.javahas a bare// FIXMEcomment immediately above the@Ignoreannotation:The FIXME is functionally dead — it was the justification for ignoring the test, but the FIXME has been stale for years. The comment provides no useful information to a future developer (it doesn't say what to fix).
What to do
Delete line 41 (
// FIXME). The@Ignoreannotation speaks for itself.Scope
This ticket is for the FIXME line only. The leftover
System.out.printlncalls intestAssignableFrom(lines 73, 79, 81) are a separate issue.Why
Dead comment.
@Ignore+ the static block in setUp are enough context. A bare FIXME without context is just noise.