molecule-ai/molecule-core
41
0
Fork 2
Code Issues 47 Pull Requests 6 Actions 132 Packages Projects Releases Wiki Activity

test(canvas+settings): add component test coverage for TokensTab, AttachmentImage, AttachmentAudio, AttachmentPDF, AttachmentTextPreview, ServiceGroup #641

Closed
core-uiux wants to merge 10 commits from test/settings-guard-coverage into main
pull from: test/settings-guard-coverage
merge into: molecule-ai:main
Conversation 4 Commits 10 Files Changed 16 +3645 -5

10 Commits

Author SHA1 Message Date
claude-ceo-assistant
8c5fa6fdb0 Merge branch 'main' into test/settings-guard-coverage
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 13s
Details
CI / Detect changes (pull_request) Successful in 23s
Details
E2E API Smoke Test / detect-changes (pull_request) Successful in 27s
Details
Handlers Postgres Integration / detect-changes (pull_request) Successful in 25s
Details
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 27s
Details
qa-review / approved (pull_request) Failing after 17s
Details
security-review / approved (pull_request) Failing after 18s
Details
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 27s
Details
gate-check-v3 / gate-check (pull_request) Successful in 28s
Details
sop-tier-check / tier-check (pull_request) Successful in 21s
Details
CI / Platform (Go) (pull_request) Successful in 10s
Details
CI / Shellcheck (E2E scripts) (pull_request) Successful in 6s
Details
CI / Python Lint & Test (pull_request) Successful in 9s
Details
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 9s
Details
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 8s
Details
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 7s
Details
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 7m41s
Details
CI / Canvas (Next.js) (pull_request) Successful in 12m14s
Details
CI / Canvas Deploy Reminder (pull_request) Has been skipped
Details
CI / all-required (pull_request) Successful in 4s
Details
audit-force-merge / audit (pull_request) Has been skipped
Details
2026-05-12 03:45:57 +00:00
Molecule AI Core-UIUX
63da98350e test(settings): add TokensTab coverage (12 cases)
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 2s
Details
Harness Replays / detect-changes (pull_request) Successful in 7s
Details
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 10s
Details
CI / Detect changes (pull_request) Successful in 12s
Details
qa-review / approved (pull_request) Failing after 10s
Details
security-review / approved (pull_request) Failing after 10s
Details
E2E API Smoke Test / detect-changes (pull_request) Successful in 14s
Details
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 14s
Details
sop-tier-check / tier-check (pull_request) Successful in 12s
Details
Handlers Postgres Integration / detect-changes (pull_request) Successful in 14s
Details
Harness Replays / Harness Replays (pull_request) Successful in 3s
Details
gate-check-v3 / gate-check (pull_request) Successful in 13s
Details
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 14s
Details
CI / Platform (Go) (pull_request) Successful in 5s
Details
CI / Shellcheck (E2E scripts) (pull_request) Successful in 3s
Details
CI / Python Lint & Test (pull_request) Successful in 4s
Details
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 4s
Details
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 4s
Details
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 4s
Details
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 8m5s
Details
CI / Canvas (Next.js) (pull_request) Successful in 12m10s
Details
CI / Canvas Deploy Reminder (pull_request) Has been skipped
Details
CI / all-required (pull_request) Successful in 5s
Details 
12 passing: loading spinner, empty state, token list rendering,
each token's prefix/age/Revoke button, API URL correctness, revoke
confirm + cancel dialogs, new-token creation + dismiss, create error,
network error banner.

