From 3c9dc9296235baf3bbc8ea40d2b9e2e77025144b Mon Sep 17 00:00:00 2001 From: Levi Neuwirth Date: Fri, 7 Aug 2026 22:49:59 +0200 Subject: [PATCH] docs(test): five adopters, four of them decorator families MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The suite header still said "the four call sites", written before the selection painter joined `Viewport::visible_cols`. Four is now the count of decorator FAMILIES — syntax/LSP styling, diagnostic underlines, search washes, `BufferStyleOverlay` — and five is the count of adopters, selection being the fifth. Also points at where selection's own witnesses live, since a reader of this file would otherwise look for them here and find nothing: `paint_local_selection` is private, so they are in `src/editor.rs`. Comment only; no behavior and no assertion changed. Co-Authored-By: Claude Opus 5 (1M context) Claude-Session: https://claude.ai/code/session_016bqGA6s9tTUFzYpbeW3tai --- tests/horizontal_scroll_acceptance.rs | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/tests/horizontal_scroll_acceptance.rs b/tests/horizontal_scroll_acceptance.rs index 39f023e..93261b5 100644 --- a/tests/horizontal_scroll_acceptance.rs +++ b/tests/horizontal_scroll_acceptance.rs @@ -246,10 +246,15 @@ fn round_trip_is_identity_at_a_non_zero_offset() { // or vanished: decorations drifting off the characters they describe, // silently, and only once a window had been scrolled. // -// `Viewport::visible_cols` is the one rule they now share. These -// witnesses pin it from three directions, because the four call sites -// were identical and a single test would have let a missed adopter -// through. +// `Viewport::visible_cols` is the one rule they now share, across FIVE +// adopters: four decorator families — syntax/LSP styling, diagnostic +// underlines, search washes, `BufferStyleOverlay` — plus the selection +// painter, whose own witnesses live in `src/editor.rs` because +// `paint_local_selection` is private. +// +// These witnesses pin the rule from three directions, because the +// decorator sites were textually identical and a single test would have +// let a missed adopter through. // --------------------------------------------------------------------------- use std::sync::{Arc, Mutex};