molecule-ai/molecule-core
41
0
Fork 2
Code Issues 51 Pull Requests 21 Actions 8 Packages Projects Releases Wiki Activity

test(canvas+settings): add component test coverage (15 files, ~2500 lines) + 2 component fixes #666

Closed
app-fe wants to merge 8 commits from cleanup/pr-641-clean into main
pull from: cleanup/pr-641-clean
merge into: molecule-ai:main
Conversation 0 Commits 8 Files Changed 15 +3431 -5

8 Commits

Author SHA1 Message Date
Molecule AI Core-UIUX
c0d0d6bd1a test(canvas): add form-inputs coverage (35 cases) + Section accessibility fix
Some checks failed
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 10s
Details
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 17s
Details
Harness Replays / detect-changes (pull_request) Successful in 19s
Details
qa-review / approved (pull_request) Failing after 16s
Details
CI / Detect changes (pull_request) Successful in 28s
Details
security-review / approved (pull_request) Failing after 16s
Details
E2E API Smoke Test / detect-changes (pull_request) Successful in 32s
Details
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 32s
Details
gate-check-v3 / gate-check (pull_request) Successful in 27s
Details
Handlers Postgres Integration / detect-changes (pull_request) Successful in 31s
Details
CI / Platform (Go) (pull_request) Successful in 7s
Details
Runtime PR-Built Compatibility / detect-changes (pull_request) Successful in 31s
Details
CI / Shellcheck (E2E scripts) (pull_request) Successful in 7s
Details
CI / Python Lint & Test (pull_request) Successful in 8s
Details
Harness Replays / Harness Replays (pull_request) Successful in 8s
Details
sop-tier-check / tier-check (pull_request) Successful in 13s
Details
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 7s
Details
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 6s
Details
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Successful in 7s
Details
CI / Canvas (Next.js) (pull_request) Successful in 5m56s
Details
CI / Canvas Deploy Reminder (pull_request) Has been skipped
Details
CI / all-required (pull_request) Successful in 3s
Details
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 10m12s
Details
audit-force-merge / audit (pull_request) Has been skipped
Details 
+ 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 05:33:09 +00:00
Molecule AI Core-UIUX
f3fd486d4e 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 05:33:09 +00:00
Molecule AI Core-UIUX
2aceeb9ac3 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 05:33:09 +00:00
Molecule AI Core-UIUX
747cae8c15 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 05:33:09 +00:00
Molecule AI Core-UIUX
b52fa5c065 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 05:33:09 +00:00
Molecule AI Core-UIUX
2793e2425b 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 05:33:09 +00:00
Molecule AI Core-UIUX
9fc718cd3d 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 05:33:08 +00:00
Molecule AI Core-UIUX
59f738a5d3 test(settings): add TokensTab coverage (12 cases) 
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 05:33:08 +00:00