Skip to content

test(webui): fix cross-platform path and hover expectations - #37

Merged
Fectivnfy112357 merged 1 commit into
mainfrom
fix/cross-platform-test-expectations
Nov 11, 2026
Merged

Fectivnfy112357 merged 1 commit into
mainfrom
fix/cross-platform-test-expectations

Conversation

@Fectivnfy112357

Copy link
Copy Markdown
Contributor

Change

Fix two cross-platform test failures introduced by #36 without changing product behavior.

  • Compare the external file reference's workspace directory with toPosixPath(artifacts), matching the preview state's normalized path contract on Windows.
  • Read the Settings and Usage menu centers once, then move the mouse directly during the repeated hover loop. Locator stability waits must not turn a rapid hover scenario into a legitimate 180ms fly-out close and subsequent quota refresh.

The quota request count, subscription-label stability, and close/reopen refresh assertions remain unchanged. No screenshot baselines or first-frame timeout budgets are changed.

Validation

  • External absolute reference integration case: 1 passed on Linux; exercises real file reading, authenticated image bytes, Range requests, and missing-token rejection.
  • Subscription hover case: light and dark themes, each repeated three times; 6 passed on Linux.
  • pnpm build:webui and git diff --check: passed.
  • Performance: basic; test-only change.
  • NOT RUN: Windows/macOS execution and full repository verification. Platform CI must confirm the affected runner behavior.

Publication and contribution checks

  • Changes use existing applicable licenses; no imported material.
  • No credentials, account data, real user content, or private review material.
  • No new files or test suites; no source inventory regeneration needed.
  • No product behavior or documentation contract changes; fixture evidence is not live-service acceptance.

Maintainer handoff

Publication scope and license changes: none. Shared-source port: not needed; WebUI tests only.

@Fectivnfy112357 Fectivnfy112357 added the bug Something isn't working label Nov 11, 2026
@Fectivnfy112357
Fectivnfy112357 merged commit 18c5e21 into main Nov 11, 2026
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant