Skip to content

Cover large mobile restore confirmations - #19

Merged
vrtmrz merged 1 commit into
mainfrom
adopt-test-session-layout-assertions
Jul 15, 2026
Merged

Cover large mobile restore confirmations#19
vrtmrz merged 1 commit into
mainfrom
adopt-test-session-layout-assertions

Conversation

@vrtmrz

@vrtmrz vrtmrz commented Jul 15, 2026

Copy link
Copy Markdown
Owner

Summary

  • update @vrtmrz/obsidian-test-session from 0.1.0 to 0.2.0;
  • add an opt-in large restore plan with 60 restore entries across three ZIPs and 24 local mirror deletion candidates;
  • exercise normal restore and delete-missing confirmation paths at a phone viewport with expanded file lists; and
  • use the shared safe-area, horizontal-overflow, and minimum touch-target assertions for the confirmation dialogue.

Impact

This changes development tooling and the local real-Obsidian E2E suite only. It does not change production behaviour or the distributed plug-in bundle.

The large fixture is selected only by the restore-confirmation scenario. Other scenarios retain the original single-file plan. Both confirmation paths deliberately cancel before mutation; post-confirmation deletion semantics are outside this pull request.

Verification

  • npm run test:ui — 11 tests passed;
  • npm run lint passed;
  • npm run prettyCheck passed;
  • npm run check:e2e:obsidian passed;
  • npm run test:e2e:obsidian:restore-confirmation passed with Obsidian 1.12.7 on Linux arm64, including expanded normal-restore and delete-missing lists, safe-area checks, horizontal-overflow checks, and 44 CSS-pixel Close, Restore, and Cancel targets;
  • npm run test:e2e:obsidian:wake-lock passed, including backup and ZIP restore round-trip; and
  • git diff --check passed.

@vrtmrz
vrtmrz marked this pull request as ready for review July 15, 2026 03:41
@vrtmrz
vrtmrz merged commit edb8f34 into main Jul 15, 2026
1 check passed
@vrtmrz
vrtmrz deleted the adopt-test-session-layout-assertions branch July 15, 2026 03:43
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