Update MSTest analyzer documentation for 4.0.0 (#48969)

Youssef1313 · web-flow · commit cb1cfee0a3d3 · 2025-10-08T08:58:59.000-04:00
* Update MSTest analyzer documentation for 4.0.0

* Apply suggestions from code review

* Table formatting

* Update unit-testing-mstest-migration-v3-v4.md
diff --git a/docs/core/testing/mstest-analyzers/mstest0001.md b/docs/core/testing/mstest-analyzers/mstest0001.md
@@ -23,7 +23,7 @@ dev_langs:
 | **Category**                        | Performance                                        |
 | **Fix is breaking or non-breaking** | Non-breaking                                       |
 | **Enabled by default**              | Yes                                                |
-| **Default severity**                | Info                                               |
+| **Default severity**                | Warning starting with 4.0.0, Info before           |
 | **Introduced in version**           | 3.2.0                                              |
 | **Is there a code fix**             | No                                                 |
 
diff --git a/docs/core/testing/mstest-analyzers/mstest0006.md b/docs/core/testing/mstest-analyzers/mstest0006.md
@@ -27,6 +27,9 @@ dev_langs:
 | **Introduced in version**           | 3.2.0                                              |
 | **Is there a code fix**             | Yes, starting with 3.7.0                           |
 
+> [NOTE]
+> This analyzer is no longer relevant for MSTest 4 as the attribute was removed.
+
 ## Cause
 
 A method is marked with the `[ExpectedException]` attribute.
diff --git a/docs/core/testing/mstest-analyzers/mstest0007.md b/docs/core/testing/mstest-analyzers/mstest0007.md
@@ -20,7 +20,7 @@ ms.author: copoiena
 | **Category**                        | Usage                                    |
 | **Fix is breaking or non-breaking** | Non-breaking                             |
 | **Enabled by default**              | Yes                                      |
-| **Default severity**                | Info                                     |
+| **Default severity**                | Warning starting with 4.0.0, Info before |
 | **Introduced in version**           | 3.3.0                                    |
 | **Is there a code fix**             | Yes                                      |
 
diff --git a/docs/core/testing/mstest-analyzers/mstest0017.md b/docs/core/testing/mstest-analyzers/mstest0017.md
@@ -20,7 +20,7 @@ ms.author: amauryleve
 | **Category**                        | Usage                                                               |
 | **Fix is breaking or non-breaking** | Non-breaking                                                        |
 | **Enabled by default**              | Yes                                                                 |
-| **Default severity**                | Info                                                                |
+| **Default severity**                | Warning starting with 4.0.0, Info before                            |
 | **Introduced in version**           | 3.4.0                                                               |
 | **Is there a code fix**             | Yes                                                                 |
 
diff --git a/docs/core/testing/mstest-analyzers/mstest0023.md b/docs/core/testing/mstest-analyzers/mstest0023.md
@@ -13,16 +13,16 @@ ms.author: amauryleve
 ---
 # MSTEST0023: Do not negate boolean assertions
 
-| Property                            | Value                            |
-|-------------------------------------|----------------------------------|
-| **Rule ID**                         | MSTEST0023                       |
-| **Title**                           | Do not negate boolean assertions |
-| **Category**                        | Usage                            |
-| **Fix is breaking or non-breaking** | Non-breaking                     |
-| **Enabled by default**              | Yes                              |
-| **Default severity**                | Info                             |
-| **Introduced in version**           | 3.4.0                            |
-| **Is there a code fix**             | No                               |
+| Property                            | Value                                    |
+|-------------------------------------|------------------------------------------|
+| **Rule ID**                         | MSTEST0023                               |
+| **Title**                           | Do not negate boolean assertions         |
+| **Category**                        | Usage                                    |
+| **Fix is breaking or non-breaking** | Non-breaking                             |
+| **Enabled by default**              | Yes                                      |
+| **Default severity**                | Warning starting with 4.0.0, Info before |
+| **Introduced in version**           | 3.4.0                                    |
+| **Is there a code fix**             | No                                       |
 
 ## Cause
 
diff --git a/docs/core/testing/mstest-analyzers/mstest0024.md b/docs/core/testing/mstest-analyzers/mstest0024.md
@@ -20,7 +20,7 @@ ms.author: amauryleve
 | **Category**                        | Usage                                       |
 | **Fix is breaking or non-breaking** | Non-breaking                                |
 | **Enabled by default**              | Yes                                         |
-| **Default severity**                | Info                                        |
+| **Default severity**                | Warning starting with 4.0.0, Info before    |
 | **Introduced in version**           | 3.4.0                                       |
 | **Is there a code fix**             | No                                          |
 
diff --git a/docs/core/testing/mstest-analyzers/mstest0025.md b/docs/core/testing/mstest-analyzers/mstest0025.md
@@ -17,10 +17,10 @@ ms.author: amauryleve
 |-------------------------------------|-------------------------------------------------------|
 | **Rule ID**                         | MSTEST0025                                            |
 | **Title**                           | Use 'Assert.Fail' instead of an always-failing assert |
-| **Category**                        | Design                                                |
+| **Category**                        | Usage                                                 |
 | **Fix is breaking or non-breaking** | Non-breaking                                          |
 | **Enabled by default**              | Yes                                                   |
-| **Default severity**                | Info                                                  |
+| **Default severity**                | Warning starting with 4.0.0, Info before              |
 | **Introduced in version**           | 3.4.0                                                 |
 | **Is there a code fix**             | Yes                                                   |
 
diff --git a/docs/core/testing/mstest-analyzers/mstest0030.md b/docs/core/testing/mstest-analyzers/mstest0030.md
@@ -20,7 +20,7 @@ ms.author: enjieid
 | **Category**                        | Usage                                                              |
 | **Fix is breaking or non-breaking** | Non-breaking                                                       |
 | **Enabled by default**              | Yes                                                                |
-| **Default severity**                | Info                                                               |
+| **Default severity**                | Warning starting with 4.0.0, Info before                           |
 | **Introduced in version**           | 3.5.0                                                              |
 | **Is there a code fix**             | No                                                                 |
 
diff --git a/docs/core/testing/mstest-analyzers/mstest0031.md b/docs/core/testing/mstest-analyzers/mstest0031.md
@@ -20,7 +20,7 @@ ms.author: enjieid
 | **Category**                        | Usage                                                                       |
 | **Fix is breaking or non-breaking** | Non-breaking                                                                |
 | **Enabled by default**              | Yes                                                                         |
-| **Default severity**                | Info                                                                        |
+| **Default severity**                | Warning starting with 4.0.0, Info before                                    |
 | **Introduced in version**           | 3.5.0                                                                       |
 | **Is there a code fix**             | No                                                                          |
 
diff --git a/docs/core/testing/mstest-analyzers/mstest0032.md b/docs/core/testing/mstest-analyzers/mstest0032.md
@@ -20,7 +20,7 @@ ms.author: enjieid
 | **Category**                        | Usage                                                                       |
 | **Fix is breaking or non-breaking** | Non-breaking                                                                |
 | **Enabled by default**              | Yes                                                                         |
-| **Default severity**                | Info                                                                        |
+| **Default severity**                | Warning starting with 4.0.0, Info before                                    |
 | **Introduced in version**           | 3.5.0                                                                       |
 | **Is there a code fix**             | No                                                                          |
 
diff --git a/docs/core/testing/mstest-analyzers/mstest0034.md b/docs/core/testing/mstest-analyzers/mstest0034.md
@@ -24,6 +24,9 @@ ms.author: enjieid
 | **Introduced in version**           | 3.6.0                                                            |
 | **Is there a code fix**             | No                                                               |
 
+> [NOTE]
+> This analyzer is no longer relevant for MSTest 4 as `ClassCleanupBehavior` was removed.
+
 ## Cause
 
 This rule raises a diagnostic when `ClassCleanupBehavior.EndOfClass` isn't set with the `[ClassCleanup]`.
diff --git a/docs/core/testing/mstest-analyzers/mstest0035.md b/docs/core/testing/mstest-analyzers/mstest0035.md
@@ -20,7 +20,7 @@ ms.author: enjieid
 | **Category**                        | Usage                                                                  |
 | **Fix is breaking or non-breaking** | Non-breaking                                                           |
 | **Enabled by default**              | Yes                                                                    |
-| **Default severity**                | Info                                                                   |
+| **Default severity**                | Warning starting with 4.0.0, Info before                               |
 | **Introduced in version**           | 3.6.0                                                                  |
 | **Is there a code fix**             | No                                                                     |
 
diff --git a/docs/core/testing/mstest-analyzers/mstest0037.md b/docs/core/testing/mstest-analyzers/mstest0037.md
@@ -20,7 +20,7 @@ ms.author: ygerges
 | **Category**                        | Usage                                                                  |
 | **Fix is breaking or non-breaking** | Non-breaking                                                           |
 | **Enabled by default**              | Yes                                                                    |
-| **Default severity**                | Info                                                                   |
+| **Default severity**                | Warning starting with 4.0.0, Info before                               |
 | **Introduced in version**           | 3.7.0                                                                  |
 | **Is there a code fix**             | Yes                                                                    |
 
diff --git a/docs/core/testing/mstest-analyzers/mstest0039.md b/docs/core/testing/mstest-analyzers/mstest0039.md
@@ -24,6 +24,9 @@ ms.author: ygerges
 | **Introduced in version**           | 3.8.0                                                                  |
 | **Is there a code fix**             | Yes                                                                    |
 
+> [NOTE]
+> This analyzer is no longer relevant for MSTest 4 as the old assertion APIs were removed.
+
 ## Cause
 
 The use of `Assert.ThrowsException` or `Assert.ThrowsExceptionAsync`, which are no longer recommended.
diff --git a/docs/core/testing/mstest-analyzers/mstest0045.md b/docs/core/testing/mstest-analyzers/mstest0045.md
@@ -13,16 +13,16 @@ ms.author: amauryleve
 ---
 # MSTEST0045: Use cooperative cancellation for timeout
 
-| Property                            | Value                                                                                    |
-|-------------------------------------|------------------------------------------------------------------------------------------|
-| **Rule ID**                         | MSTEST0045                                                                               |
-| **Title**                           | Use cooperative cancellation for timeout                                                 |
-| **Category**                        | Design                                                                                   |
-| **Fix is breaking or non-breaking** | Non-breaking                                                                             |
-| **Enabled by default**              | Yes                                                                                      |
-| **Default severity**                | Info                                                                                     |
-| **Introduced in version**           | 3.10.0                                                                                   |
-| **Is there a code fix**             | Yes                                                                                       |
+| Property                            | Value                                    |
+|-------------------------------------|------------------------------------------|
+| **Rule ID**                         | MSTEST0045                               |
+| **Title**                           | Use cooperative cancellation for timeout |
+| **Category**                        | Usage                                    |
+| **Fix is breaking or non-breaking** | Non-breaking                             |
+| **Enabled by default**              | Yes                                      |
+| **Default severity**                | Warning starting with 4.0.0, Info before |
+| **Introduced in version**           | 3.10.0                                   |
+| **Is there a code fix**             | Yes                                      |
 
 ## Cause
 
diff --git a/docs/core/testing/mstest-analyzers/mstest0053.md b/docs/core/testing/mstest-analyzers/mstest0053.md
@@ -25,6 +25,9 @@ ai-usage: ai-generated
 | **Introduced in version**           | 3.11.0                                                                                   |
 | **Is there a code fix**             | Yes                                                                                      |
 
+> [NOTE]
+> This analyzer is no longer relevant for MSTest 4 as the assertion APIs with format parameters were removed.
+
 ## Cause
 
 An assertion method call uses the `message` and `parameters` arguments for string formatting instead of using string interpolation.
diff --git a/docs/core/testing/unit-testing-mstest-migration-v3-v4.md b/docs/core/testing/unit-testing-mstest-migration-v3-v4.md
@@ -8,7 +8,7 @@ ms.date: 07/22/2025
 
 # Migrate from MSTest v3 to MSTest v4
 
-The preview version MSTest v4 is now available. This migration guide explores what's changed in MSTest v4 and how you can migrate to this version.
+The stable version MSTest v4 is now available. This migration guide explores what's changed in MSTest v4 and how you can migrate to this version.
 
 > [!NOTE]
 > Generally speaking, MSTest v4 isn't binary compatible with MSTest v3. Any library compiled against v3 must be recompiled against v4.
@@ -172,6 +172,23 @@ The property was added on individual data sources like `DataRowAttribute` and `D
 
 The `ConditionBaseAttribute.ShouldRun` property was renamed to `IsConditionMet`. That makes it clearer that `ConditionMode` shouldn't be used in the implementation.
 
+### Multiple analyzers are updated to be warning by default
+
+The default severity of the following analyzers changed from Info to Warning:
+
+- [MSTEST0001](./mstest-analyzers/mstest0001.md)
+- [MSTEST0007](./mstest-analyzers/mstest0007.md)
+- [MSTEST0017](./mstest-analyzers/mstest0017.md)
+- [MSTEST0023](./mstest-analyzers/mstest0023.md)
+- [MSTEST0024](./mstest-analyzers/mstest0024.md)
+- [MSTEST0025](./mstest-analyzers/mstest0025.md)
+- [MSTEST0030](./mstest-analyzers/mstest0030.md)
+- [MSTEST0031](./mstest-analyzers/mstest0031.md)
+- [MSTEST0032](./mstest-analyzers/mstest0032.md)
+- [MSTEST0035](./mstest-analyzers/mstest0035.md)
+- [MSTEST0037](./mstest-analyzers/mstest0037.md)
+- [MSTEST0045](./mstest-analyzers/mstest0045.md)
+
 ## Behavior breaking changes
 
 These are breaking changes that might affect the behavior at run time.
