Remove Azure.Identity dependency from data plane test projects (batch 3)#58388
Merged
Remove Azure.Identity dependency from data plane test projects (batch 3)#58388
Conversation
…tch 3) Azure.Core 1.53+ now contains the identity types, so these projects no longer need an explicit Azure.Identity reference. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
m-nash
requested review from
a team,
Han-msft,
PallabPaul,
ShaoAnLin,
alexathomases,
andpiccione,
anilba06,
asaflevi-ms,
christothes,
dargilco,
dipidoo,
gkostal,
glharper,
ivarprudnikov,
joseharriaga,
jpalvarezl,
koen-mertens,
leareai,
longli0,
msftsettiy,
musabbir,
ryazhang-microsoft,
tomsft,
trrwilson and
weirenlai
as code owners
Contributor
There was a problem hiding this comment.
Pull request overview
This PR continues the repo-wide cleanup to remove explicit Azure.Identity dependencies from data plane test projects, relying on Azure.Core (1.53+) to provide the identity types.
Changes:
- Removed
PackageReference Include="Azure.Identity"from multiple data plane test projects. - Removed
ProjectReferencetoAzure.Identity.csproj(and the associated TODO comment) from the confidentialledger test projects.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| sdk/ai/Azure.AI.Extensions.OpenAI/tests/Azure.AI.Extensions.OpenAI.Tests.csproj | Drops explicit Azure.Identity reference from test project dependencies. |
| sdk/ai/Azure.AI.Projects.Agents/tests/Azure.AI.Projects.Agents.Tests.csproj | Drops explicit Azure.Identity reference from test project dependencies. |
| sdk/attestation/Azure.Security.Attestation/tests/Azure.Security.Attestation.Tests.csproj | Drops explicit Azure.Identity reference from test project dependencies. |
| sdk/confidentialledger/Azure.Security.CodeTransparency/tests/Azure.Security.CodeTransparency.Tests.csproj | Removes Azure.Identity project reference/TODO (but currently leaves malformed XML). |
| sdk/confidentialledger/Azure.Security.ConfidentialLedger/tests/Azure.Security.ConfidentialLedger.Tests.csproj | Removes Azure.Identity project reference/TODO from test project dependencies. |
| sdk/face/Azure.AI.Vision.Face/tests/Azure.AI.Vision.Face.Tests.csproj | Drops explicit Azure.Identity reference from test project dependencies. |
| sdk/healthdataaiservices/Azure.Health.Deidentification/tests/Azure.Health.Deidentification.Tests.csproj | Drops explicit Azure.Identity reference from test project dependencies. |
| sdk/healthinsights/Azure.Health.Insights.RadiologyInsights/tests/Azure.Health.Insights.RadiologyInsights.Tests.csproj | Drops explicit Azure.Identity reference from test project dependencies. |
| sdk/loadtestservice/Azure.Developer.Playwright/tests/Azure.Developer.Playwright.Tests.csproj | Drops explicit Azure.Identity reference from test project dependencies. |
| sdk/openai/Azure.AI.OpenAI/tests/Azure.AI.OpenAI.Tests.csproj | Drops explicit Azure.Identity reference from test project dependencies. |
Comments suppressed due to low confidence (1)
sdk/confidentialledger/Azure.Security.CodeTransparency/tests/Azure.Security.CodeTransparency.Tests.csproj:26
- The project file appears to have malformed XML after removing the Azure.Identity reference: there are two consecutive
<ItemGroup>start tags (one empty/unclosed). This will cause MSBuild to fail parsing the .csproj. Remove the extra<ItemGroup>and ensure the remaining groups are properly opened/closed.
<ItemGroup>
<ItemGroup>
<Folder Include="SessionRecords\" />
<EmbeddedResource Include="TestFiles\**" LinkBase="EmbeddedTestFiles" />
</ItemGroup>
jsquire
approved these changes
Apr 20, 2026
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
jsquire
approved these changes
Apr 20, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Continues the effort to remove explicit
Azure.Identitydependencies from projects that no longer need them. Azure.Core 1.53+ now contains all identity types, so these projects can rely on the transitive Azure.Core reference.This is batch 3 of a multi-PR cleanup. Previous batches:
Changes
Removed
Azure.IdentityPackageReference(orProjectReference) from 10 data plane test projects:sdk/ai/Azure.AI.Extensions.OpenAI/tests/Azure.AI.Extensions.OpenAI.Tests.csprojsdk/ai/Azure.AI.Projects.Agents/tests/Azure.AI.Projects.Agents.Tests.csprojsdk/attestation/Azure.Security.Attestation/tests/Azure.Security.Attestation.Tests.csprojsdk/confidentialledger/Azure.Security.CodeTransparency/tests/Azure.Security.CodeTransparency.Tests.csprojsdk/confidentialledger/Azure.Security.ConfidentialLedger/tests/Azure.Security.ConfidentialLedger.Tests.csprojsdk/face/Azure.AI.Vision.Face/tests/Azure.AI.Vision.Face.Tests.csprojsdk/healthdataaiservices/Azure.Health.Deidentification/tests/Azure.Health.Deidentification.Tests.csprojsdk/healthinsights/Azure.Health.Insights.RadiologyInsights/tests/Azure.Health.Insights.RadiologyInsights.Tests.csprojsdk/loadtestservice/Azure.Developer.Playwright/tests/Azure.Developer.Playwright.Tests.csprojsdk/openai/Azure.AI.OpenAI/tests/Azure.AI.OpenAI.Tests.csprojTotal: 10 files changed, 14 deletions.
Notes
confidentialledgerprojects had aProjectReferenceto Azure.Identity (with aTODO: Revert to PackageReferencecomment); both the reference and the comment were removed since the types are now in Azure.Core.Validation
Built locally:
Azure.Security.Attestation.Tests✅Azure.Security.ConfidentialLedger.Tests✅Azure.AI.Vision.Face.Tests✅Azure.AI.OpenAIhas a pre-existing CS0115 build error onmain(AzureFileClient.Protocol.cs(114,39)) unrelated to this change.