Objective

Add comprehensive unit and integration tests for the new idiomatic spinner implementation to ensure correctness and prevent regressions.

Context

The new spinner implementation needs thorough testing to validate:

• Bubble Tea model behavior (if using tea.Model)
• TTY detection logic
• Accessibility fallback
• Start/stop semantics
• Message updates

Approach

1. Create test file: pkg/console/spinner_v2_test.go
2. Test Bubble Tea model methods (Init/Update/View) if applicable
3. Test TTY detection scenarios
4. Test accessibility mode
5. Test concurrent start/stop calls
6. Test message updates during animation

Files to Create

• pkg/console/spinner_v2_test.go

Test Scenarios

Unit Tests

• Test spinner initialization
• Test message updates
• Test View() output format
• Test Update() with Tick messages (if using tea.Model)
• Test stop during animation

Integration Tests

• Test TTY detection (terminal vs pipe)
• Test accessibility mode (ACCESSIBLE=1)
• Test start -> message update -> stop flow
• Test multiple start/stop cycles

Edge Cases

• Test stop before start
• Test double start
• Test rapid start/stop cycles

Acceptance Criteria

• Test file created with >80% coverage of new code
• All test scenarios pass
• Tests use table-driven test pattern where appropriate
• TTY and accessibility scenarios covered
• No race conditions detected with go test -race
  Related to [plan] Refactor spinner to use idiomatic Bubble Tea pattern #9189

AI generated by Plan Command for discussion #9188