bugfix: filtered test cases were run, even if their test results were…

…n't registered (#36)
csoltenborn · Mar 15, 2016 · 4ce65bf · 4ce65bf
1 parent a07fec8
commit 4ce65bf
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/GoogleTestAdapter/TestAdapter/TestExecutor.cs b/GoogleTestAdapter/TestAdapter/TestExecutor.cs
@@ -42,12 +42,12 @@ public void RunTests(IEnumerable<string> executables, IRunContext runContext, IF
 
                 ISet<string> allTraitNames = GetAllTraitNames(allTestCasesInExecutables);
                 TestCaseFilter filter = new TestCaseFilter(runContext, allTraitNames, TestEnvironment);
-                List<TestCase> vsTestCases =
+                List<TestCase> vsTestCasesToRun =
                     filter.Filter(allTestCasesInExecutables.Select(DataConversionExtensions.ToVsTestCase)).ToList();
-                allTestCasesInExecutables =
-                    allTestCasesInExecutables.Where(tc => vsTestCases.Any(vtc => tc.FullyQualifiedName == vtc.FullyQualifiedName)).ToList();
+                IEnumerable<Model.TestCase> testCasesToRun =
+                    allTestCasesInExecutables.Where(tc => vsTestCasesToRun.Any(vtc => tc.FullyQualifiedName == vtc.FullyQualifiedName)).ToList();
 
-                DoRunTests(allTestCasesInExecutables, allTestCasesInExecutables, runContext, frameworkHandle);
+                DoRunTests(allTestCasesInExecutables, testCasesToRun, runContext, frameworkHandle);
             }
             catch (Exception e)
             {