Skip to content

Remove unused dependencies from Library.IntegrationTests - #15527

Merged
Jakub Jareš (nohwnd) merged 1 commit into
microsoft:mainfrom
nohwnd:fix/15491-extra-deps
Mar 20, 2026
Merged

Remove unused dependencies from Library.IntegrationTests#15527
Jakub Jareš (nohwnd) merged 1 commit into
microsoft:mainfrom
nohwnd:fix/15491-extra-deps

Conversation

@nohwnd

Copy link
Copy Markdown
Member

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

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.

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
Copy Markdown
Member Author

🚢

@nohwnd Jakub Jareš (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
Jakub Jareš (nohwnd) enabled auto-merge (squash) March 20, 2026 08:33
@nohwnd
Jakub Jareš (nohwnd) merged commit a4c9885 into microsoft:main Mar 20, 2026
7 checks passed
This was referenced May 26, 2026
This was referenced Aug 24, 2026
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