Walkthrough

This PR enhances test failure marking with inheritance-aware handling, allowing parent class test markers to be set when all child inheritors fail, and enabling stripping of unreasoned failure decorators. It includes a minor formatting adjustment to a Rust debug assertion and comprehensive test updates.

Changes

Sequence Diagram

sequenceDiagram
    participant FileSystem as File System
    participant AutoMark as Auto-Mark Engine
    participant CodeAnalysis as Code Analysis
    participant TestRunner as Test Runner
    participant Consolidator as Consolidator

    FileSystem->>AutoMark: Load test file
    AutoMark->>AutoMark: Strip reasonless `@expectedFailure` decorators
    AutoMark->>FileSystem: Write modified file
    AutoMark->>TestRunner: Run tests
    TestRunner->>TestRunner: Collect failures (parent & child classes)
    TestRunner->>AutoMark: Return failing tests
    AutoMark->>CodeAnalysis: Expand stripped markers to inheritors
    CodeAnalysis->>CodeAnalysis: Find all classes inheriting methods
    CodeAnalysis->>AutoMark: Return expanded failure set
    AutoMark->>Consolidator: Attempt consolidation
    Consolidator->>Consolidator: Check if all inheritors failed
    Consolidator->>Consolidator: Consolidate to parent class
    Consolidator->>AutoMark: Return consolidated failures
    AutoMark->>FileSystem: Apply `@expectedFailure` markers

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~50 minutes

Possibly related PRs

• Improve deps output #6874: Modifies the same scripts/update_lib/cmd_auto_mark.py module with direct code-level changes to auto-marking logic.
• auto mark parent tests #6778: Adds inheritance-aware auto-marking logic with class hierarchy resolution and parent consolidation functionality.
• rename fix_test and support removing unexpected success #6748: Modifies expected-failure removal and super-call detection logic that overlaps with removal range computation.

Suggested reviewers

• moreal
• ShaharNaveh

Poem

🐰 Hops through class hierarchies with glee,
Parent and child, now in harmony!
Strip the unmarked, expand the bound,
Consolidate failures all around,
Test inheritance, safe and sound! ✨

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

• 📝 Generate docstrings

• X
• Mastodon
• Reddit
• LinkedIn