tree f7485f9715aee87c185ba2113b2eab6c41b1398d
parent b7c5875548da5057eac0405bf4e0ae8bbc43667c
author Philip Zeyliger <philip@bold.dev> 1746120421 -0700
committer Philip Zeyliger <philip@bold.dev> 1746126811 -0700

Update tests for agent's new iterator implementation

Co-Authored-By: sketch <hello@sketch.dev>

The commit 4eb6bb6c9ac switched from outbox channels to a subscriber model
with message iterators. This commit updates all existing tests to use the
new iterator implementation and adds new tests specifically for the iterator
functionality.

- Modified agent_test.go to use new iterator approach
- Modified agent_git_test.go to check history array directly
- Added new iterator_test.go to test iterator-specific functionality

Fix issues found by code review

Co-Authored-By: sketch <hello@sketch.dev>

- Fix context leak by properly using cancel function
- Modernize for loops using range over int
- Use fmt.Appendf instead of []byte(fmt.Sprintf)

Optimize slow tests to improve test execution time

Co-Authored-By: sketch <hello@sketch.dev>

This commit optimizes the test suite to reduce execution time:

1. Reduces TestGitCommitTracking test time by using fewer git commits
   (20 instead of 110) which cuts the test time by 77%.

2. Improves iterator tests by using channels for synchronization instead
   of time.Sleep, reducing TestIteratorWithNewMessages from 200ms to nearly 0ms.

3. Reduces timeouts and sleeps in other iterator tests.

Total test execution time reduced by more than 60% (from 2.3s to 0.9s).