-
Notifications
You must be signed in to change notification settings - Fork 289
Open
Labels
Area: AnalyzersArea: MSTestIssues with MSTest that are not specific to more refined area (e.g. analyzers or assertions)Issues with MSTest that are not specific to more refined area (e.g. analyzers or assertions)
Description
Today, the following is allowed:
[TestMethod]
[DataRow("Hello", "World")]
public void TestMethod1(out string s, ref string s2)
{
s = "";
}I think we should:
- Fail at run time when we encounter such a test method.
- Update the analyzer for test method validity to also produce a compile-time warning.
It's technically a breaking change, but I think it's acceptable and is unlikely that people are doing this today.
Metadata
Metadata
Assignees
Labels
Area: AnalyzersArea: MSTestIssues with MSTest that are not specific to more refined area (e.g. analyzers or assertions)Issues with MSTest that are not specific to more refined area (e.g. analyzers or assertions)