molecule-ai/molecule-core
39
0
Fork 2
Code Issues 38 Pull Requests 28 Actions 5 Packages Projects Releases Wiki Activity

[CLOSED] superseded by PR #341 #319

Closed
core-be wants to merge 1 commits from fix/qa-audit-307-308-clean into main
pull from: fix/qa-audit-307-308-clean
merge into: molecule-ai:main
Conversation 12 Commits 1 Files Changed 2 +45 -35

1 Commits

Author SHA1 Message Date
Molecule AI Core-BE
1b68fef734 fix(QA-audit #307 #308): asyncio lifecycle fix + push-mode queue test coverage
All checks were successful
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 11s
Details
audit-force-merge / audit (pull_request) Has been skipped
Details
sop-tier-check / tier-check (pull_request) Successful in 13s
Details 
Issue #307 (regression, medium):
- test_a2a_tools_inbox_wrappers.py: replace _run() helper that called
  asyncio.get_event_loop().run_until_complete() — bypasses
  pytest-asyncio lifecycle, causing 14 tests to pass in isolation but
  exit-1 in full suite. Convert all test methods to
  @pytest.mark.asyncio async def / await.

Issue #308 (test gap, low):
- test_a2a_response.py: push-mode queue handling (PR #278, a2a_proxy.go
  push-at-capacity path) had no dedicated tests despite ~17 uncovered
  lines. Add 3 fixtures (push_queued_full/notify/no_method), 4 test
  cases covering classification, method field, method sentinel, and
  queue_id log output. Also add adversarial inputs for
  queued="yes" (string) and queued=False to confirm is True check.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-11 03:21:08 +00:00