Skip to content

Integration tests in CI#113

Merged
alan-george-lk merged 46 commits intomainfrom
feature/ci_integration_tests
May 7, 2026
Merged

Integration tests in CI#113
alan-george-lk merged 46 commits intomainfrom
feature/ci_integration_tests

Conversation

@alan-george-lk
Copy link
Copy Markdown
Collaborator

@alan-george-lk alan-george-lk commented Apr 29, 2026

This PR does the following:

  • Moves all testing into a new tests.yml CI stage
  • New more specific/performance improved runners for Windows/Mac
  • Adds the ability to run integration tests in CI via a locally-installed LiveKit server (Linux only)
  • Adjusts unstable integration tests to improve stability
    • PublishesAndReceivesFramesEndToEnd -> Adjusted to more closely match the Rust SDK layout
    • Timestamp test -> Matches the E2E TypeScript test by re-using the same timestamp throughout
  • A more explicit parallel jobs function for improving CI times (has helped slightly on Windows, for example)
  • Consolidates 5 different prior env vars for integration/stress tests down to LIVEKIT_TOKEN_A/B
  • Changes Rust cache action to match Rust SDK
  • Add timeouts to testing stages to catch deadlock bugs
  • Disables GMock to save time on build (not used)

Some caveats:

  • Still runs unit tests on all prior platforms
  • Mac runner performance is not passing on C++ integration tests through FFI. Internal ticket was created to further debug this

@alan-george-lk alan-george-lk changed the title Draft - Integration tests in CI Integration tests in CI May 4, 2026
Copy link
Copy Markdown
Collaborator

@stephen-derosa stephen-derosa left a comment

Choose a reason for hiding this comment

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

sweet looks good to me thanks for doing! Consider waiting on this until the clang format stuff is in to make a smaller diff in that MR

Copy link
Copy Markdown
Collaborator

@stephen-derosa stephen-derosa left a comment

Choose a reason for hiding this comment

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

LGTM -- great work, really excited to have this in!

Comment thread src/tests/CMakeLists.txt
Comment thread AGENTS.md
Comment thread build.sh
Comment thread README.md
@alan-george-lk alan-george-lk merged commit 0aa60a3 into main May 7, 2026
21 checks passed
@alan-george-lk alan-george-lk deleted the feature/ci_integration_tests branch May 7, 2026 16:55
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