Skip to content

Verify client-location spector tests are complete - #865

Closed
antkmsft with Copilot wants to merge 1 commit into
mainfrom
copilot/add-spector-test-azure-http-specs-again
Closed

Verify client-location spector tests are complete#865
antkmsft with Copilot wants to merge 1 commit into
mainfrom
copilot/add-spector-test-azure-http-specs-again

Conversation

Copilot AI commented Feb 17, 2026

Copy link
Copy Markdown
Contributor

The issue requested adding spector tests for azure/client-generator-core/client-location/, but all tests already exist and pass.

Verification Results

All 4 client-location scenarios are implemented and functional:

  • move-to-root-client: Tests @clientLocation moving operations to root client (2 tests)
  • move-method-parameter-to-client: Tests moving method parameters to client constructor (1 test)
  • move-to-existing-sub-client: Tests moving operations between existing sub-clients (3 tests)
  • move-to-new-sub-client: Tests creating new sub-clients via @clientLocation (2 tests)

All 8 tests pass against spector server and are tracked as "pass" in spec-coverage.json. Tests are properly configured in tspcompile.js and workspace Cargo.toml.

No code changes required.

Original prompt

This section details on the original issue you should resolve

<issue_title>Add spector test for azure-http-specs\azure\client-generator-core\client-location</issue_title>
<issue_description>Do not modify the generated rust source code (in src/generated), only modify the emitter code (in /packages/typespec-rust/src/) if necessary.</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Add spector test for azure-http-specs client location Verify client-location spector tests are complete Feb 17, 2026
Copilot AI requested a review from antkmsft February 17, 2026 16:26
@antkmsft antkmsft closed this Feb 18, 2026
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.

Add spector test for azure-http-specs\azure\client-generator-core\client-location\

2 participants