zot/packages/tui/render_test.go
patriceckhart fa7d8d8be5 refactor: split source into packages/{provider,core,tui,agent}
Single Go module, four top-level packages under packages/. Import
paths become github.com/patriceckhart/zot/packages/<name>; downstream
consumers can depend on individual packages without pulling the rest.

Layout:
  packages/provider/     LLM clients + catalog
  packages/provider/auth/ credential store + OAuth + login server
  packages/core/         agent loop, sessions, cost
  packages/tui/          terminal toolkit + chat view
  packages/agent/        CLI wiring, system prompt
    extensions/ extproto/ modes/ tools/ skills/ swarm/
    sdk/  (was pkg/zotcore, package renamed zotcore -> sdk)
    ext/  (was pkg/zotext, package renamed zotext -> ext)

internal/ and pkg/ removed. The internal/assets logo moved into
packages/provider/auth/assets.

Public Go SDK identifiers renamed:
  pkg/zotcore (package zotcore) -> packages/agent/sdk (package sdk)
  pkg/zotext  (package zotext)  -> packages/agent/ext (package ext)

This breaks Go-based extensions and embedders; the JSON wire protocol
for extensions and RPC is unchanged, so non-Go extensions, already-
built extension binaries, and zot rpc consumers are unaffected.

Docs, examples, and the built-in write-zot-extension skill updated
for the new paths and identifiers. Shadow-bug fixes in code samples
(ext := ext.New -> e := ext.New).
2026-05-27 09:07:15 +02:00

103 lines
3.4 KiB
Go

package tui
import (
"bytes"
"strings"
"testing"
)
// TestDrawLogIdleNoOpEmitsNothing pins the cursor-blink fix: when
// DrawLog is called with the exact same buffer and cursor position
// as the previous call, it must emit ZERO bytes.
//
// The bug this regresses: at the 120ms animation tick the renderer
// used to always emit SeqHideCursor + cursor-position +
// SeqShowCursor, which resets the terminal's blink timer. Faster
// than the OS blink interval, so an idle dialog editor (e.g. a
// re-opened swarm transcript whose agent isn't producing output)
// rendered the caret as a solid non-blinking block.
//
// With the no-op fast path the renderer leaves the screen alone
// on idle frames, letting the terminal run its own blink cycle.
func TestDrawLogIdleNoOpEmitsNothing(t *testing.T) {
var buf bytes.Buffer
r := NewRenderer(&buf)
r.Resize(80, 24)
chat := []string{"hello", "world"}
bottom := []string{"▌ "}
// First draw populates the renderer's cached buffer.
r.DrawLog(chat, bottom, 0, 2)
first := buf.Len()
if first == 0 {
t.Fatal("first DrawLog wrote nothing; setup is broken")
}
buf.Reset()
// Identical second draw: same content, same cursor placement.
r.DrawLog(chat, bottom, 0, 2)
if buf.Len() != 0 {
t.Fatalf("idle re-draw emitted %d bytes; expected 0 so terminal blink keeps ticking\n%q",
buf.Len(), buf.String())
}
}
// TestDrawLogContentChangeBreaksFastPath proves the no-op fast path
// only fires when nothing changed. A buffer mutation must still
// produce output, otherwise streaming agent replies would freeze on
// screen.
func TestDrawLogContentChangeBreaksFastPath(t *testing.T) {
var buf bytes.Buffer
r := NewRenderer(&buf)
r.Resize(80, 24)
r.DrawLog([]string{"hello"}, []string{"▌ "}, 0, 2)
buf.Reset()
// New chat row lands.
r.DrawLog([]string{"hello", "world"}, []string{"▌ "}, 0, 2)
if buf.Len() == 0 {
t.Fatal("content change suppressed by fast path; streaming output would freeze")
}
}
// TestDrawLogCursorMoveBreaksFastPath proves a cursor-only change
// (no buffer change) still produces output. Without this, typing in
// the editor would visually move the caret but the terminal would
// keep drawing it at the old column.
func TestDrawLogCursorMoveBreaksFastPath(t *testing.T) {
var buf bytes.Buffer
r := NewRenderer(&buf)
r.Resize(80, 24)
r.DrawLog([]string{"hi"}, []string{"▌ "}, 0, 2)
buf.Reset()
// Same buffer, different cursor column.
r.DrawLog([]string{"hi"}, []string{"▌ "}, 0, 3)
if buf.Len() == 0 {
t.Fatal("cursor-only change suppressed by fast path; caret would lag behind typing")
}
// And the emitted bytes must at least reposition the cursor.
if !strings.Contains(buf.String(), "\x1b[") {
t.Errorf("cursor move emission missing CSI escapes: %q", buf.String())
}
}
// TestDrawLogResizeForcesFullRedraw confirms a resize invalidates
// the cache so the next DrawLog with identical inputs still emits.
// Resize sets logInit=false; without that, a resize followed by an
// identical buffer would falsely no-op and leave a stale frame.
func TestDrawLogResizeForcesFullRedraw(t *testing.T) {
var buf bytes.Buffer
r := NewRenderer(&buf)
r.Resize(80, 24)
r.DrawLog([]string{"hi"}, []string{"▌ "}, 0, 2)
buf.Reset()
r.Resize(100, 30)
r.DrawLog([]string{"hi"}, []string{"▌ "}, 0, 2)
if buf.Len() == 0 {
t.Fatal("post-resize redraw skipped; the new frame would never reach the terminal")
}
}