fix: Remove obsolete PH2035 analyzer (TestMethodsMustHaveTheCorrectNumberOfArguments) (#861)

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: bcollamore <57269455+bcollamore@users.noreply.github.com>

Author: Copilot

Documentation/Diagnostics/PH2035.md

@@ -34,7 +34,6 @@ public enum DiagnosticId
 		AvoidTryParseWithoutCulture = 2031,
 		AvoidEmptyTypeInitializer = 2032,
 		TestMethodsMustBeInTestClass = 2034,
-		TestMethodsMustHaveTheCorrectNumberOfArguments = 2035,
 		TestMethodsMustBePublic = 2036,
 		TestMethodsMustHaveUniqueNames = 2037,
 		TestClassesMustBePublic = 2038,

Philips.CodeAnalysis.Common/DiagnosticId.cs

@@ -16,8 +16,7 @@
 | [PH2017](../Documentation/Diagnostics/PH2017.md)  | Avoid ClassInitialize attribute                      | This attribute is unnecessary in the C# programming language. They circumvent TestTimeouts, allowing for slower Test Runs.  They can contribute to non-deterministic test execution order, resulting complications while debugging. |
 | [PH2018](../Documentation/Diagnostics/PH2018.md)  | Avoid ClassCleanup attribute                         | This attribute is unnecessary in the C# programming language. They circumvent TestTimeouts, allowing for slower Test Runs.  They can contribute to non-deterministic test execution order, resulting complications while debugging. |
 | [PH2019](../Documentation/Diagnostics/PH2019.md)  | Avoid TestCleanup attribute                          | This attribute is unnecessary in the C# programming language. They circumvent TestTimeouts, allowing for slower Test Runs.  They can contribute to non-deterministic test execution order, resulting complications while debugging. |
-| [PH2034](../Documentation/Diagnostics/PH2034.md)  | TestMethod requires TestClass                        | TestMethods not inside a TestClass are not executed.  They are dead code. |
-| [PH2035](../Documentation/Diagnostics/PH2035.md)  | DataTestMethod requires correct parameter count      | DataTestMethods must have the same number of parameters of the DataRows, TestMethods should have no arguments. |
+| [PH2034](../Documentation/Diagnostics/PH2034.md)  | TestMethod requires TestClass                        | TestMethods not inside a TestClass are not executed.  They are dead code. |
 | [PH2036](../Documentation/Diagnostics/PH2036.md)  | TestMethod is public                                 | TestMethods must be public.                                  |
 | [PH2037](../Documentation/Diagnostics/PH2037.md)  | TestMethods and DataTestMethods require unique names | TestMethods and DataTestMethods require unique names.        |
 | [PH2038](../Documentation/Diagnostics/PH2038.md)  | TestClass is public                                  | TestClass must be public.                                    |