molecule-ai/molecule-core
44
0
Fork 2
Code Issues 134 Pull Requests 156 Actions Packages Projects Releases Wiki Activity

test(workspace): add 21-case coverage for builtin_tools/a2a_tools and send_message_wrapper #367

Closed
fullstack-engineer wants to merge 2 commits from test/a2a-tools-and-send-message-coverage into main
pull from: test/a2a-tools-and-send-message-coverage
merge into: molecule-ai:main
Conversation 10 Commits 2 Files Changed 22
+1032 -338

2 Commits

Author SHA1 Message Date
fullstack-engineer 57f7d5a178 test(workspace): add 21-case coverage for builtin_tools/a2a_tools and
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 17s
Details
sop-tier-check / tier-check (pull_request) Successful in 32s
Details
audit-force-merge / audit (pull_request) Has been skipped
Details 
adapters/smolagents/send_message_wrapper

builtin_tools/a2a_tools: list_peers (3 cases), delegate_task (6 cases),
get_peers_summary (2 cases) — all using respx httpx mocking.

adapters/smolagents/send_message_wrapper: safe_send_message (10 cases)
covering HTML entity escaping, truncation, unicode, type coercion.

C1 HIGH security wrapper has 100% test coverage now.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-11 03:30:59 +00:00
fullstack-engineer bebf41a899 fix(canvas): repair 31 failing vitest tests (closes #344) 
- TestConnectionButton: replace toHaveAttribute with .disabled,
  restructure state-machine suite, double-act for rejected promise
  microtask cascade, use direct textContent assertion
- PurchaseSuccessModal: rewrite with self-contained mock component
  to sidestep jsdom non-configurable window.location.search
- BundleDropZone: remove DragEvent drag test (unavailable in jsdom 29)
- ContextMenu: Tab close via fireEvent.keyDown on menu element,
  offline item disabled check via .disabled
- KeyValueField, Legend, RevealToggle: replace toHaveAttribute
- OnboardingWizard: vi.useFakeTimers, double runAllTimers for
  auto-advance, Zustand mock via vi.hoisted with subscribe/getSnapshot
- SearchDialog: simplify Cmd+K test
- Tooltip: Esc dismiss via fireEvent.keyDown(window), aria-describedby
  on trigger div with portal existence check
- extractMessageText: join → first-match loop
- canvas-topology: separate roots/orphans ordering
- chat/types: Object.freeze + conditional attachments
- tree.ts: null-safe file extension

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-11 02:25:45 +00:00