Skip to content

test(term): remove TTY-dependent assertions - #117

Merged
tylerbutler merged 1 commit into
mainfrom
fix/deterministic-term-tests
Sep 1, 2026
Merged

test(term): remove TTY-dependent assertions#117
tylerbutler merged 1 commit into
mainfrom
fix/deterministic-term-tests

Conversation

@tylerbutler

@tylerbutler tylerbutler commented Sep 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • pass terminal state explicitly when testing automatic color detection
  • test padded ANSI rendering through the pure paint helper
  • remove assertions that depend on whether Cargo inherits a terminal

Cargo can leave test stdout attached to a terminal, so test terminal detection with an explicit input and keep rendering assertions pure.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@netlify

netlify Bot commented Sep 1, 2026

Copy link
Copy Markdown

Deploy Preview for trellis-gleam canceled.

Name Link
🔨 Latest commit 045cd53
🔍 Latest deploy log https://app.netlify.com/projects/trellis-gleam/deploys/6a9748899a01c500084cb43d

@tylerbutler
tylerbutler merged commit 3871d3c into main Sep 1, 2026
12 checks passed
@tylerbutler
tylerbutler deleted the fix/deterministic-term-tests branch September 1, 2026 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant