Skip to content

Strengthen SaidIt service tests with Robolectric coverage - #9

Merged
ElliotBadinger merged 1 commit into
masterfrom
codex/refactor-and-migrate-saidit-test-suite
Oct 7, 2025
Merged

Strengthen SaidIt service tests with Robolectric coverage#9
ElliotBadinger merged 1 commit into
masterfrom
codex/refactor-and-migrate-saidit-test-suite

Conversation

@ElliotBadinger

Copy link
Copy Markdown
Owner

Summary

  • add an AudioSampleWriter abstraction and update SaidItService to support injectable writers, coroutine dispatchers, and audio memory for testability
  • expose AudioMemory for extension in tests and allow overriding the service dispatcher to drive coroutine callbacks deterministically
  • rewrite SaidItService Robolectric tests in Kotlin using PCM helpers to verify audio persistence and error handling, remove Java tests, and migrate legacy Kotlin tests to src/test/kotlin

Testing

  • ./gradlew --console=plain :SaidIt:test (fails: Robolectric cannot download dependencies in offline environment)
  • bash scripts/agent/healthcheck.sh --tier 0-3

https://chatgpt.com/codex/tasks/task_b_68e5255151048323a86ca27881dd16d1

@claude

claude Bot commented Oct 7, 2025

Copy link
Copy Markdown

Claude encountered an error —— View job


I'll analyze this and get back to you.

@ElliotBadinger

Copy link
Copy Markdown
Owner Author

@codex fix failing tests

@ElliotBadinger

Copy link
Copy Markdown
Owner Author

@codex fix failing tests on pr #9

@ElliotBadinger
ElliotBadinger merged commit 22ed05c into master Oct 7, 2025
24 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant