Investigate potential false positive for MSTEST0016: Test class should have at least one test method

[Youssef1313]

[TestClass]
public class MyTestClass : MyTestClassBase // False positive?
{
    protected override async Task<Something> GetSomething()
    {
        // Customizes behavior for MyTestClass
    }
}

public abstract class MyTestClassBase
{
    protected abstract Task<Something> GetSomething();

    [TestMethod]
    public async Task MyTestMethod()
    {
        var something = await GetSomething();
        // ...
    }
}

AB#2335052

[Evangelink]

You should not even need the abstract. MyTestMethod is a test method so it should be considered in the scope of MyTestClass.

[Youssef1313]

@Evangelink The test cannot be run in the context of "MyTestClassBase" because GetSomething is intended to be provided by subtypes. We need the method be abstract, hence the class as well. But do you agree it's a false positive?

[Youssef1313]

Hmm, that actually was supposed to be fixed by #2715. I will need to double check again.

[Evangelink]

@Youssef1313 Is the issue on base class or sub class?

[Youssef1313]

The diagnostic I got was on the subclass. But note that I haven't checked with exactly the code in the issue. I wrote it directly on GitHub based on an original repro, but the code is structured the same. So I need to check if maybe there is something I'm missing.