thread/hyprvoice
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

mark task 40 complete: streaming adapter tests verified

Browse Source
This commit is contained in:
leonardotrapani
2026-02-01 02:02:19 +01:00
parent 1865de74ef
commit 4b6e6d87ed
2 changed files with 18 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
progress.txt
+17
View File
@@ -424,4 +424,21 @@ Started: Sun Feb 1 12:22:47 AM CET 2026
- Maps to providerName="deepgram" and envVar="DEEPGRAM_API_KEY"
- Updated config template in save.go with commented deepgram section
- Added "deepgram" to AllProviders and providerDisplayNames in configure.go for TUI
- All tests passing, typecheck passes
### Task 40: Write tests for streaming adapters
- Tests already existed in comprehensive form across multiple files (implemented with tasks 31-39)
- Verified test coverage in:
- `adapter_elevenlabs_streaming_test.go` (744 lines): Start, SendChunk, Results, Error, Language, Close, Reconnect logic
- `adapter_deepgram_test.go` (435 lines): Creation, URL building, Results, Binary audio, Errors, Context
- `adapter_openai_realtime_test.go` (545 lines): Start, SendChunk, Transcription, Errors, Reconnect, Close, resample
- `transcriber_test.go` (StreamingTranscriber tests): Accumulation, Errors, Context cancellation, Concurrent access
- Tests verify:
- StreamingTranscriber accumulates final results (TestStreamingTranscriber_AccumulatesResults)
- Error handling (TestStreamingTranscriber_HandlesErrors, adapter error tests)
- Context cancellation (TestStreamingTranscriber_ContextCancellation, TestDeepgramAdapter_ContextCancellation)
- Concurrent GetFinalTranscription safety (TestStreamingTranscriber_GetFinalTranscriptionSafe)
- Reconnection logic with exponential backoff (multiple reconnect tests)
- Close cleanup (TestElevenLabsStreamingAdapter_Close, TestOpenAIRealtimeAdapter_Close)
- `go test -race ./internal/transcriber/...` passes with no race conditions
- All tests passing, typecheck passes