Skip to content

Run the AnkiQuest home screenshots at the default SDK - #26

Merged
float3 merged 3 commits into
ankiquestfrom
ci/retry-screenshot-record
Sep 24, 2026
Merged

float3 merged 3 commits into
ankiquestfrom
ci/retry-screenshot-record

Conversation

@float3

@float3 float3 commented Sep 24, 2026 •

Copy link
Copy Markdown
Owner

Store Baseline has failed intermittently since 22 Sep: UnsatisfiedLinkError in NativeMethods.openBackend, or Extracted file was not found: /tmp/librsdroid-….so, on a different test each time. Upstream: 15/15 green.

  • AnkiquestHomeScreenshotTest pinned @Config(sdk = [33]); every other screenshot test runs at the default targetSdk 36. Two Robolectric sandboxes in one JVM both load the Rust backend, and a JNI library binds to one classloader per JVM, so tests in the losing sandbox fail. The first failure was on the branch that added this test, the same day.
  • Removes the pin. These screenshots now render at SDK 36; the next baseline records them.
  • Reverts the retry this PR first added, which would have hidden the cause.

Store Baseline failed on ankiquest, merge/upstream-main and
share-client-logic, each time on a different test, always because a test
JVM could not load librsdroid: its extracted /tmp copy had been deleted by
a retiring fork. Retry the step like the warm-up step already is.
AnkiquestHomeScreenshotTest pinned sdk 33 while every other screenshot
test runs at targetSdk 36, so screenshot JVMs held two Robolectric
sandboxes. The Rust backend binds to one classloader per JVM; whichever
sandbox loaded it first won, and tests in the other failed with
UnsatisfiedLinkError or a deleted /tmp/librsdroid copy. Store Baseline has
failed only since this test was added.

The retry is reverted: it would hide the cause.
@float3 float3 changed the title ci: retry the flaky screenshot recording Run the AnkiQuest home screenshots at the default SDK Sep 24, 2026
@github-actions

Copy link
Copy Markdown

Snapshot diff report vs ankiquest. Open screenshot-diff for diffs.

  • AnkiquestHomeScreenshotTest: 2 changes
All 2 changed screenshots

AnkiquestHomeScreenshotTest

  • today_320dp_200percent_compare.png
  • today_320dp_compare.png

@float3
float3 merged commit 1bb9ec7 into ankiquest Sep 24, 2026
28 checks passed
@float3
float3 deleted the ci/retry-screenshot-record branch September 24, 2026 09:41
@float3 float3 mentioned this pull request Sep 24, 2026
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