Skip to content

fix: resolve Android bridge test failures#1242

Closed
rmi22186 wants to merge 1 commit intomasterfrom
fix/android-bridge-test-timeout
Closed

fix: resolve Android bridge test failures#1242
rmi22186 wants to merge 1 commit intomasterfrom
fix/android-bridge-test-timeout

Conversation

@rmi22186
Copy link
Copy Markdown
Member

@rmi22186 rmi22186 commented Apr 8, 2026

Summary

  • Points the Android bridge tests workflow at the fix/increase-mplatch-timeout branch in mparticle-android-sdk, which increases the MPLatch timeout from 5s to 30s
  • The Android bridge tests have been consistently failing since March 12, 2026 due to a GitHub Actions runner image update (ubuntu24/20260309.50) that slowed WebView/JS initialization beyond the 5-second timeout
  • No code changes in either the web SDK or Android SDK caused the regression — it was purely environmental

Root cause investigation

We verified the following:

  • Same web SDK code (identical mparticle.js) passes on runner image 20260302.42, fails on 20260309.50
  • Same Android SDK commit in both passing and failing runs
  • Downgrading platform-tools from 37.0.0 to 36.0.2 does not fix it
  • Bumping emulator API level from 29 to 34 reduces failures but doesn't eliminate them
  • Increasing the MPLatch timeout from 5s to 30s fixes it (verified: https://github.com/mParticle/mparticle-web-sdk/actions/runs/24104711064)

Dependencies

  • Depends on mparticle-android-sdk#695
  • Once that PR is merged, the ref in this workflow should be switched back from fix/increase-mplatch-timeout to main

Test plan

  • Verified Android bridge tests pass with the timeout fix (run)
  • Merge mparticle-android-sdk#695 first
  • Update ref back to main before merging this PR

🤖 Generated with Claude Code

The Android bridge tests have been failing since March 12, 2026 due
to a GitHub Actions runner image update (ubuntu24/20260309.50) that
slowed WebView/JS initialization beyond the 5-second MPLatch timeout.

Points the android-sdk ref at fix/increase-mplatch-timeout which
increases the timeout from 5s to 30s. Should be switched back to
main once mParticle/mparticle-android-sdk#695 is merged.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@rmi22186 rmi22186 requested a review from a team as a code owner April 8, 2026 12:33
@cursor
Copy link
Copy Markdown

cursor bot commented Apr 8, 2026

PR Summary

Low Risk
Low risk because it only changes CI configuration, but it temporarily pins tests to a non-main Android SDK ref which could mask upstream changes until reverted.

Overview
Android bridge CI now checks out mparticle-android-sdk from fix/increase-mplatch-timeout instead of main, so instrumented bridge tests run with the increased latch timeout in that branch.

No product/runtime code changes are introduced in this repo; this is purely a workflow ref update to stabilize failing tests.

Reviewed by Cursor Bugbot for commit c49f1a5. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit c49f1a5. Configure here.

with:
repository: mParticle/mparticle-android-sdk
ref: main
ref: fix/increase-mplatch-timeout
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Workflow points at temporary branch that will be deleted

High Severity

The ref is set to fix/increase-mplatch-timeout, a temporary feature branch in mparticle-android-sdk. The PR's own test plan has an unchecked item acknowledging this needs to be changed back to main before merge. Once mparticle-android-sdk#695 is merged and the branch is deleted, every push to this repo will fail the Android Bridge Tests workflow because the referenced branch no longer exists.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit c49f1a5. Configure here.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 8, 2026

@rmi22186 rmi22186 closed this Apr 8, 2026
@rmi22186
Copy link
Copy Markdown
Member Author

rmi22186 commented Apr 8, 2026

This PR was just a test to confirm if the proposed fix in mParticle/mparticle-android-sdk#695 works or not. and it appears to, so I am closing this PR

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.

1 participant