Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

More E2E test fixes - #6962

#6962
Merged
nkolev92 merged 4 commits into
devNuGet/NuGet.Client:devfrom
dev-nkolev92-moreE2EFixesNuGet/NuGet.Client:dev-nkolev92-moreE2EFixesCopy head branch name to clipboard
Nov 24, 2025
Merged

More E2E test fixes#6962
nkolev92 merged 4 commits into
devNuGet/NuGet.Client:devfrom
dev-nkolev92-moreE2EFixesNuGet/NuGet.Client:dev-nkolev92-moreE2EFixesCopy head branch name to clipboard

Conversation

@nkolev92

Copy link
Copy Markdown
Member

Bug

Progress: https://github.com/NuGet/Client.Engineering/issues/3543

Description

As part of improving the VS validation, we need to fix up our E2E tests so that they're something we can rely on.
After this change, I think most of the failing tests are actually failing or they're impacted by this migration issue with website projects.

I'll do that in a separate PR, but at this point I'd lean towards disabling those tests and making the pipeline green and something we can rely on.

PR Checklist

  • Meaningful title, helpful description and a linked NuGet/Home issue
  • Added tests
  • Link to an issue or pull request to update docs if this PR changes settings, environment variables, new feature, etc.

@nkolev92
nkolev92 marked this pull request as ready for review November 20, 2025 22:44
@nkolev92
nkolev92 requested a review from a team as a code owner November 20, 2025 22:44

@donnie-msft donnie-msft 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.

Glad to see more project.json dead code going away!

Comment thread test/EndToEnd/tests/BuildIntegratedTest.ps1
@nkolev92
nkolev92 merged commit 48f1ab1 into dev Nov 24, 2025
17 of 18 checks passed
@nkolev92
nkolev92 deleted the dev-nkolev92-moreE2EFixes branch November 24, 2025 19:56
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.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.