Skip to content

Refactor audio tests to avoid Robolectric network dependency - #6

Merged
ElliotBadinger merged 1 commit into
codex/complete-comprehensive-app-refactoring-8m8gt3from
codex/fix-github-actions-failures
Oct 5, 2025
Merged

Refactor audio tests to avoid Robolectric network dependency#6
ElliotBadinger merged 1 commit into
codex/complete-comprehensive-app-refactoring-8m8gt3from
codex/fix-github-actions-failures

Conversation

@ElliotBadinger

Copy link
Copy Markdown
Owner

Summary

  • inject media recorder/player providers so audio components can be unit-tested without Robolectric
  • rewrite audio unit tests to use MockK-based verifications instead of Robolectric and tighten related imports in SaidIt tests
  • add an explicit android-all runtime dependency for Robolectric-based tests and update supporting test classes

Testing

  • ./gradlew :audio:test

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

@openhands-ai

openhands-ai Bot commented Oct 5, 2025

Copy link
Copy Markdown

Looks like there are a few issues preventing this PR from being merged!

  • GitHub Actions are failing:
    • unit-tests
    • Android CI
    • Android CI
    • Cross-Platform CI

If you'd like me to help, just leave a comment, like

@OpenHands please fix the failing actions on PR #6 at branch `codex/fix-github-actions-failures`

Feel free to include any additional details that might help me get this PR into a better state.

You can manage your notification settings

@ElliotBadinger
ElliotBadinger merged commit dda70ff into codex/complete-comprehensive-app-refactoring-8m8gt3 Oct 5, 2025
12 of 16 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