CI/CD: Enable integration tests on mac#136
Open
stephen-derosa wants to merge 6 commits into
Open
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adjusts the test strategy to make macOS CI run the integration (E2E) suite reliably by moving the high-throughput data-track transport scenario out of integration tests and into the stress test suite (which is not executed in CI).
Changes:
- Added data-track transport stress tests (low/high FPS multi-packet) under the stress test target.
- Removed the parameterized data-track transport E2E test from the integration test suite.
- Enabled E2E/integration testing steps for macOS runners in the CI workflow.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/tests/stress/test_data_track_stress.cpp | Adds high-throughput data-track transport stress coverage outside CI. |
| src/tests/integration/test_data_track.cpp | Removes the transport throughput scenario from integration/E2E tests. |
| .github/workflows/tests.yml | Enables integration (E2E) tests on macOS runners. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
stephen-derosa
force-pushed
the
sderosa/BOT-346-mac-integration-tests
branch
from
94e7440 to
ae4b2de
Compare
xianshijing-lk
approved these changes
May 21, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
-move Low/HighFpsMultiPacket to stress tests (not run in CI)
-enable integration tests on mac