Root bug fixed: confirm button search was unscoped — when the dialog
opened, two "Revoke" buttons existed (tok2's row + dialog confirm);
find() returned tok2's button first. Scoped the search to
document.querySelector('[role="dialog"]') to hit the correct target.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-12 03:40:31 +00:00
Molecule AI Core-UIUX
d0e54f3bd2 test(canvas/chat): add AttachmentTextPreview coverage (12 cases) 
Adds Vitest coverage for AttachmentTextPreview — inline text/code
preview with streaming fetch and expand/truncate.

Covers:
  - Loading skeleton (320x80) with aria-label
  - Ready state with correct text content
  - Filename shown in header
  - Expand button appears when lines > 10
  - Expand button hidden when all lines shown
  - Expand button updates display to full content
  - Download button calls onDownload
  - tone=user -> blue/accent border
  - tone=agent -> neutral border
  - Truncated notice when file exceeds 256 KB
  - Error -> AttachmentChip fallback
  - Cleanup on unmount

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-12 03:40:31 +00:00
Molecule AI Core-UIUX
e114706247 test(canvas/chat): add AttachmentAudio + AttachmentPDF coverage (18 cases) 
Adds Vitest coverage for two missing attachment renderers:

AttachmentAudio (9 cases):
  - Loading skeleton (280x40) with aria-label
  - <audio controls> with blob src when ready
  - Filename label in ready state
  - tone=user -> blue/accent border
  - tone=agent -> neutral border
  - Error -> AttachmentChip fallback
  - audio onError -> chip transition
  - External URI -> direct href, no fetch
  - Blob URL cleanup on unmount

AttachmentPDF (9 cases):
  - Loading skeleton with PdfGlyph + filename
  - Preview button with glyph, filename, "PDF" label
  - Lightbox opens with <embed> on click
  - Lightbox closes on Escape
  - tone=user -> blue/accent classes on button
  - tone=agent -> neutral border
  - Error -> AttachmentChip fallback
  - External URI -> direct href, no fetch
  - Blob URL cleanup on unmount

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-12 03:40:31 +00:00
Molecule AI Core-UIUX
ccb959571b test(canvas/chat): add AttachmentImage coverage (10 cases) 
Adds Vitest coverage for AttachmentImage — inline image thumbnail with
click-to-fullscreen lightbox. Covers: loading skeleton (240×180),
ready state with blob URL, tone=user/agent border classes, lightbox
open/close on click and Escape, AttachmentChip error fallback, img
onError transition to chip, external URI direct href (no fetch), and
blob URL cleanup on unmount.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-12 03:40:31 +00:00
Molecule AI Core-UIUX
2741f6db0a test(canvas/settings): add ServiceGroup coverage (10 cases) 
- role=group with aria-label containing service label
- Service icon aria-hidden, correct emoji per service name
- Count label: "1 key" vs "N keys"
- Renders SecretRow for each secret
- Header and rows div structure

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-12 03:40:31 +00:00
Molecule AI Core-UIUX
9e63981384 test(canvas/settings): add DeleteConfirmDialog + SettingsButton coverage (26 cases) 
- DeleteConfirmDialog (15 cases): dialog open via secret:delete-request event,
  title/body text, Cancel closes, dependents loading/list/none states,
  deleteSecret call, confirm 1s delay, disabled→enabled button transition
- SettingsButton (11 cases): aria-label, aria-expanded, gear SVG aria-hidden,
  toggle openPanel/closePanel, active class, tooltip Mac/Ctrl shortcut
  ResizeObserver polyfill for Radix Tooltip

No breaking changes. 2413 tests passing.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-12 03:40:31 +00:00
Molecule AI Core-UIUX
b2ab086f6c test(canvas/settings,chat): add coverage for EmptyState, SearchBar, UnsavedChangesGuard, AttachmentVideo 
- EmptyState: 6 cases — icon aria-hidden, title, body text, CTA button
- SearchBar: 14 cases — store binding, onChange, Escape, Ctrl/Cmd+F focus
- UnsavedChangesGuard: 7 cases — dialog states, Keep/Discard actions, backdrop
  FIX: UnsavedChangesGuard now wires onDiscard via pendingDiscard ref so
  clicking Discard correctly calls the callback on dialog close
- AttachmentVideo: 8 cases — loading/ready/error states, tone borders,
  blob URL cleanup, external URI direct href

No breaking changes. 2387 tests passing.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-12 03:40:31 +00:00
Molecule AI Core-UIUX
9df28b66b1 test(canvas/chat): add AttachmentLightbox coverage (13 cases) 
+ AttachmentLightbox.test.tsx: 13 cases across render states,
  interaction, and focus management for the shared fullscreen modal.

Per RFC #2991 Phase 2, AttachmentLightbox owns: backdrop + centered
viewport, Esc to close, click-outside to close, focus trap (focus
enters close button on open, restores on close), reduced-motion respect.

Coverage:
  - open=false → renders nothing
  - role=dialog + aria-modal + aria-label
  - Close button aria-label="Close preview"
  - Click backdrop → onClose (e.target === e.currentTarget)
  - Click content → onClose NOT called (stopPropagation)
  - Escape key → onClose
  - Focus moves to close button on open
  - Focus restores to previous element on close
  - motion-reduce class on backdrop

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-12 03:40:31 +00:00
Molecule AI Core-UIUX
4119edee23 test(canvas): add form-inputs coverage (35 cases) + Section accessibility fix 
+ form-inputs.test.tsx: 35 cases across TextInput, NumberInput, Toggle,
  TagList, and Section — pure presentational components in the Config tab.
  Uses vi.hoisted() patterns from established suite; no jest-dom matchers.

+ form-inputs.tsx (Section): add aria-expanded + aria-controls to the
  collapsible toggle button for WCAG 2.1 AA compliance. The content div
  gets a stable id derived from the title; aria-controls links button to
  region. Indicator span gains aria-hidden="true" (decorative only).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-12 03:40:31 +00:00