Skip to content

Test: animation.spec.ts integration suite skipped (needs full pipeline rework) #79

@jjeff

Description

@jjeff

Problem

`tests/e2e/animation.spec.ts:5` is a whole-file `test.describe.skip`:

TODO: These tests require full drag-and-drop implementation to work properly. They are currently skipped until the drag system is complete.

The suite has 8 tests covering FLIP animations, animation duration, easing, cross-list animation, etc. Many of them call `firstItem.dragTo(...)` (high-level Playwright helper, atomic) and expect intermediate animation states to be observable.

Suggested approach

  • Replace `dragTo` with the low-level `page.mouse` pattern used by `on-move.spec.ts` and the fallback-mode suites — gives us time-window observability mid-drag.
  • Some tests measure `boundingBox` of items mid-animation; those need to either wait for animations to finish (using the `dragAndDropWithAnimation` helper that exists), or use Playwright's clock API to advance through animation durations deterministically.

This is partly tied to #73 (atomic dragAndDrop swallows intermediate events) — same root cause, different observability concern.

Refs #35 — discovered during E2E skip triage.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingtestTest coverage / quality

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions