[python] Add body-or-no-content Spector coverage

[Copilot]

Add Python SDK coverage for operations whose successful response is either a model body (HTTP 200) or no content (HTTP 204).

• Coverage: Shared sync and async tests for Azure and unbranded clients assert BlobLayout(content="hello") for 200 and None for 204.
• Response metadata: Verify raw status codes and x-ms-request-id headers for both responses.
• Changelog: Add an internal entry for the Python emitter.

• Fixes [python] add test case #12152

[azure-pipelines]

Azure Pipelines:
Successfully started running 1 pipeline(s).
1 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

[Copilot]

🟢 Approval recommended

The tests accurately cover both Spector scenarios across Azure and unbranded clients, with a valid changelog entry.

0 open findings

What changed in this PR

Adds Python SDK Spector coverage for successful responses containing either a model body or no content.

Changes:

• Adds synchronous and asynchronous shared tests for HTTP 200 and 204 responses.
• Verifies deserialized values, status codes, and response headers.
• Adds an internal Python emitter changelog entry.

File	Description
packages/​http-client-python/​tests/​mock_api/​shared/​test_response_body_or_no_content.py	Adds synchronous response tests.
packages/​http-client-python/​tests/​mock_api/​shared/​asynctests/​test_response_body_or_no_content_async.py	Adds equivalent asynchronous tests.
.chronus/​changes/​python-response-body-or-no-content-2026-10-12-3-14-0.md	Records the internal test coverage update.

🧠 Review effort: Balanced

---

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[azure-sdk-automation]

You can try these changes here

🛝 Playground	🌐 Website	🛝 VSCode Extension

[Yuchao Yan (msyyc)]

For each test case, we could use "cls" to check response header. But for response body, I hope to check it without "cls" so that we could confirm whether generated sdk code work as expected. Copilot

[Copilot]

For each test case, we could use "cls" to check response header. But for response body, I hope to check it without "cls" so that we could confirm whether generated sdk code work as expected. Copilot

Updated in 9811c2f: all sync and async cases assert BlobLayout or None through calls without cls; separate cls calls retain status/header checks. Runtime tests were blocked by dependency-feed DNS failure and missing test tools; syntax and whitespace checks passed.

[Copilot]

🟢 Approval recommended

The tests match the Spector scenarios and follow established shared sync/async testing conventions.

0 open findings

🧠 Review effort: Balanced

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/@typespec/http-client-python@12153

commit: d2d7169

[Copilot]

🟢 Approval recommended

The tests match the Spector scenarios and established shared-test conventions, with an appropriate changelog entry.

0 open findings

🧠 Review effort: Balanced

[github-actions]

All changed packages have been documented.

• ✅ @typespec/http-client-python

Show changes

@typespec/http-client-python - internal ✏️

Add sync and async Spector tests for responses with a model body or no content, including raw response status and headers.

[Yuchao Yan (msyyc)]

Fix CI failure https://github.com/microsoft/typespec/actions/runs/38210537898/job/114683453074?pr=12153 Copilot

[github-actions]

Python emitter diff

Baseline gh:2a5cea68b131b42688b6723eb1c9af44efc0e1e6 vs this PR.

No changes to generated output.

Rendered diff: inline on the run summary, or the emitter-diff-html artifact.

Informational check (eng/emitter-diff); does not block the PR.

[Copilot]

Fix CI failure https://github.com/microsoft/typespec/actions/runs/38210537898/job/114683453074?pr=12153 Copilot

Fixed in 1a261ab: added the generated bodyornocontent module name to CSpell’s dictionary. The exact CI spelling command now passes across all 9,166 files; formatting and lint also pass. Body assertions remain without cls. Full build was blocked locally by a certificate error fetching VS Code license notices.

[Copilot]

🟢 Approval recommended

The tests match the Spector scenarios and established shared sync/async testing conventions.

0 open findings

🧠 Review effort: Balanced

[Copilot]

🟢 Approval recommended

The tests match the Spector scenarios and established shared-test conventions with no unresolved issues.

0 open findings

🧠 Review effort: Balanced