Skip to content

Conversation

@CFKevinRef
Copy link
Contributor

This reverts commit 3ee9488 with the in-progress integration test server. Original PR text follows.

For testing I used a Local Server, which is documented here. The purpose of this server is to control different network conditions so that we can test that the events are created successfully. I had to modify Integration Test Host capabilities to allow incoming and outgoing connections for App Sandbox to run tests in Mac Catalyst (and possibly real devices too)

I also added some random fixes in the Integration Tests suite to try to fix some issues that were causing random fails and reorganized some functions to reuse across tests. Main function being waitForPlaybackToStart, which we should add to most existing tests.

Added a new test plan, LocalServerTests, to run specifically the new suite of tests that requires the new local server. That test plan holds two environment variables TEST_HOST and TEST_PORT, defaulted to localhost:8080, but open to modification. The other test plans exclude this new test suite, however this is apparently ignored by SauceLabs, and also Sauce Labs doesn´t allow env variables in real devices, hence why I added RUN_SERVER_TESTS_FLAG

…s events and connected that logic to Player Monitor using publishers
- Switched to isCalculatingBandwidthMetrics
@CFKevinRef CFKevinRef requested a review from a team as a code owner July 28, 2025 15:59
@CFKevinRef CFKevinRef marked this pull request as draft July 28, 2025 15:59
@snyk-io
Copy link

snyk-io bot commented Jul 28, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

@CFKevinRef CFKevinRef force-pushed the sp/use_avmetrics_for_bandwidtMetrics_requests branch 2 times, most recently from 80abc26 to af9c5ee Compare July 29, 2025 17:43
Base automatically changed from sp/use_avmetrics_for_bandwidtMetrics_requests to master July 30, 2025 22:34
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.

4 participants