Skip to content

Integration suite: 7 remaining failures after keymap fix (PR #165) #166

@BunsDev

Description

@BunsDev

After #165 lands (1-line keymap fix that unblocks ~46 tests panicking at the KeymapMatcher validator), the integration suite drops from 50+ failures to 7 remaining, all on macOS / Linux / Windows in CI and locally.

Tracking them here so they can be picked up individually (or as a single follow-up sweep) when there's bandwidth for proper Rust investigation.

Remaining failures (local macOS reproduction, 2026-05-30)

Vertical context menu metadata copy (2 tests)

  • ui_tests::test_vertical_tab_context_menu_copies_metadata
  • ui_tests::test_vertical_pane_context_menu_copies_metadata

Symptom: Both panic with Test step 'Copy branch from {tab,pane} context menu' failed after 1 attempts: No position for Copy branch.

Diagnosis (partial): The harness opens the context menu and tries to click the "Copy branch" item. tab.rs:354 only pushes that item if terminal_view.current_git_label(ctx) returns Some. The previous assertion in the test (assert_current_git_branch(0, METADATA_BRANCH) at workspace.rs:201) passes, so the model has the branch. So either:

  • A timing/ordering issue between current_git_branch (assertion path) and current_git_label (menu builder path)
  • A cast-codes-fork-specific change broke one of those two readers
  • A snapshot/state copy is taking a stale view at menu-build time

Worth git blame-ing app/src/tab.rs around line 354 and the model layer that backs current_git_label.

SSH harness (5 tests)

  • shell_integration_tests::test_legacy_ssh_into_bash
  • shell_integration_tests::test_legacy_ssh_into_zsh
  • shell_integration_tests::test_ssh_into_sh
  • shell_integration_tests::test_ssh_into_ash
  • ui_tests::test_ssh_with_shell_override

Diagnosis: Not investigated past confirming they fail; likely a single shared harness issue (sshd setup, container/image, or test helper drift). Worth running one with RUST_BACKTRACE=1 --nocapture to capture the exact failure, then grouping the others against the same root cause.

Tests that passed unexpectedly after the keymap fix landed locally

Local run reference

cd /path/to/cast-codes
cargo test -p integration --test integration

With #165 applied: 211 passed, 7 failed, 63 ignored. Was ~50+ failures + panics before #165.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions