Skip to content

chore(tests): update tests to use Inference whenever possible AGT-2304#5632

Merged
chenghao-mou merged 2 commits into
mainfrom
chenghao/chore/update-tests-with-inference
May 29, 2026
Merged

chore(tests): update tests to use Inference whenever possible AGT-2304#5632
chenghao-mou merged 2 commits into
mainfrom
chenghao/chore/update-tests-with-inference

Conversation

@chenghao-mou
Copy link
Copy Markdown
Member

Seems like only one spot is missed. Other plugin-focused tests are unchanged.

Seems like only one spot is missed. Other plugin-focused tests were unchanged.
@chenghao-mou chenghao-mou requested a review from a team May 2, 2026 13:00
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

Comment thread tests/test_audio_decoder.py Outdated
@pytest.mark.skipif(
os.getenv("DEEPGRAM_API_KEY") is None,
reason="DEEPGRAM_API_KEY not set",
os.getenv("LIVEKIT_API_KEY") is None or os.getenv("LIVEKIT_API_SECRET") is None,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

we also need to check is_cloud

@chenghao-mou chenghao-mou merged commit c64f75a into main May 29, 2026
25 checks passed
@chenghao-mou chenghao-mou deleted the chenghao/chore/update-tests-with-inference branch May 29, 2026 17:07
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.

3 participants