Skip to content

Remove unused dependencies from Library.IntegrationTests#15527

Merged
nohwnd merged 1 commit intomicrosoft:mainfrom
nohwnd:fix/15491-extra-deps
Mar 20, 2026
Merged

Remove unused dependencies from Library.IntegrationTests#15527
nohwnd merged 1 commit intomicrosoft:mainfrom
nohwnd:fix/15491-extra-deps

Conversation

@nohwnd
Copy link
Member

@nohwnd nohwnd commented Mar 19, 2026

Remove package and project references that were copied from Acceptance.IntegrationTests but are not used by Library.IntegrationTests:

  • Microsoft.TestPlatform.QTools.Assets (WebTest assets, not used here)
  • NuGet.Versioning (no source usage)
  • System.Data.DataSetExtensions (no source usage)
  • Microsoft.TestPlatform.Extensions.HtmlLogger (no source usage)
  • Microsoft.TestPlatform.AdapterUtilities (no source usage)

Verified build succeeds after removal.

Fixes #15491

Remove package and project references that were copied from
Acceptance.IntegrationTests but are not used by Library.IntegrationTests:

- Microsoft.TestPlatform.QTools.Assets (WebTest assets, not used)
- NuGet.Versioning (no source usage)
- System.Data.DataSetExtensions (no source usage)
- Microsoft.TestPlatform.Extensions.HtmlLogger (no source usage)
- Microsoft.TestPlatform.AdapterUtilities (no source usage)

Fixes microsoft#15491

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 19, 2026 17:38
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes unused package and project references from Library.IntegrationTests to reduce dependency surface and avoid carrying over references copied from Acceptance.IntegrationTests.

Changes:

  • Dropped unused ProjectReferences to HtmlLogger and AdapterUtilities.
  • Dropped unused PackageReferences (QTools assets, NuGet.Versioning, DataSetExtensions).
  • Keeps build working while simplifying the test project’s dependencies (per description).

@nohwnd
Copy link
Member Author

nohwnd commented Mar 20, 2026

🚢

@nohwnd nohwnd added the 🚢 Ship it! Add to PRs where owner approves automated PR, but cannot approve because they "wrote it". label Mar 20, 2026
@nohwnd nohwnd enabled auto-merge (squash) March 20, 2026 08:33
@nohwnd nohwnd merged commit a4c9885 into microsoft:main Mar 20, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚢 Ship it! Add to PRs where owner approves automated PR, but cannot approve because they "wrote it".

Projects

None yet

Development

Successfully merging this pull request may close these issues.

extra dependencies in the new libary.integration tests

3 participants