Skip to content

Visual Editor: Re-enable and simplify canvas interaction tests - #13467

Merged
NigelBreslaw merged 4 commits into
masterfrom
nigel/enable-canvas-tests
Sep 18, 2026
Merged

NigelBreslaw merged 4 commits into
masterfrom
nigel/enable-canvas-tests

Conversation

@NigelBreslaw

@NigelBreslaw NigelBreslaw commented Sep 17, 2026

Copy link
Copy Markdown
Member

Summary

Many tests were being skipped while the features didn't exist. They now do and this hooks them up.

  • Re-enable canvas movement and corner-radius tests, including artboard boundaries and radius clamping.
  • Replace synthetic pointer-exit cancellation expectations with outside-window movement, resize, rotation, and radius release coverage.
  • Retain live selection-frame and tooltip references during drags so off-screen items can still be observed.
  • Simplify older tests by removing redundant fixture rewrites, sharing boundary calculations and geometry formatting, and trimming unused rotation history.

Enable movement and corner-radius cases and check dragging outside the window through pointer release.

Remove redundant fixture rewrites, parameter construction, and the unused cancellation helper. The headless UI suite passes with 411 passed and 16 skipped.
Descendant queries omit fully clipped items. Keep the live selection frame and radius tooltip references during the interaction so off-screen drags still verify transient changes before release.

Remove the unused single-transient-state option; boundary drags now retain the multiple-state assertion.
Remove unchanged fixture rewrites and unused baseline reads. Share boundary target calculations and geometry formatting, simplify radius parameters, and read only the final frame rotation.

Retain all 134 canvas cases and the live-state, exact-source, and installed-preview assertions. Canvas and undo/redo validation passed with 143 tests and 3 existing skips. The full headless suite had 420 passed, 16 skipped, and the existing unrelated outline-preview failure.
CI read the previous stop color immediately after switching gradient stops. Wait for the color field to show the selected stop before asserting its exact value in both directions.
@NigelBreslaw
NigelBreslaw merged commit 54f3084 into master Sep 18, 2026
47 checks passed
@NigelBreslaw
NigelBreslaw deleted the nigel/enable-canvas-tests branch September 18, 2026 06:57
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.

2 participants