[Findit] Strip platform from step_name before triggering try job.

appengine/findit/waterfall/test/process_swarming_task_result_pipeline_test.py

 # Copyright 2016 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import datetime
 
 from testing_utils import testing
 
 from model import wf_analysis_status
 from model.wf_swarming_task import WfSwarmingTask
@@ skipping 107 matching lines @@
 }
 
 
 class ProcessSwarmingTaskResultPipelineTest(testing.AppengineTestCase):
 
   def _MockedGetSwarmingTaskResultById(self, task_id, _):
     swarming_task_results = {
         'task_id1': {
             'state': 'COMPLETED',
             'tags': [
-                'priority:25'
+                'priority:25',
+                'ref_name:abc_tests'
             ],
             'outputs_ref': {
                 'isolatedserver': _ISOLATED_SERVER,
                 'namespace': 'default-gzip',
                 'isolated': 'shard1_isolated'
             },
             'created_ts': '2016-02-10T18:32:06.538220',
             'started_ts': '2016-02-10T18:32:09.090550',
             'completed_ts': '2016-02-10T18:33:09'
         },
         'task_id2': {
             'state': 'TIMED_OUT',
             'outputs_ref': None
         }
     }
 
     mocked_result = swarming_task_results[task_id]
     return mocked_result
 
   def setUp(self):
     super(ProcessSwarmingTaskResultPipelineTest, self).setUp()
     self.master_name = 'm'
     self.builder_name = 'b'
     self.build_number = 121
-    self.step_name = 'abc_tests'
+    self.step_name = 'abc_tests on platform'
     def _MockGetSwarmingSettings():
       return _MOCK_SWARMING_SETTINGS
     self.mock(waterfall_config, 'GetSwarmingSettings', _MockGetSwarmingSettings)
     self.mock(swarming_util, 'GetSwarmingTaskResultById',
               self._MockedGetSwarmingTaskResultById)
 
   def testCheckTestsRunStatusesNoOutPutJson(self):
     tests_statuses = (
         process_swarming_task_result_pipeline._CheckTestsRunStatuses(None))
     self.assertEqual({}, tests_statuses)
@@ skipping 11 matching lines @@
     task_id = 'task_id1'
 
     self.mock(swarming_util, 'GetSwarmingTaskFailureLog',
               self._MockedGetSwarmingTaskFailureLog)
 
     WfSwarmingTask.Create(
         self.master_name, self.builder_name,
         self.build_number, self.step_name).put()
 
     pipeline = ProcessSwarmingTaskResultPipeline()
-    step_name, classified_tests = pipeline.run(
+    step_name, task_info = pipeline.run(
         self.master_name, self.builder_name,
         self.build_number, self.step_name, task_id)
 
-    self.assertEqual(_EXPECTED_CLASSIFIED_TESTS, classified_tests)
+    self.assertEqual(self.step_name, step_name)
+    self.assertEqual('abc_tests', task_info[0])
+    self.assertEqual(_EXPECTED_CLASSIFIED_TESTS, task_info[1])
 
     task = WfSwarmingTask.Get(
         self.master_name, self.builder_name, self.build_number, self.step_name)
 
-    self.assertEqual(self.step_name, step_name)
     self.assertEqual(wf_analysis_status.ANALYZED, task.status)
     self.assertEqual(_EXPECTED_TESTS_STATUESE, task.tests_statuses)
     self.assertEqual(_EXPECTED_CLASSIFIED_TESTS, task.classified_tests)
     self.assertEqual(datetime.datetime(2016, 2, 10, 18, 32, 6, 538220),
                      task.created_time)
     self.assertEqual(datetime.datetime(2016, 2, 10, 18, 32, 9, 90550),
                      task.started_time)
     self.assertEqual(datetime.datetime(2016, 2, 10, 18, 33, 9),
                      task.completed_time)
 
 
   def testProcessSwarmingTaskResultPipelineTaskNotRunning(self):
     task_id = 'task_id2'
 
     WfSwarmingTask.Create(
         self.master_name, self.builder_name,
         self.build_number, self.step_name).put()
 
     pipeline = ProcessSwarmingTaskResultPipeline()
-    step_name, classified_tests = pipeline.run(
+    step_name, task_info = pipeline.run(
         self.master_name, self.builder_name,
         self.build_number, self.step_name, task_id)
 
-    self.assertEqual({}, classified_tests)
+    self.assertEqual(self.step_name, step_name)
+    self.assertIsNone(task_info[0])
+    self.assertEqual({}, task_info[1])
 
     task = WfSwarmingTask.Get(
         self.master_name, self.builder_name, self.build_number, self.step_name)
 
     self.assertEqual(wf_analysis_status.ERROR, task.status)
     self.assertEqual({}, task.tests_statuses)
     self.assertEqual({}, task.classified_tests)
-    self.assertEqual(self.step_name, step_name)
 
   def testProcessSwarmingTaskResultPipelineTaskTimeOut(self):
-    task_id = 'task_id2'
+    task_id = 'task_id1'
     old_mock_timeout = _MOCK_SWARMING_SETTINGS['task_timeout_hours']
     _MOCK_SWARMING_SETTINGS['task_timeout_hours'] = -1
 
     WfSwarmingTask.Create(
         self.master_name, self.builder_name,
         self.build_number, self.step_name).put()
 
     pipeline = ProcessSwarmingTaskResultPipeline()
-    step_name, classified_tests = pipeline.run(
+    step_name, task_info = pipeline.run(
         self.master_name, self.builder_name,
         self.build_number, self.step_name, task_id)
 
-    self.assertEqual({}, classified_tests)
+    self.assertEqual(self.step_name, step_name)
+    self.assertEqual('abc_tests', task_info[0])
+    self.assertEqual({}, task_info[1])
 
     task = WfSwarmingTask.Get(
         self.master_name, self.builder_name, self.build_number, self.step_name)
 
     self.assertEqual(wf_analysis_status.ERROR, task.status)
     self.assertEqual({}, task.tests_statuses)
     self.assertEqual({}, task.classified_tests)
-    self.assertEqual(self.step_name, step_name)
 
     _MOCK_SWARMING_SETTINGS['task_timeout_hours'] = old_mock_timeout