Refactor and cleanup how we store type information for test methods

[Youssef1313]

We have our TestMethod model. It contains multiple properties related to type information:

• FullClassName (the semantic type)
• ManagedTypeName (the semantic type too!)
• DeclaringClassFullName (the syntactic type)

Semantic and syntactic type are different when inheritance is involved.

[TestClass]
public abstract class MyBaseTestClass
{
    [TestMethod]
    public void M() { }
}

[TestClass]
public class ConcreteTestClass : MyBaseTestClass
{
    // this class implicitly has the test method 'M'.
    // When it's run in context of this class, its semantic type is ConcreteTestClass, but the syntactic type is MyBaseTestClass.
}

We should find ways to reduce the information we store. Ideally, only have one property for the semantic type, and not need the syntactic type at all.

[nohwnd]

What is the benefit of doing this? More speed? Less memory consumed? Apis get more obvious?

[Youssef1313]

Potentially all of that.