Skip to content

Update MSTest analyzer documentation for 4.0.0#48969

Merged
BillWagner merged 4 commits intomainfrom
dev/ygerges/update-docs-analyzers
Oct 8, 2025
Merged

Update MSTest analyzer documentation for 4.0.0#48969
BillWagner merged 4 commits intomainfrom
dev/ygerges/update-docs-analyzers

Conversation

@Youssef1313
Copy link
Copy Markdown
Member

@Youssef1313 Youssef1313 commented Oct 8, 2025


Internal previews

Toggle expand/collapse
📄 File 🔗 Preview link
docs/core/testing/mstest-analyzers/mstest0001.md MSTEST0001: Explicitly enable or disable tests parallelization
docs/core/testing/mstest-analyzers/mstest0006.md "MSTEST0006: Avoid '[ExpectedException]'"
docs/core/testing/mstest-analyzers/mstest0007.md MSTEST0007: Use test attributes only on test methods
docs/core/testing/mstest-analyzers/mstest0017.md docs/core/testing/mstest-analyzers/mstest0017
docs/core/testing/mstest-analyzers/mstest0023.md MSTEST0023: Do not negate boolean assertions
docs/core/testing/mstest-analyzers/mstest0024.md "MSTEST0024: Do not store TestContext in a static member"
docs/core/testing/mstest-analyzers/mstest0025.md MSTEST0025: Use 'Assert.Fail' instead of an always-failing assert
docs/core/testing/mstest-analyzers/mstest0030.md docs/core/testing/mstest-analyzers/mstest0030
docs/core/testing/mstest-analyzers/mstest0031.md MSTEST0031: System.ComponentModel.DescriptionAttribute has no effect on test methods
docs/core/testing/mstest-analyzers/mstest0032.md "MSTEST0032: Review or remove the assertion as its condition is known to be always true."
docs/core/testing/mstest-analyzers/mstest0034.md MSTEST0034: Use ClassCleanupBehavior.EndOfClass with the [ClassCleanup]
docs/core/testing/mstest-analyzers/mstest0035.md docs/core/testing/mstest-analyzers/mstest0035
docs/core/testing/mstest-analyzers/mstest0037.md MSTEST0037: Use proper 'Assert' methods
docs/core/testing/mstest-analyzers/mstest0039.md "MSTEST0039: Use newer 'Assert.Throws' methods"
docs/core/testing/mstest-analyzers/mstest0045.md MSTEST0045: Use cooperative cancellation for timeout
docs/core/testing/mstest-analyzers/mstest0053.md docs/core/testing/mstest-analyzers/mstest0053
docs/core/testing/unit-testing-mstest-migration-v3-v4.md Migrate from MSTest v3 to MSTest v4

Copilot AI review requested due to automatic review settings October 8, 2025 09:18
@Youssef1313 Youssef1313 requested a review from a team as a code owner October 8, 2025 09:18
@dotnetrepoman dotnetrepoman bot added this to the October 2025 milestone Oct 8, 2025
Copy link
Copy Markdown
Contributor

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 updates the MSTest analyzer documentation to reflect changes in MSTest 4.0.0. The documentation is being updated to show that several analyzers have had their severity levels increased from Info to Warning, while others are no longer relevant due to API removals in the new version.

  • Updated severity levels for multiple analyzers from "Info" to "Info (Warning starting with 4.0.0)"
  • Added deprecation notes for analyzers that are no longer relevant in MSTest 4
  • Fixed table formatting inconsistencies in some files

Reviewed Changes

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

Show a summary per file
File Description
mstest0001.md Updated default severity to include warning note for 4.0.0
mstest0006.md Added note that analyzer is no longer relevant due to attribute removal
mstest0007.md Updated default severity to include warning note for 4.0.0
mstest0017.md Updated default severity to include warning note for 4.0.0
mstest0023.md Updated default severity and fixed table formatting
mstest0024.md Updated default severity to include warning note for 4.0.0
mstest0025.md Updated category and default severity, including warning note for 4.0.0
mstest0030.md Updated default severity to include warning note for 4.0.0
mstest0031.md Updated default severity to include warning note for 4.0.0
mstest0032.md Updated default severity to include warning note for 4.0.0
mstest0034.md Added note that analyzer is no longer relevant due to ClassCleanupBehavior removal
mstest0035.md Updated default severity to include warning note for 4.0.0
mstest0037.md Updated default severity to include warning note for 4.0.0
mstest0039.md Added note that analyzer is no longer relevant due to old assertion API removal
mstest0045.md Updated category and default severity, including warning note for 4.0.0
mstest0053.md Added note that analyzer is no longer relevant due to assertion API parameter removal

@Youssef1313 Youssef1313 requested a review from gewarren October 8, 2025 11:53
Copy link
Copy Markdown
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @Youssef1313

This LGTM, and I'll :shipit: now.

@BillWagner BillWagner merged commit cb1cfee into main Oct 8, 2025
10 checks passed
@BillWagner BillWagner deleted the dev/ygerges/update-docs-analyzers branch October 8, 2025 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants