Reverting back to FQN and and removing namespace from FindTestCase() if exists.

Previously using tc.DisplayName causes a regression where paramaterized tests were not run. Therefore, we are switching back to FQN, but removing the namespace if it exists in the test case FQN, otherwise the names will not match since the console output name does not include namespaces.

Author: DavidARaygoza

GoogleTestAdapter/Core/Runners/SequentialTestRunner.cs

@@ -63,7 +63,7 @@ public void RunTests(IEnumerable<TestCase> testCasesToRun, string baseDir,
 
                         foreach (var testCase in groupedTestCases[executable])
                         {
-                            var key = Path.GetFullPath(testCase.Source) + ":" + testCase.DisplayName;
+                            var key = Path.GetFullPath(testCase.Source) + ":" + testCase.FullyQualifiedName;
                             ITestPropertySettings settings;
                             // Tests with default settings are treated as not having settings and can be run together
                             if (_settings.TestPropertySettingsContainer.TryGetSettings(key, out settings)

GoogleTestAdapter/Core/TestResults/StandardOutputTestResultParser.cs

@@ -59,8 +59,8 @@ private TestResult CreateTestResult(int indexOfTestcase)
             int currentLineIndex = indexOfTestcase;
 
             string line = _consoleOutput[currentLineIndex++];
-            string testDisplayName = RemovePrefix(line).Trim();
-            TestCase testCase = FindTestcase(testDisplayName);
+            string qualifiedTestname = RemovePrefix(line).Trim();
+            TestCase testCase = FindTestcase(qualifiedTestname);
             if (testCase == null)
             {
                 _logger.DebugWarning(String.Format(Resources.NoKnownTestCaseMessage, line));
@@ -189,14 +189,31 @@ private int FindIndexOfNextTestcase(int currentIndex)
             return -1;
         }
 
-        private TestCase FindTestcase(string testDisplayName)
+        private TestCase FindTestcase(string qualifiedTestname)
         {
-            return FindTestcase(testDisplayName, _testCasesRun);
+            return FindTestcase(qualifiedTestname, _testCasesRun);
         }
 
-        public static TestCase FindTestcase(string testDisplayName, IList<TestCase> testCasesRun)
+        public static TestCase FindTestcase(string qualifiedTestname, IList<TestCase> testCasesRun)
         {
-            return testCasesRun.SingleOrDefault(tc => tc.DisplayName == testDisplayName);
+            foreach (TestCase tc in testCasesRun)
+            {
+                // If namespace exists then remove it so we can compare the test display names.
+                //  Using just tc.DisplayName does not work for paramaterized test cases.
+                string fullyQualifiedName = tc.FullyQualifiedName;
+                int frequency = fullyQualifiedName.Where(x => (x == '.')).Count();
+                if (frequency > 1)
+                {
+                    fullyQualifiedName = fullyQualifiedName.Substring(fullyQualifiedName.IndexOf('.') + 1);
+                }
+
+                if (fullyQualifiedName == qualifiedTestname)
+                {
+                    return tc;
+                }
+            }
+
+            return null;
         }
 
         public static bool IsRunLine(string line)