Tests maintenance: Document @TestWithoutNeovim reasons in InsertBackspaceActionTest

[claude]

Summary

• Reviewed InsertBackspaceActionTest for test maintenance issues
• Added missing description parameters to two @TestWithoutNeovim annotations

Area Inspected

Randomly selected InsertBackspaceActionTest.kt and found missing description parameters on @TestWithoutNeovim annotations.

Issues Found

Two tests were using @TestWithoutNeovim annotations without the required description parameter:

1. Line 17: @TestWithoutNeovim(SkipNeovimReason.DIFFERENT) - lacked explanation of what differs
2. Line 29: @TestWithoutNeovim(SkipNeovimReason.OPTION) - lacked explanation of which option behavior is being tested

Changes Made

Added descriptive explanations to both annotations:

• First test now documents that "Backspace behavior in insert mode differs"
• Second test now documents that it "Tests 'sidescrolloff' and 'sidescroll' option behavior during backspace"

These changes improve test documentation and make it clearer why these tests are excluded from Neovim verification.

🤖 Generated with Claude Code

[claude]

Review Summary

Verdict: Approve with minor suggestions

The changes correctly add description parameters to @TestWithoutNeovim annotations, improving test documentation. However, there are opportunities for more specificity.

Issues Found

Description Quality

The descriptions are somewhat generic. Consider being more specific:

Line 17: "Backspace behavior in insert mode differs"

• What specifically differs? Character deletion? Cursor positioning? State management?
• Suggestion: "IdeaVim deletes 'o' character while Neovim handles backspace differently in insert mode"

Line 29: "Tests 'sidescrolloff' and 'sidescroll' option behavior during backspace"

• This describes what the test does, not WHY it's excluded from Neovim testing
• Suggestion: "IdeaVim scrolling implementation differs from Neovim when sidescrolloff/sidescroll options are set"

Annotation Definition Reference

For context, the annotation definition at NeovimTesting.kt:283 shows the description parameter is optional but recommended for maintainability.

Codebase-Wide Pattern

Note: Many tests use @TestWithoutNeovim without descriptions. This PR follows the same pattern used elsewhere, though more descriptive text would benefit future maintainers.

Test Coverage

✓ No test changes needed - this is purely documentation improvement

Security/Performance

✓ No concerns - annotation-only changes

[claude]

The description "Backspace behavior in insert mode differs" is too generic. Be specific about what differs - is it character deletion behavior, cursor positioning, or state management?

Example: "IdeaVim deletes 'o' character while Neovim handles backspace after cursor movement differently"

[claude]

The description explains what the test does, not WHY it's excluded from Neovim verification. Focus on the difference between implementations.

Example: "IdeaVim scrolling implementation differs from Neovim when sidescrolloff/sidescroll affect backspace positioning"