test(canvas/tabs): add tree.test.ts to fix/canvas-keyboard-shortcuts-dialog-guard #727

Merged
core-uiux merged 9 commits from design/704-tree-test-fix into fix/canvas-keyboard-shortcuts-dialog-guard 2026-05-12 16:10:06 +00:00

9 Commits

Author SHA1 Message Date
e34bee510d test(FilesTab): add FilesToolbar + NotAvailablePanel coverage
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 5s
sop-checklist / all-items-acked (pull_request) [soft-fail tier:low] acked: 0/7 — missing: comprehensive-testing, local-postgres-e2e, staging-smoke, +4 — body-unfilled: 7
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 8s
security-review / approved (pull_request) Failing after 7s
qa-review / approved (pull_request) Failing after 7s
gate-check-v3 / gate-check (pull_request) Successful in 8s
sop-checklist-gate / gate (pull_request) Successful in 8s
sop-tier-check / tier-check (pull_request) Successful in 7s
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 52s
audit-force-merge / audit (pull_request) Successful in 14s
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-12 15:39:14 +00:00
b67269aee7 test(FilesTab): add useFilesApi coverage — 7 cases
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 18s
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 20s
qa-review / approved (pull_request) Failing after 23s
security-review / approved (pull_request) Failing after 22s
gate-check-v3 / gate-check (pull_request) Successful in 35s
sop-checklist / all-items-acked (pull_request) [soft-fail tier:low] acked: 0/7 — missing: comprehensive-testing, local-postgres-e2e, staging-smoke, +4 — body-unfilled: 7
sop-checklist-gate / gate (pull_request) Successful in 17s
sop-tier-check / tier-check (pull_request) Successful in 18s
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 1m25s
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-12 14:41:44 +00:00
ef4426e6b1 test: add components-pure + TestConnectionButton coverage
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 14s
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 17s
sop-checklist / all-items-acked (pull_request) [soft-fail tier:low] acked: 0/7 — missing: comprehensive-testing, local-postgres-e2e, staging-smoke, +4 — body-unfilled: 7
security-review / approved (pull_request) Failing after 17s
qa-review / approved (pull_request) Failing after 18s
sop-checklist-gate / gate (pull_request) Successful in 16s
sop-tier-check / tier-check (pull_request) Successful in 16s
gate-check-v3 / gate-check (pull_request) Successful in 25s
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 1m18s
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-12 14:26:17 +00:00
429111ec81 test(canvas): add TopBar + FileEditor + AttachmentLightbox coverage
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 4s
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 4s
qa-review / approved (pull_request) Failing after 5s
security-review / approved (pull_request) Failing after 5s
sop-checklist / all-items-acked (pull_request) [soft-fail tier:low] acked: 0/7 — missing: comprehensive-testing, local-postgres-e2e, staging-smoke, +4 — body-unfilled: 7
sop-checklist-gate / gate (pull_request) Successful in 4s
gate-check-v3 / gate-check (pull_request) Successful in 4s
sop-tier-check / tier-check (pull_request) Successful in 4s
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 56s
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-12 14:12:06 +00:00
886cc3f152 test(mobile): add MobileHome + MobileMe + MobileChat + MobileDetail coverage
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 4s
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 5s
qa-review / approved (pull_request) Failing after 5s
security-review / approved (pull_request) Failing after 6s
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 55s
sop-checklist / all-items-acked (pull_request) [soft-fail tier:low] acked: 0/7 — missing: comprehensive-testing, local-postgres-e2e, staging-smoke, +4 — body-unfilled: 7
sop-checklist-gate / gate (pull_request) Successful in 9s
sop-tier-check / tier-check (pull_request) Successful in 9s
gate-check-v3 / gate-check (pull_request) Successful in 11s
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-12 13:55:59 +00:00
0a81621437 test(canvas): add SidePanel + TemplatePalette coverage
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 4s
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 5s
qa-review / approved (pull_request) Failing after 6s
security-review / approved (pull_request) Failing after 6s
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 51s
sop-checklist / all-items-acked (pull_request) [soft-fail tier:low] acked: 0/7 — missing: comprehensive-testing, local-postgres-e2e, staging-smoke, +4 — body-unfilled: 7
gate-check-v3 / gate-check (pull_request) Successful in 4s
sop-checklist-gate / gate (pull_request) Successful in 3s
sop-tier-check / tier-check (pull_request) Successful in 4s
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-12 13:32:55 +00:00
27c5bab2a3 test(settings): add AddKeyForm + OrgTokensTab + SecretRow + SecretsTab coverage
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 5s
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 5s
security-review / approved (pull_request) Failing after 6s
qa-review / approved (pull_request) Failing after 6s
sop-checklist / all-items-acked (pull_request) [soft-fail tier:low] acked: 0/7 — missing: comprehensive-testing, local-postgres-e2e, staging-smoke, +4 — body-unfilled: 7
sop-checklist-gate / gate (pull_request) Successful in 3s
gate-check-v3 / gate-check (pull_request) Successful in 3s
sop-tier-check / tier-check (pull_request) Successful in 4s
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 59s
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-12 13:29:32 +00:00
6d5ce30ff3 test(settings): add SettingsPanel coverage — 14 cases
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 13s
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 13s
sop-checklist / all-items-acked (pull_request) [soft-fail tier:low] acked: 0/7 — missing: comprehensive-testing, local-postgres-e2e, staging-smoke, +4 — body-unfilled: 7
qa-review / approved (pull_request) Failing after 16s
security-review / approved (pull_request) Failing after 14s
sop-checklist-gate / gate (pull_request) Successful in 14s
sop-tier-check / tier-check (pull_request) Successful in 15s
gate-check-v3 / gate-check (pull_request) Successful in 18s
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 1m9s
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-12 13:26:44 +00:00
c16e125617 test(canvas/tabs): add tree.test.ts — 29 cases for FilesTab getIcon + buildTree
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 11s
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 17s
sop-checklist / all-items-acked (pull_request) acked: 0/7 — missing: comprehensive-testing, local-postgres-e2e, staging-smoke, +4 — body-unfilled: 7
qa-review / approved (pull_request) Failing after 19s
security-review / approved (pull_request) Failing after 19s
sop-tier-check / tier-check (pull_request) Successful in 18s
sop-checklist-gate / gate (pull_request) Successful in 20s
gate-check-v3 / gate-check (pull_request) Successful in 25s
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 1m9s
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-12 12:55:06 +00:00