Visitar URL original
Update matrix by nohwnd · Pull Request #15477 · microsoft/vstest · GitHub
Skip to content

Update matrix - #15477

Merged
Jakub Jareš (nohwnd) merged 33 commits into
microsoft:mainfrom
nohwnd:update-matrix
Mar 16, 2026
Merged

Update matrix#15477
Jakub Jareš (nohwnd) merged 33 commits into
microsoft:mainfrom
nohwnd:update-matrix

Conversation

@nohwnd

@nohwnd Jakub Jareš (nohwnd) commented Mar 10, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Description

Update versions. Look at how to do this automatically later.

Related issue

Fix #15475

Copilot AI review requested due to automatic review settings March 10, 2026 21:45
Copilot AI reviewed Mar 10, 2026
View reviewed changes

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

Updates the test/compatibility matrix by bumping framework/tooling versions and adjusting MSTest execution settings in test assets.

Changes:

  • Updated NUnit and MSTest version properties used by compatibility/test-asset builds.
  • Updated VSTest console version properties used for compatibility testing.
  • Added MSTest assembly-level parallelization configuration to two test asset projects.

Reviewed changes

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

File Description
test/TestAssets/MSTestProject2/UnitTest1.cs Adds assembly-level MSTest parallelization configuration to influence test execution behavior.
test/TestAssets/MSTestProject1/UnitTest1.cs Adds assembly-level MSTest parallelization configuration to influence test execution behavior.
eng/Versions.props Bumps NUnit/MSTest/VSTest versions used by build/test compatibility matrices.

Comment thread test/TestAssets/MSTestProject1/UnitTest1.cs Resolved
Comment thread test/TestAssets/MSTestProject2/UnitTest1.cs Resolved
Comment thread test/TestAssets/MSTestProject1/UnitTest1.cs Resolved
Jakub Jareš (nohwnd) and others added 2 commits March 10, 2026 22:57
@nohwnd
Remove runtimes
6577f1c
@nohwnd
Update test/TestAssets/MSTestProject2/UnitTest1.cs
f49a123 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 11, 2026 07:30
@nohwnd
Update test/TestAssets/MSTestProject1/UnitTest1.cs
4fb1487 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI reviewed Mar 11, 2026
View reviewed changes

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

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

Comment thread test/TestAssets/MSTestProject2/UnitTest1.cs Resolved
Comment thread eng/Versions.props Resolved
@nohwnd

Jakub Jareš (nohwnd) commented Mar 11, 2026

Copy link
Copy Markdown
Member Author

needs #15480 where we are using only recent vstest.console to run compatibility checks on vstest.console, this avoids needing the old runtimes which we remove here.

(testhost does not need the runtime, because we pass runtime.config.json of the project, if someone is using netcoreapp2.1 that is their fault and they should update from this unsupported version of .NET)

@nohwnd

Jakub Jareš (nohwnd) commented Mar 11, 2026

Copy link
Copy Markdown
Member Author

The other errors I don't know yet. Need to merge stuff first, too many moving parts right now.

@nohwnd
Update eng/Versions.props
fa5e4e1 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 12, 2026 07:28
Copilot AI reviewed Mar 12, 2026
View reviewed changes

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

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

Jakub Jareš (nohwnd) and others added 14 commits March 12, 2026 13:20
This was referenced Nov 11, 2026
Open
Open
Open
Merged
Open
Open
Open
Open
Merged
Open
Closed
Open
Open
Open
Merged
Open
Merged
Closed
Closed
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Open
Open
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
+1 more reviewer
@Youssef1313 Youssef Fahmy (Youssef1313) Youssef1313 approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Update versions of tested dependencies

3 participants

@nohwnd @Youssef1313