Skip to content

test: add displacement and profile tests#2572

Merged
yoreley777 merged 12 commits into
testfrom
Yoreley-Displacement
May 20, 2026
Merged

test: add displacement and profile tests#2572
yoreley777 merged 12 commits into
testfrom
Yoreley-Displacement

Conversation

@yoreley777
Copy link
Copy Markdown
Collaborator

Summary

Adds Playwright coverage for:

  • Displacement
    • velocity layer
    • flight direction toggle
    • draw point
    • highlight series
    • delete series
    • delete all series
    • no points selected
    • select multiple points
  • Profile
    • dark mode
    • login state sync across instances

Related Issues

N/A

Screenshots / UI Changes

N/A

i18n

N/A

Tests & Build

  • npm run lint passes
  • npm test passes (or N/A)
  • npm run build succeeds

Notes

Some larger Ghost Inspector tests were split into smaller, focused specs:

  • Layer Cumulative Velocity Asc & Desc was covered as velocity-layer.spec.ts and flight-direction-toggle.spec.ts
  • AOI Interaction Test: Draw, Highlight & Delete was covered as draw-point.spec.ts, highlight-series.spec.ts, delete-series.spec.ts, and delete-all-series.spec.ts

login-state-sync.spec.ts uses two pages in the same browser context and a mocked auth cookie response to verify BroadcastChannel-based sync behavior.

Possible follow-up: add coverage for logout state syncing across instances.

@yoreley777 yoreley777 requested a review from a team as a code owner May 15, 2026 20:52
Comment thread e2e/profile/login-state-sync.spec.ts Fixed
Comment thread e2e/profile/login-state-sync.spec.ts Outdated
@yoreley777 yoreley777 requested a review from tylercchase May 19, 2026 23:42
Comment thread e2e/profile/login-state-sync.spec.ts Outdated
@yoreley777 yoreley777 requested a review from tylercchase May 20, 2026 19:32
@yoreley777 yoreley777 merged commit 4266fc5 into test May 20, 2026
71 of 76 checks passed
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.

4 participants