[Findit] don't included skipped or unknown tests in swarming tasks into failed tests.

appengine/findit/handlers/test/handlers_util_test.py

Index: appengine/findit/handlers/test/handlers_util_test.py
diff --git a/appengine/findit/handlers/test/handlers_util_test.py b/appengine/findit/handlers/test/handlers_util_test.py
index 25ad562e2b2985cfb6cb9e410c85b62815f61056..2e692c511ba2d930a527957928f900f4fb1b39bb 100644
--- a/appengine/findit/handlers/test/handlers_util_test.py
+++ b/appengine/findit/handlers/test/handlers_util_test.py
@@ -134,8 +134,9 @@ class HandlersUtilResultTest(wf_testcase.WaterfallTestCase):
         self.master_name, self.builder_name, self.build_number)
     analysis.failure_result_map = {
         'step1 on platform': {
-            'test1': '%s/%s/%s' % (self.master_name, self.builder_name, 120),
-            'test2': '%s/%s/%s' % (
+            'PRE_test1': '%s/%s/%s' % (
+                self.master_name, self.builder_name, 120),
+            'PRE_PRE_test2': '%s/%s/%s' % (
                 self.master_name, self.builder_name, self.build_number),
             'test3': '%s/%s/%s' % (
                 self.master_name, self.builder_name, self.build_number),
@@ -159,6 +160,10 @@ class HandlersUtilResultTest(wf_testcase.WaterfallTestCase):
     task0.tests_statuses = {
         'test1': {
             'total_run': 2,
+            'SKIPPED': 2
+        },
+        'PRE_test1': {
+            'total_run': 2,
             'FAILURE': 2
         }
     }
@@ -173,10 +178,18 @@ class HandlersUtilResultTest(wf_testcase.WaterfallTestCase):
         'tests': ['test2', 'test3', 'test4']
     }
     task1.tests_statuses = {
-        'test2': {
+        'PRE_PRE_test2': {
             'total_run': 2,
             'FAILURE': 2
         },
+        'PRE_test2': {
+            'total_run': 2,
+            'SKIPPED': 2
+        },
+        'test2': {
+            'total_run': 2,
+            'SKIPPED': 2
+        },
         'test3': {
             'total_run': 4,
             'SUCCESS': 2,
@@ -206,8 +219,8 @@ class HandlersUtilResultTest(wf_testcase.WaterfallTestCase):
                         'task_url': ('https://chromium-swarm.appspot.com/user'
                                      '/task/task1')
                     },
-                    'all_tests': ['test2', 'test3', 'test4'],
-                    'reliable_tests': ['test2'],
+                    'all_tests': ['PRE_PRE_test2', 'test3', 'test4'],
+                    'reliable_tests': ['PRE_PRE_test2'],
                     'flaky_tests': ['test3', 'test4'],
                     'ref_name': 'step1'
                 },
@@ -218,8 +231,8 @@ class HandlersUtilResultTest(wf_testcase.WaterfallTestCase):
                         'task_url': ('https://chromium-swarm.appspot.com/user/'
                                      'task/task0')
                     },
-                    'all_tests': ['test1'],
-                    'reliable_tests': ['test1'],
+                    'all_tests': ['PRE_test1'],
+                    'reliable_tests': ['PRE_test1'],
                     'flaky_tests': [],
                     'ref_name': 'step1'
                 }
@@ -231,13 +244,13 @@ class HandlersUtilResultTest(wf_testcase.WaterfallTestCase):
                     'task_info': {
                         'status': analysis_status.PENDING
                     },
-                    'all_tests': ['test1'],
+                    'all_tests': ['PRE_test1'],
                     'ref_name': 'step2'
                 }
             }
         }
     }
-    self.assertEqual(expected_data, data)
+    self.assertEqual(sorted(expected_data), sorted(data))
 
   def testGetAllTryJobResultsNoAnalysis(self):
     data = handlers_util.GetAllTryJobResults(