Skip to content

test(python): add response replacement mock API coverage - #12010

Draft
Yuchao Yan (msyyc) with Copilot wants to merge 4 commits into
mainfrom
copilot/add-test-case-for-5471
Draft

test(python): add response replacement mock API coverage#12010
Yuchao Yan (msyyc) with Copilot wants to merge 4 commits into
mainfrom
copilot/add-test-case-for-5471

Conversation

Copilot AI commented Sep 18, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Adds Python mock API coverage for Azure Spector response-replacement scenarios introduced in Azure/typespec-azure#5471.

Changes

  • Spector dependency

    • Updates @azure-tools/azure-http-specs to 0.1.0-alpha.46-dev.2.

  • Mock API coverage

    • Adds synchronous and asynchronous tests for void_response and bytes_response.
    • Verifies both operations deserialize the wire response:
assert client.void_response() == Widget(name="widget")
assert client.bytes_response() == Widget(name="widget")
  • Changelog
    • Adds an internal @typespec/http-client-python change entry.

@azure-pipelines

azure-pipelines Bot commented Sep 18, 2026

Copy link
Copy Markdown
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 AI linked an issue Sep 18, 2026 that may be closed by this pull request
[python] add test case for https://github.com/Azure/typespec-azure/pull/5471 #12006
Open
@msyyc
test(python): add response replacement mock API coverage
997baf9 
Co-authored-by: msyyc <70930885+msyyc@users.noreply.github.com>
@microsoft-github-policy-service microsoft-github-policy-service Bot added the emitter:client:python Issue for the Python client emitter: @typespec/http-client-python label Sep 18, 2026
Copilot AI changed the title [WIP] Add test case for Azure typespec pull request 5471 test(python): add response replacement mock API coverage Sep 18, 2026
@msyyc

Yuchao Yan (msyyc) commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

Copilot resolve the merge conflicts in this pull request

1 similar comment
@msyyc

Yuchao Yan (msyyc) commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

Copilot resolve the merge conflicts in this pull request

@msyyc
Merge remote-tracking branch 'origin/main' into copilot/add-test-case…
2dce6e2 
…-for-5471

# Conflicts:
#	packages/http-client-python/package.json

Co-authored-by: msyyc <70930885+msyyc@users.noreply.github.com>

Copilot AI commented Sep 24, 2026

Copy link
Copy Markdown
Contributor Author

Copilot resolve the merge conflicts in this pull request

Resolved the merge conflict and pushed merge commit 2dce6e2c.

Copilot AI commented Sep 24, 2026

Copy link
Copy Markdown
Contributor Author

Copilot resolve the merge conflicts in this pull request

The branch already contains the current main merge (2dce6e2c); no conflicts remain to resolve.

msyyc
Yuchao Yan (msyyc) reviewed Sep 29, 2026
View reviewed changes
Comment thread ...sts/mock_api/azure/asynctests/test_azure_client_generator_core_response_replacement_async.py


@pytest_asyncio.fixture
async def client():

@msyyc Yuchao Yan (msyyc) Sep 29, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot add "namespace" configuration in https://github.com/microsoft/typespec/blob/main/packages/http-client-python/eng/scripts/ci/regenerate-common.ts to keep consistent with other similar test cases then update test files.

Copilot AI Sep 29, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added the response-replacement namespace override and updated both test imports in d910354c.

@msyyc
test(python): configure response replacement namespace
d910354 
Co-authored-by: msyyc <70930885+msyyc@users.noreply.github.com>

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

Reviewers

@msyyc Yuchao Yan (msyyc) Awaiting requested review from msyyc msyyc is a code owner
@iscai-msft iscai-msft Awaiting requested review from iscai-msft iscai-msft will be requested when the pull request is marked ready for review iscai-msft is a code owner
@tadelesh Chenjie Shi (tadelesh) Awaiting requested review from tadelesh tadelesh will be requested when the pull request is marked ready for review tadelesh is a code owner
@timotheeguerin Timothee Guerin (timotheeguerin) Awaiting requested review from timotheeguerin timotheeguerin will be requested when the pull request is marked ready for review timotheeguerin is a code owner
@lmazuel Laurent Mazuel (lmazuel) Awaiting requested review from lmazuel lmazuel will be requested when the pull request is marked ready for review lmazuel is a code owner
@catalinaperalta catalinaperalta Awaiting requested review from catalinaperalta catalinaperalta will be requested when the pull request is marked ready for review catalinaperalta is a code owner
@ChenxiJiang333 ChenxiJiang333 Awaiting requested review from ChenxiJiang333 ChenxiJiang333 will be requested when the pull request is marked ready for review ChenxiJiang333 is a code owner
@l0lawrence Libba Lawrence (l0lawrence) Awaiting requested review from l0lawrence l0lawrence will be requested when the pull request is marked ready for review l0lawrence is a code owner
@kashifkhan Kashif Khan (kashifkhan) Awaiting requested review from kashifkhan kashifkhan will be requested when the pull request is marked ready for review kashifkhan is a code owner
@xirzec Jeff Fisher (xirzec) Awaiting requested review from xirzec xirzec will be requested when the pull request is marked ready for review xirzec is a code owner
@markcowl Mark Cowlishaw (markcowl) Awaiting requested review from markcowl markcowl will be requested when the pull request is marked ready for review markcowl is a code owner
Copilot code review Copilot
Copilot will automatically review once the pull request is marked ready for review

At least 1 approving review is required to merge this pull request.

Assignees

@msyyc Yuchao Yan (msyyc)

Copilot code review Copilot

Labels

emitter:client:python Issue for the Python client emitter: @typespec/http-client-python

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[python] add test case for https://github.com/Azure/typespec-azure/pull/5471

2 participants

@msyyc