test(canvas): add test coverage for canvas, mobile, settings, and FilesTab (22 files) #783

Merged
devops-engineer merged 9 commits from design/704-tree-test-fix into main 2026-05-13 09:29:25 +00:00

9 Commits

Author SHA1 Message Date
de21d4a482 test(FilesTab): add FilesToolbar + NotAvailablePanel coverage
All checks were successful
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 13s
CI / Detect changes (pull_request) Successful in 24s
E2E API Smoke Test / detect-changes (pull_request) Successful in 29s
Harness Replays / detect-changes (pull_request) Successful in 20s
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 36s
Handlers Postgres Integration / detect-changes (pull_request) Successful in 35s
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 17s
CI / Platform (Go) (pull_request) Successful in 7s
sop-checklist-gate / gate (pull_request) Successful in 17s
qa-review / approved (pull_request) Successful in 20s
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 29s
CI / Shellcheck (E2E scripts) (pull_request) Successful in 6s
CI / Python Lint & Test (pull_request) Successful in 7s
sop-tier-check / tier-check (pull_request) Successful in 23s
gate-check-v3 / gate-check (pull_request) Successful in 34s
Harness Replays / Harness Replays (pull_request) Successful in 6s
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 9s
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 6s
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 7s
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 1m15s
sop-checklist / all-items-acked (pull_request) [info tier:low] acked: 0/7 — missing: comprehensive-testing, local-postgres-e2e, staging-smoke, +4 — body-unfilled: comprehensive-testing, l
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 7m43s
CI / Canvas (Next.js) (pull_request) Successful in 12m8s
CI / Canvas Deploy Reminder (pull_request) Has been skipped
CI / all-required (pull_request) Successful in 2s
security-review / approved (pull_request) bootstrap-ok: test-only PR, no security-sensitive changes
audit-force-merge / audit (pull_request) Successful in 4s
Cherry-picked from test/settings-tab-coverage.
- FilesToolbar.test.tsx: 349 lines
- NotAvailablePanel.test.tsx: 101 lines

Total: 197 test files, 3076 tests passing.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-13 09:15:14 +00:00
d0ad8c76fa test(FilesTab): add useFilesApi coverage — 7 cases
Cherry-picked from test/settings-tab-coverage (commit 46086ef6).
Covers file entry walking and API interactions.

Total: 195 test files, 3047 tests passing.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-13 09:15:14 +00:00
5c2238265f test: add components-pure + TestConnectionButton coverage
Cherry-picked from test/settings-tab-coverage (commit 226b7679).
- components-pure.test.ts: 184 lines, toMobileAgent + classifyForFilter
- TestConnectionButton.test.tsx: 245 lines, 29 test cases

Total: 194 test files, 3040 tests passing.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-13 09:15:14 +00:00
9378720c96 test(canvas): add TopBar + FileEditor + AttachmentLightbox coverage
Cherry-picked from test/settings-tab-coverage (commit 36d93f21).
- canvas/TopBar.test.tsx: 97 lines, canvas header scaffold rendering
- FileEditor.test.tsx: 312 lines, file editor rendering + interactions
- AttachmentLightbox.test.tsx: 247 lines, image lightbox rendering

Total: 192 test files, 3006 tests passing.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-13 09:15:14 +00:00
2eb3f3eade test(mobile): add MobileHome + MobileMe + MobileChat + MobileDetail coverage
Cherry-picked from test/settings-tab-coverage (commit fd424dba).
- MobileHome.test.tsx: 245 lines, agent list + filter chips
- MobileMe.test.tsx: 212 lines, Me screen rendering
- MobileChat.test.tsx: 323 lines, chat thread + composer
- MobileDetail.test.tsx: 367 lines, agent detail view

Makes #727 a complete superset of all mobile screen test coverage.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-13 09:15:14 +00:00
0e9709b2bf test(canvas): add SidePanel + TemplatePalette coverage
Cherry-picked from test/settings-tab-coverage (PRs #708/#726).
- SidePanel.general.test.tsx: 390 lines
- TemplatePalette.test.tsx: 260 lines

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-13 09:15:14 +00:00
2ca269fec0 test(settings): add AddKeyForm + OrgTokensTab + SecretRow + SecretsTab coverage
Cherry-picked from test/settings-tab-coverage (PRs #708/#726).
- AddKeyForm: 340 lines, form validation + submission tests
- OrgTokensTab: 407 lines, org token CRUD + display tests
- SecretRow: 291 lines, secret display + reveal/copy/delete actions
- SecretsTab: 308 lines, secrets list + empty state + add form

Makes #704 a true superset of all settings test coverage.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-13 09:15:14 +00:00
ec51e5f381 test(settings): add SettingsPanel coverage — 14 cases
Covers: closed-by-default, open/close, tab navigation (Secrets/Tokens/Org API Keys),
unsaved guard integration (keep editing, discard), fetchSecrets on open,
aria-label accessibility.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-13 09:15:14 +00:00
be6ca035a8 test(canvas/tabs): add tree.test.ts — 29 cases for FilesTab getIcon + buildTree
Cherry-picked from test/settings-tab-coverage (PR #726).
Covers: getIcon extension matching (upper/lowercase, no-ext), buildTree
node-counting (file/folder/total), root-vs-nested classification.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-13 09:15:14 +00:00