Skip to content

Conversation

daymxn
Copy link
Member

@daymxn daymxn commented Oct 8, 2025

This PR adds Integration tests for the Live API.

The tests currently test the following:

  • Text - Text
  • Text - Audio
  • Audio - Audio
  • Audio - Text
  • Function Calling
  • Function Cancellation
  • Interruption
  • Incremental (just that it works)

Test are written via the swift testing framework (matching our other [newer] integration tests), and are parametrized to run across both backends and different services. I've removed the global variants of the parameters though, as the vertex model only supports us-central1 anyhow.

#no-changelog

@daymxn daymxn self-assigned this Oct 8, 2025
@daymxn daymxn changed the title chore(ai): Add tests chore(ai): Add tests for Live API Oct 8, 2025
Copy link
Contributor

@andrewheard andrewheard left a comment

Choose a reason for hiding this comment

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

I'd recommend splitting off the unit tests into a separate PR. They LGTM.

@daymxn
Copy link
Member Author

daymxn commented Oct 10, 2025

I'd recommend splitting off the unit tests into a separate PR. They LGTM.

@andrewheard Oh I already intended to do this (see the PR description for reference). I just have everything here for my own centralization purposes, until I split them off.

@daymxn daymxn changed the title chore(ai): Add tests for Live API chore(ai): Add integration tests for Live API Oct 10, 2025
@daymxn daymxn marked this pull request as ready for review October 10, 2025 22:46
@daymxn daymxn requested a review from andrewheard October 14, 2025 15:40
Copy link
Member

@ncooke3 ncooke3 left a comment

Choose a reason for hiding this comment

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

LGTM on green CI

@daymxn daymxn requested a review from andrewheard October 14, 2025 16:56
Copy link
Contributor

@andrewheard andrewheard left a comment

Choose a reason for hiding this comment

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

Thanks, @daymxn! You might want to share this with the other SDK teams after we run them for a few days on CI (to verify no flakiness). They could likely re-use your system instructions, audio file and the scenarios for their own integration tests.

@daymxn daymxn merged commit d4d24af into main Oct 14, 2025
51 checks passed
@daymxn daymxn deleted the dlr/add-live-tests branch October 14, 2025 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants