mirror of
https://github.com/l0ng-ai/tty7.git
synced 2026-09-22 08:02:24 +00:00
The gpui test modules were gated `#[cfg(all(test, unix))]` for one reason, written down in three of their own doc comments: the pane harnesses handed back a `std::os::unix::net::UnixStream`. That reason went away when `quiet_test_pane` grew a loopback fallback (#736) and nobody came back for the gates, so 180 window, pane, switcher, file tree and SCM tests silently did not run on the platform tty7 ships on. Give the pair its own helper, `terminal::view::test_stream_pair`, and let the harnesses hand back `daemon::transport::Stream` — a socketpair on Unix, a loopback pair on Windows, which is what `quiet_test_pane` already returned. Everything downstream then only names the portable type, and the gates come off. `--bin tty7-app` on Windows goes from 1468 tests to 1638. What stays gated stays for a reason now named at the gate rather than assumed: `/etc/hosts` is not an absolute path on Windows, so a pane's link probe never wants it; and `git rev-parse --show-toplevel` prints `C:/Users/…` while the OS spells the same directory `C:\Users\…`, so the SCM panel, the commit detail and the diff overlay never recognise the repository they are already showing. Both are product divergences, not test ones, and neither is fixed here.