Skip to content

test: extend selected scenarios to REST - #1370

Open
faheem-shah-umer wants to merge 2 commits into
a2aproject:mainfrom
faheem-shah-umer:test/rest-scenarios
Open

faheem-shah-umer wants to merge 2 commits into
a2aproject:mainfrom
faheem-shah-umer:test/rest-scenarios

Conversation

@faheem-shah-umer

Copy link
Copy Markdown

Extend simple streaming, terminal-task cancellation and artifact delivery
scenarios to run over HTTP+JSON alongside gRPC, covering both current
and legacy handlers.

Add REST test-server setup and regression coverage for resource cleanup
after setup failures, exceptions and cancellation.

Validation on Windows/Python 3.10:

  • Lint, formatting and type checks passed.
  • Scenario and cleanup tests: 133 passed, 1 existing XPASS.
  • Non-integration suite: 1,878 passed, 169 skipped, 3 expected failures.
  • Live REST streaming and client cleanup passed for both handlers.
  • Full-suite runs encountered Windows subprocess failures involving
    os.killpg and FastAPI pickling. Full Linux CI verification remains pending.

@github-actions

github-actions Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

🧪 Code Coverage (vs main)

⬇️ Download Full Report

Base PR Delta
src/a2a/server/agent_execution/active_task_registry.py 97.37% 100.00% 🟢 +2.63%
src/a2a/server/cluster/database_event_stream.py 96.94% 92.86% 🔴 -4.08%
Total 93.00% 92.98% 🔴 -0.02%

Generated by coverage-comment.yml

@faheem-shah-umer
faheem-shah-umer marked this pull request as ready for review October 22, 2026 21:10
@faheem-shah-umer
faheem-shah-umer requested a review from a team as a code owner October 22, 2026 21:10

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant