test: enabling network api calls from browserstack#25655
test: enabling network api calls from browserstack#25655javiergarciavera merged 9 commits intomainfrom
Conversation
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection:
These changes do NOT affect:
The changes are purely infrastructure improvements for performance test reporting - adding network log capture capabilities. No Detox E2E tests are needed as the application behavior is completely unchanged. Performance Test Selection: |
| entries: [], | ||
| error: 'No build ID found in session details', | ||
| }; | ||
| } |
There was a problem hiding this comment.
Redundant BrowserStack API calls for session details
Low Severity
The new getNetworkLogs method makes its own getSessionDetails API call internally, while the same session details are already being fetched by fetchCompleteProfilingData earlier in the same loop iteration. For each session, getSessionDetails is now called 2-3 times instead of once, making redundant HTTP requests to BrowserStack's API. This adds unnecessary latency to test reporting and could trigger rate limiting with many sessions.
Additional Locations (1)
|
Description
Adds tracking of BrowserStack network (API) calls into the performance E2E pipeline. The custom reporter fetches network logs per session and attaches them to each test metric as apiCalls (and apiCallsError when the fetch fails). The aggregation script is updated so these fields are kept in the cleaned and aggregated reports. Removes the unused getDeviceInfoFromProfiling method and fixes the missing comma in the aggregation script after the new apiCalls / apiCallsError properties.
Changelog
CHANGELOG entry:
Related issues
Fixes: https://consensyssoftware.atlassian.net/browse/MMQA-1361
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Medium Risk
Touches the test infrastructure and BrowserStack/Appium capability set, which can change test stability, runtime, and artifact sizes. Risk is limited to CI/e2e tooling but includes new API calls and additional logged data.
Overview
Adds BrowserStack network log (HAR) collection to the performance-reporting pipeline:
AppProfilingDataHandlernow fetches/networklogs,custom-reporterretrieves and stores per-session entries/errors, and emittedperformance-metrics-*.jsonplus the aggregation script now includeapiCalls/apiCallsErrorfields.Updates the patched
appwrightBrowserStack capabilities to enable richer telemetry (network profile/log content capture, app profiling, self-heal, various Appium settings), adds env-driven overrides (BROWSERSTACK_BUILD_NAME,DISABLE_VIDEO_DOWNLOAD), and adjusts runner behavior (CI retries/workers, locator text-matching behavior/logging) via the Yarn patch.Written by Cursor Bugbot for commit 44ee531. This will update automatically on new commits. Configure here.