Skip to content

Update NUnit to v4#6287

Open
Trenly wants to merge 3 commits into
microsoft:masterfrom
Trenly:NUnitUpdate
Open

Update NUnit to v4#6287
Trenly wants to merge 3 commits into
microsoft:masterfrom
Trenly:NUnitUpdate

Conversation

@Trenly

@Trenly Trenly commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

📖 Description

Upgrade the AppInstallerCLIE2ETests suite to NUnit 4 and modernize the supporting test code to the NUnit 4 constraint model. This lands on the latest available NUnit 4.x release in NuGet (4.6.1), while leaving NUnit3TestAdapter unchanged.

The update converts the E2E suite off NUnit 3 classic assertions, fixes the delegate-based assertion overloads required by NUnit 4, and carries the package version change through the affected test projects that use centralized package management.

I opted for full migration instead of the simpler Directory.Build.props change as I expect this will be a one-time migration and I generally believe that using the newer format will be more performant in the future (plus I had the extra tokens to burn, so why not)

🔗 References

🔍 Validation

  • Search-based verification confirmed the old NUnit 3 classic assertions were removed from src/AppInstallerCLIE2ETests.

✅ Checklist

📋 Issue Type

  • Bug fix
  • Feature
  • Task
Microsoft Reviewers: Open in CodeFlow

@Trenly Trenly marked this pull request as ready for review June 16, 2026 03:03
@Trenly Trenly requested a review from a team as a code owner June 16, 2026 03:03
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.

1 participant