Skip to content

test: cover stt transcription and redis helpers#72

Merged
derogab merged 1 commit into
masterfrom
test/coverage
Jun 6, 2026
Merged

test: cover stt transcription and redis helpers#72
derogab merged 1 commit into
masterfrom
test/coverage

Conversation

@derogab

@derogab derogab commented Jun 6, 2026

Copy link
Copy Markdown
Owner

Summary

Adds coverage for STT transcription paths and Redis helper behavior so regressions in message handling and storage parsing are easier to catch.

Changes

  • Cover Cloudflare audio transcription, whisper.cpp voice transcription, and the no-STT fallback in message handling tests.
  • Mock STT, file lookup, fetch, and filesystem checks needed for isolated transcription tests.
  • Add Redis storage coverage for bounded reconnect retries, stored history parsing, and active chat ID extraction.

Test plan

  • npm test

Summary by cubic

Add tests for STT transcription flows (Cloudflare, whisper.cpp, and no-STT fallback) and Redis helpers to prevent regressions in message handling and storage parsing. Tests mock fetch, filesystem, and STT to isolate behavior; verify audio/voice file retrieval and transcription updates; and check bounded Redis reconnect retries, stored history parsing, and active chat ID extraction.

Written for commit d52a6a2. Summary will update on new commits.

Review in cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

Re-trigger cubic

@derogab derogab merged commit 55c389f into master Jun 6, 2026
5 checks passed
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