Skip to content

test(python): cover Azure Core API-version overrides - #12011

Merged
Yuchao Yan (msyyc) merged 5 commits into
mainfrom
copilot/microsoft-12007-add-test-case-for-pull-5176
Oct 9, 2026
Merged

test(python): cover Azure Core API-version overrides#12011
Yuchao Yan (msyyc) merged 5 commits into
mainfrom
copilot/microsoft-12007-add-test-case-for-pull-5176

Conversation

Copilot AI commented Sep 18, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Adds Python mock API coverage for the Azure Core child-client API-version override scenario introduced by Azure/typespec-azure#5176.

  • Spector dependencies

    • Updates Azure HTTP specs, Azure Core, and TCGC prereleases required to compile the scenario.

  • Mock API coverage

    • Adds synchronous and asynchronous tests for the generated ApiVersionOverrideClient.
    • Verifies the child client call satisfies the Spector expectation for the overridden API version.
client.legacy_client.get()
# Sends: api-version=2022-10-01
  • Changelog
    • Adds an internal @typespec/http-client-python Chronus 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/5176 #12007
Closed
@github-actions github-actions Bot mentioned this pull request Sep 18, 2026
Closed
@msyyc
test(python): cover API version override scenario
3dfd016 
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 typespec-azure pull request 5176 test(python): cover Azure Core API-version overrides Sep 18, 2026
@msyyc

Yuchao Yan (msyyc) commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

Copilot resolve the merge conflicts in this pull request

@msyyc
Merge remote-tracking branch 'origin/main' into copilot/microsoft-120…
12c3845 
…07-add-test-case-for-pull-5176

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

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

Copilot AI commented Sep 29, 2026

Copy link
Copy Markdown
Contributor Author

Copilot resolve the merge conflicts in this pull request

Resolved and merged main into the PR branch in 12c3845f13.

@msyyc

Yuchao Yan (msyyc) commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Copilot resolve the merge conflicts in this pull request

@msyyc
Merge remote-tracking branch 'origin/main' into copilot/microsoft-120…
9a0ac09 
…07-add-test-case-for-pull-5176

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

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

Copilot AI commented Oct 8, 2026

Copy link
Copy Markdown
Contributor Author

Copilot resolve the merge conflicts in this pull request

Resolved and merged current main into the PR branch in 9a0ac09f46.

@msyyc
Yuchao Yan (msyyc) marked this pull request as ready for review October 9, 2026 02:59
Copilot AI balanced review requested due to automatic review settings October 9, 2026 02:59
@pkg-pr-new

pkg-pr-new Bot commented Oct 9, 2026

Copy link
Copy Markdown

Open in StackBlitz

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

commit: 5253357

Copilot AI reviewed Oct 9, 2026
View reviewed changes

Copilot AI left a comment

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.

🟢 Approval recommended

The tests follow established mock API patterns and cover both client modes with correct changelog metadata.

0 open findings

What changed in this PR

Adds sync and async Python mock API coverage for Azure Core child-client API-version overrides.

Changes:

  • Exercises legacy_client.get() against the Spector expectation.
  • Adds an internal Chronus entry.
File Description
test_azure_core_api_version_override.py Adds synchronous coverage.
test_azure_core_api_version_override_async.py Adds asynchronous coverage.
.chronus/​changes/​python-api-version-override-2026-09-18-02-03-00.md Records the test addition.

🧠 Review effort: Balanced

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

@github-actions

github-actions Bot commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

All changed packages have been documented.

  • ✅ @typespec/http-client-python
Show changes

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

Add mock API coverage for Azure Core child-client API-version overrides.

@github-actions

github-actions Bot commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

Python emitter diff

Baseline gh:f0cf4a73b447e794d92193efeeaf930ebbfdbd97 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.

@azure-sdk-automation

azure-sdk-automation Bot commented Oct 9, 2026

Copy link
Copy Markdown

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

@msyyc
Yuchao Yan (msyyc) added this pull request to the merge queue Oct 9, 2026
Merged via the queue into main with commit 62f1d29 Oct 9, 2026
38 checks passed
@msyyc
Yuchao Yan (msyyc) deleted the copilot/microsoft-12007-add-test-case-for-pull-5176 branch October 9, 2026 04:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@ChenxiJiang333 ChenxiJiang333 ChenxiJiang333 approved these changes
@iscai-msft iscai-msft Awaiting requested review from iscai-msft iscai-msft is a code owner
@tadelesh Chenjie Shi (tadelesh) Awaiting requested review from tadelesh tadelesh is a code owner
@timotheeguerin Timothee Guerin (timotheeguerin) Awaiting requested review from timotheeguerin timotheeguerin is a code owner
@lmazuel Laurent Mazuel (lmazuel) Awaiting requested review from lmazuel lmazuel is a code owner
@catalinaperalta catalinaperalta Awaiting requested review from catalinaperalta catalinaperalta is a code owner
@l0lawrence Libba Lawrence (l0lawrence) Awaiting requested review from l0lawrence l0lawrence is a code owner
@kashifkhan Kashif Khan (kashifkhan) Awaiting requested review from kashifkhan kashifkhan is a code owner
@xirzec Jeff Fisher (xirzec) Awaiting requested review from xirzec xirzec is a code owner
@markcowl Mark Cowlishaw (markcowl) Awaiting requested review from markcowl markcowl is a code owner
+1 more reviewer
@msyyc Yuchao Yan (msyyc) msyyc approved these changes
Reviewers whose approvals may not affect merge requirements

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/5176

4 participants

@msyyc @ChenxiJiang333