Analyzer: DynamicData should be valid

[Evangelink]

Summary

Title: DynamicData should be valid
Category: Usage
Description:
Default severity: Warning

Please refer to MSTEST0003: Test methods should have valid layout.

Non exhaustive list of constraints:

• public (except if discover internals)
• static
• member should match kind declared on the attribute (property or method)
• member should exist
• member return type should be valid (IEnumerable<object[]> or IEnumerable<(Value)Tuple<...>>)
• if member is property it should be gettable
• if member is method, it should not have parameters

AB#1950780

[nohwnd]

Would be perfect if this could compare the count and type of parameters when IEnumerable is used.

[Evangelink]

Thanks! I definitely missed these cases here but I described them in the global analyzers ticket.

[testplatform-bot]

✅ Successfully linked to Azure Boards work item(s):

• User Story 1950780

[Evangelink]

@engyebrahim As discussed, postponing to milestone 3.4

[xcaptain]

I also have a issue with DynamicData in vscode test explorer. same cause here?

@Evangelink

[nohwnd]

Moved the above issue to a separate one. This is a work item to create the analyzer, not an issue report.