Skip to content

Remove stale // FIXME comments from shell completion tests #485

@LSantha

Description

@LSantha

Description

Two shell test files have a bare // FIXME comment immediately above the @Ignore annotation:

  • shell/src/test/org/jnode/test/shell/CompletionTest.java line 57
  • shell/src/test/org/jnode/test/shell/DefaultSyntaxCompletionTest.java line 53
// FIXME
@Ignore
public class CompletionTest {

The FIXME was the justification for ignoring the test, but it has been stale for years and provides no specific information about what needs fixing.

What to do

Delete the // FIXME line in both files. The @Ignore annotation and the static Cassowary.initEnv() call in setUp are enough context.

Why

Dead comment. @Ignore + the static initialization are sufficient; the FIXME is just noise.

Metadata

Metadata

Assignees

No one assigned

    Labels

    agent/doneThe agent finished successfully; PR opened or comment posted.area/shellshell/ — command framework, built-in commands, bjorne evaluator.kind/choreRefactor, typo sweep, dead-code removal; no behavior change.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions