Fix or add more details about some MSTest rules

Author: Evangelink

docs/core/testing/mstest-analyzers/mstest0008.md

@@ -40,6 +40,13 @@ Methods marked with `[TestInitialize]` should follow the following layout to be
 - it should not take any parameter
 - return type should be `void`, `Task` or `ValueTask`
 
+The type declaring these methods should also respect the following rules:
+
+- the type should be a class
+- the class should be `public` or `internal` (if the test project is using the `[DiscoverInternals]` attribute)
+- the class should not be `static`
+- if the class is sealed, it should be marked with `[TestClass]` (or a derived attribute)
+
 ## How to fix violations
 
 Ensure that the method matches the layout described above.

docs/core/testing/mstest-analyzers/mstest0009.md

@@ -40,6 +40,13 @@ Methods marked with `[TestCleanup]` should follow the following layout to be val
 - it should not take any parameter
 - return type should be `void`, `Task` or `ValueTask`
 
+The type declaring these methods should also respect the following rules:
+
+- the type should be a class
+- the class should be `public` or `internal` (if the test project is using the `[DiscoverInternals]` attribute)
+- the class should not be `static`
+- if the class is sealed, it should be marked with `[TestClass]` (or a derived attribute)
+
 ## How to fix violations
 
 Ensure that the method matches the layout described above.

docs/core/testing/mstest-analyzers/mstest0010.md

@@ -40,6 +40,14 @@ Methods marked with `[ClassInitialize]` should follow the following layout to be
 - it should take one parameter of type `TestContext`
 - return type should be `void`, `Task` or `ValueTask`
 
+The type declaring these methods should also respect the following rules:
+
+- the type should be a class
+- the class should be `public` or `internal` (if the test project is using the `[DiscoverInternals]` attribute)
+- the class should not be `static`
+- the class should not be generic
+- if the class is sealed, it should be marked with `[TestClass]` (or a derived attribute)
+
 ## How to fix violations
 
 Ensure that the method matches the layout described above.

docs/core/testing/mstest-analyzers/mstest0011.md

@@ -40,6 +40,14 @@ Methods marked with `[ClassCleanup]` should follow the following layout to be va
 - it should not take any parameter
 - return type should be `void`, `Task` or `ValueTask`
 
+The type declaring these methods should also respect the following rules:
+
+- the type should be a class
+- the class should be `public` or `internal` (if the test project is using the `[DiscoverInternals]` attribute)
+- the class should not be `static`
+- the class should not be generic
+- if the class is sealed, it should be marked with `[TestClass]` (or a derived attribute)
+
 ## How to fix violations
 
 Ensure that the method matches the layout described above.

docs/core/testing/mstest-analyzers/mstest0012.md

@@ -40,6 +40,14 @@ Methods marked with `[AssemblyInitialize]` should follow the following layout to
 - it should take one parameter of type `TestContext`
 - return type should be `void`, `Task` or `ValueTask`
 
+The type declaring these methods should also respect the following rules:
+
+- the type should be a class
+- the class should be `public` or `internal` (if the test project is using the `[DiscoverInternals]` attribute)
+- the class should not be `static`
+- the class should not be generic
+- if the class is sealed, it should be marked with `[TestClass]` (or a derived attribute)
+
 ## How to fix violations
 
 Ensure that the method matches the layout described above.

docs/core/testing/mstest-analyzers/mstest0013.md

@@ -40,6 +40,14 @@ Methods marked with `[AssemblyCleanup]` should follow the following layout to be
 - it should not take any parameter
 - return type should be `void`, `Task` or `ValueTask`
 
+The type declaring these methods should also respect the following rules:
+
+- the type should be a class
+- the class should be `public` or `internal` (if the test project is using the `[DiscoverInternals]` attribute)
+- the class should not be `static`
+- the class should not be generic
+- if the class is sealed, it should be marked with `[TestClass]` (or a derived attribute)
+
 ## How to fix violations
 
 Ensure that the method matches the layout described above.

docs/core/testing/mstest-analyzers/mstest0029.md

@@ -25,11 +25,11 @@ ms.author: enjieid
 
 ## Cause
 
-A Public method should be a test method.
+A `public` method should be a test method.
 
 ## Rule description
 
-A Public method should be a test method (marked with `[TestMethod]`).
+A `public` method of a class marked with `[TestClass]` should be a test method (marked with `[TestMethod]`). The rule ignores methods that are marked with `[TestInitialize]`, or `[TestCleanup]` attributes.
 
 ## How to fix violations
 

docs/core/testing/mstest-analyzers/mstest0030.md

@@ -25,7 +25,7 @@ ms.author: enjieid
 
 ## Cause
 
-Type contaning `[TestMethod]` should be marked with `[TestClass]`, otherwise the test method will be silently ignored.
+Type containing `[TestMethod]` should be marked with `[TestClass]`, otherwise the test method will be silently ignored.
 
 ## Rule description