Skip to content

Conversation

Romfos
Copy link
Contributor

@Romfos Romfos commented Aug 17, 2025

Changes:

  • Run unit tests in Microsoft.Testing.Platform mode
  • Remove test dependency Microsoft.NET.Test.Sdk (needed for old vstest mode)
  • Update test dependencies

@Romfos Romfos changed the title Mtp Run unit tests in Microsoft.Testing.Platform mode Aug 17, 2025
@Romfos Romfos requested a review from Copilot August 17, 2025 12:15
Copy link

@Copilot 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

This PR migrates the NSubstitute test projects from the traditional test runner to Microsoft.Testing.Platform mode, updating test dependencies and configurations.

  • Configures test projects to use Microsoft.Testing.Platform with NUnit runner
  • Removes Microsoft.NET.Test.Sdk dependency and updates NUnit version
  • Updates BenchmarkDotNet version for performance testing

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
tests/NSubstitute.Documentation.Tests/NSubstitute.Documentation.Tests.csproj Configures Microsoft.Testing.Platform mode and updates test dependencies
tests/NSubstitute.Benchmarks/NSubstitute.Benchmarks.csproj Updates BenchmarkDotNet dependency version
tests/NSubstitute.Acceptance.Specs/NSubstitute.Acceptance.Specs.csproj Configures Microsoft.Testing.Platform mode and updates test dependencies

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

@Romfos Romfos merged commit b56e768 into nsubstitute:main Aug 23, 2025
8 checks passed
@Romfos Romfos mentioned this pull request Jul 27, 2025
23 tasks
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.

2 participants