test(desktop): cover workspace panel persistence #197

Merged
admin merged 1 commits from test/workspace-panel-persistence into dev 2026-10-14 10:27:32 +00:00
Owner

Summary

  • Add Electron E2E coverage for panel open state and saved widths across reload and process relaunch.
  • Verify topology drag-close/reopen keeps pointer control, keyboard toggling works, and History reopens via a double-click on its collapsed handle.
  • Distinguish pointer taps from actual drag movement so a click cannot accidentally start resize or a short drag toggle the panel.
  • Confirm the globe canvas stays mounted through repeated Workspace/Node/Operations/Settings transitions and the viewport/WebGL drawing buffer respond to live resizing.
  • Exercise live tablet → desktop → phone layout changes, centered/adaptive dock geometry, touch swipes between Chat and Topology, and composer visibility when the viewport height is reduced.
  • Verify desktop Settings uses the full padded content width, keeps the shell/document fixed, owns its inner scroll area, and restores its scroll position after navigating away and back.

Verification

  • npm run typecheck and npm run build pass locally.
  • Full Electron E2E suite passed locally: 15/15 on the prior stack head; GitHub CI is rerunning on the rebased head.
  • Rebased onto dev after #195 landed. Interactive visual acceptance remains tracked separately in #98 and #99.
## Summary - Add Electron E2E coverage for panel open state and saved widths across reload and process relaunch. - Verify topology drag-close/reopen keeps pointer control, keyboard toggling works, and History reopens via a double-click on its collapsed handle. - Distinguish pointer taps from actual drag movement so a click cannot accidentally start resize or a short drag toggle the panel. - Confirm the globe canvas stays mounted through repeated Workspace/Node/Operations/Settings transitions and the viewport/WebGL drawing buffer respond to live resizing. - Exercise live tablet → desktop → phone layout changes, centered/adaptive dock geometry, touch swipes between Chat and Topology, and composer visibility when the viewport height is reduced. - Verify desktop Settings uses the full padded content width, keeps the shell/document fixed, owns its inner scroll area, and restores its scroll position after navigating away and back. ## Verification - `npm run typecheck` and `npm run build` pass locally. - Full Electron E2E suite passed locally: 15/15 on the prior stack head; GitHub CI is rerunning on the rebased head. - Rebased onto `dev` after #195 landed. Interactive visual acceptance remains tracked separately in #98 and #99.
Owner

Ran this branch here (Linux, under xvfb, WebGL drawn in software) at c0043fe: npm run typecheck and npm run build clean, and the full Electron suite passes, 15 of 15, including the WebGL test from #184 with the diagnostics I added there.

On #195's red mark: the macOS packaging job built and tested fine and then failed uploading its artifact (Failed to CreateArtifact: Unable to make request: ENOTFOUND), which is the runner's network and not the branch. I re-ran that one job and it passed.

Leaving the stack as it is until you say #184 can go in.

Ran this branch here (Linux, under `xvfb`, WebGL drawn in software) at `c0043fe`: `npm run typecheck` and `npm run build` clean, and the full Electron suite passes, 15 of 15, including the WebGL test from #184 with the diagnostics I added there. On #195's red mark: the macOS packaging job built and tested fine and then failed uploading its artifact (`Failed to CreateArtifact: Unable to make request: ENOTFOUND`), which is the runner's network and not the branch. I re-ran that one job and it passed. Leaving the stack as it is until you say #184 can go in.
Author
Owner

Thanks, Austin — #184 has now landed. I’m continuing bottom-up; I’m normalizing the dependent PR bases after the squash merge and will only merge each one once its own checks are green and the diff is clean.

Thanks, Austin — #184 has now landed. I’m continuing bottom-up; I’m normalizing the dependent PR bases after the squash merge and will only merge each one once its own checks are green and the diff is clean.
Sign in to join this conversation.
No Reviewers
No labels area:client testing
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Sisyphus/Sisyphus#197