Skip to content

UI test fixup#4206

Merged
jonalmeida merged 1 commit intomasterfrom
ui-test-fixup
Mar 23, 2026
Merged

UI test fixup#4206
jonalmeida merged 1 commit intomasterfrom
ui-test-fixup

Conversation

@rvandermeulen
Copy link
Copy Markdown
Contributor

@rvandermeulen rvandermeulen commented Mar 23, 2026 •
edited
Loading

Address these failures:

java.lang.RuntimeException: Intent in process org.mozilla.reference.browser.debug resolved to different process com.android.chrome: Intent { act=android.intent.action.VIEW dat=http://localhost:59959/... flg=0x10008000 (has extras) }
	at android.app.Instrumentation.startActivitySync(Instrumentation.java:543)
	at androidx.test.core.app.InstrumentationActivityInvoker.startActivity(InstrumentationActivityInvoker.java:422)
	at androidx.test.core.app.InstrumentationActivityInvoker.startActivity(InstrumentationActivityInvoker.java:428)
	at androidx.test.core.app.ActivityScenario.launchInternal(ActivityScenario.java:367)
	at androidx.test.core.app.ActivityScenario.launch(ActivityScenario.java:237)
	at org.mozilla.reference.browser.ui.CustomTabsTest.customTabNavigationTest(CustomTabsTest.kt:91)

@rvandermeulen rvandermeulen requested a review from AaronMT March 23, 2026 20:19
@rvandermeulen rvandermeulen requested a review from a team as a code owner March 23, 2026 20:25
@rvandermeulen rvandermeulen requested review from bhearsum and removed request for a team March 23, 2026 20:25
@rvandermeulen rvandermeulen marked this pull request as draft March 23, 2026 20:27
@rvandermeulen rvandermeulen removed the request for review from bhearsum March 23, 2026 20:27
@rvandermeulen rvandermeulen added the needs landing Auto lands approved and green PRs. label Mar 23, 2026
@rvandermeulen
Fix CustomTabsTest intent resolution with ActivityScenario
a9f7903 
ActivityScenario.launch resolves intents through normal Android routing,
so an ACTION_VIEW intent with an HTTP URL resolved to Chrome rather than
IntentReceiverActivity. Set the component explicitly before launching to
force the activity within the test process.

Improve error message when BrowserActivityTestRule.activity is unavailable
@rvandermeulen rvandermeulen marked this pull request as ready for review March 23, 2026 20:54
@mergify mergify bot added the queued label Mar 23, 2026
@jonalmeida jonalmeida merged commit dfa2d87 into master Mar 23, 2026
14 checks passed
@jonalmeida jonalmeida deleted the ui-test-fixup branch March 23, 2026 21:04
@mergify
Copy link
Copy Markdown

mergify bot commented Mar 23, 2026

Merge Queue Status

  • ✅ Entered queue — 2026-03-23 21:04 UTC · Rule: default
  • ✅ Checks skipped · PR is already up-to-date
  • ✅ Merged — 2026-03-23 21:04 UTC · at a9f790340375e48caf718b03140635b2aad41f7b

This pull request spent 9 seconds in the queue, with no time running CI.

Required conditions to merge
  • status-success=pr-complete

@mergify mergify bot removed the queued label Mar 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AaronMT AaronMT AaronMT approved these changes

Assignees

No one assigned

Labels

needs landing Auto lands approved and green PRs.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@rvandermeulen @AaronMT @jonalmeida