Skip to content

[WAL-752] test: stabilize Android Digital Credentials E2Es - #2086

Merged
JakeFernandes98 merged 10 commits into
mainfrom
feature/wal-752-dc-api-harness-main
Aug 17, 2026
Merged

[WAL-752] test: stabilize Android Digital Credentials E2Es#2086
JakeFernandes98 merged 10 commits into
mainfrom
feature/wal-752-dc-api-harness-main

Conversation

@szijpeter

@szijpeter szijpeter commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Summary

Stabilize the Android DC API sharing E2Es without weakening their end-to-end coverage, and make the dedicated API 37 Play Store test environment reproducible.

What changed

  • Live-issue the canonical mDL and SD-JWT once per suite through MobileWallet, then reuse and verify that shared state across presentation tests.
  • Replace the global verifier handoff with request-scoped handles and lifecycle-bound platform calls, with explicit cleanup for completion, cancellation, back/re-entry, and Activity destruction.
  • Make picker interaction state-driven and fail with focused diagnostics instead of relying on stale UI state or cross-test cleanup.
  • Add a real Credential Manager/GMS platform smoke gate and require instrumentation results, so a setup failure cannot be reported as a passing zero-test lane.
  • Cold-boot and cache a validated API 37 / Pixel 7 / Play Store AVD. The lane fingerprints the emulator, platform-tools, system image, AVD config, device build, page size, GMS version/code, and preparation scripts, and rejects stale or mismatched cache state.
  • Exclude the class-level GMS-gated sharing suite from the default API 34 compose-demo lane and run it in the dedicated Play Store lane.
  • Run the now-enabled SCA + EU age presentation case in the stabilized DC API lane.

Caveats and follow-ups

  • The dedicated lane remains tied to the current API 37 Play Store / x86_64 emulator substrate and its pinned package revisions; a platform upgrade should change the cache identity and baseline together.

Breaking

None.

@linear-code

linear-code Bot commented Aug 13, 2026

Copy link
Copy Markdown

WAL-752

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 786b6686-3da0-4ba7-a5b5-e79084582657

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@szijpeter szijpeter added the ci:android Run Android related CI workflows for this PR label Aug 13, 2026
@szijpeter szijpeter self-assigned this Aug 13, 2026
@szijpeter
szijpeter force-pushed the feature/wal-752-dc-api-harness-main branch from 53738df to 9f905f5 Compare August 13, 2026 08:18
@szijpeter szijpeter added ci:android Run Android related CI workflows for this PR and removed ci:android Run Android related CI workflows for this PR labels Aug 13, 2026
@szijpeter

Copy link
Copy Markdown
Contributor Author

Reopening immediately to trigger restored CI on the cleaned main-based SHA; the PR remains draft.

@szijpeter szijpeter closed this Aug 13, 2026
@szijpeter szijpeter reopened this Aug 13, 2026
@szijpeter
szijpeter force-pushed the feature/wal-752-dc-api-harness-main branch 8 times, most recently from 01a1a9a to 6386062 Compare August 13, 2026 22:16
@szijpeter szijpeter changed the title [WAL-752] test: share live-issued credentials across DC API E2Es [WAL-752] test: stabilize Android Digital Credentials E2Es Aug 13, 2026
@szijpeter
szijpeter marked this pull request as ready for review August 13, 2026 22:17
@szijpeter
szijpeter force-pushed the feature/wal-752-dc-api-harness-main branch from 6386062 to ca1ad86 Compare August 14, 2026 07:05
@szijpeter
szijpeter force-pushed the feature/wal-752-dc-api-harness-main branch from ca1ad86 to 4f8b652 Compare August 14, 2026 07:31
@szijpeter
szijpeter force-pushed the feature/wal-752-dc-api-harness-main branch 2 times, most recently from 2df6292 to e921a26 Compare August 14, 2026 10:51
@szijpeter
szijpeter changed the base branch from main to fix/wal-1294-public-demo-contract August 15, 2026 00:21
@szijpeter
szijpeter force-pushed the feature/wal-752-dc-api-harness-main branch from e921a26 to 3baca7b Compare August 15, 2026 00:21
@sonarqubecloud

Copy link
Copy Markdown

Base automatically changed from fix/wal-1294-public-demo-contract to main August 17, 2026 08:11
@JakeFernandes98
JakeFernandes98 merged commit 3143962 into main Aug 17, 2026
18 of 20 checks passed
@JakeFernandes98
JakeFernandes98 deleted the feature/wal-752-dc-api-harness-main branch August 17, 2026 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:android Run Android related CI workflows for this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants