Unit test display name issue fixed. (#795)

Fixes #793 Partially reverts changes introduced in #766
microsoft · Mar 10, 2021 · c3a4c9f · c3a4c9f
1 parent 5dafe82
commit c3a4c9f
Showing 1 changed file with 11 additions and 6 deletions.
diff --git a/src/Adapter/MSTest.CoreAdapter/ObjectModel/UnitTestElement.cs b/src/Adapter/MSTest.CoreAdapter/ObjectModel/UnitTestElement.cs
@@ -109,9 +109,11 @@ public UnitTestElement(TestMethod testMethod)
         /// <returns> An instance of <see cref="TestCase"/>. </returns>
         internal TestCase ToTestCase()
         {
-            string fullName = this.TestMethod.HasManagedMethodAndTypeProperties
-                            ? string.Format(CultureInfo.InvariantCulture, "{0}.{1}", this.TestMethod.ManagedTypeName, this.TestMethod.ManagedMethodName)
-                            : string.Format(CultureInfo.InvariantCulture, "{0}.{1}", this.TestMethod.FullClassName, this.TestMethod.Name);
+            // This causes compatibility problems with older runners.
+            // string fullName = this.TestMethod.HasManagedMethodAndTypeProperties
+            //                 ? string.Format(CultureInfo.InvariantCulture, "{0}.{1}", this.TestMethod.ManagedTypeName, this.TestMethod.ManagedMethodName)
+            //                 : string.Format(CultureInfo.InvariantCulture, "{0}.{1}", this.TestMethod.FullClassName, this.TestMethod.Name);
+            var fullName = string.Format(CultureInfo.InvariantCulture, "{0}.{1}", this.TestMethod.FullClassName, this.TestMethod.Name);
 
             TestCase testCase = new TestCase(fullName, TestAdapter.Constants.ExecutorUri, this.TestMethod.AssemblyName);
             testCase.DisplayName = this.GetDisplayName();
@@ -195,9 +197,12 @@ private string GetDisplayName()
         {
             if (string.IsNullOrWhiteSpace(this.DisplayName))
             {
-                return string.IsNullOrWhiteSpace(this.TestMethod.ManagedMethodName)
-                     ? this.TestMethod.Name
-                     : this.TestMethod.ManagedMethodName;
+                return this.TestMethod.Name;
+
+                // This causes compatibility problems with older runners.
+                // return string.IsNullOrWhiteSpace(this.TestMethod.ManagedMethodName)
+                //      ? this.TestMethod.Name
+                //      : this.TestMethod.ManagedMethodName;
             }
             else
             {