[Findit] Modify tryjob pipelines to trigger try jobs for test failure.

appengine/findit/waterfall/test/schedule_try_job_pipeline_test.py

Index: appengine/findit/waterfall/test/schedule_try_job_pipeline_test.py
diff --git a/appengine/findit/waterfall/test/schedule_try_job_pipeline_test.py b/appengine/findit/waterfall/test/schedule_try_job_pipeline_test.py
index 1a11afcb14961c8ed5510fd47b5cb10180a89312..37da4178306d0da06ce217963cf1e95dfe34da24 100644
--- a/appengine/findit/waterfall/test/schedule_try_job_pipeline_test.py
+++ b/appengine/findit/waterfall/test/schedule_try_job_pipeline_test.py
@@ -34,11 +34,10 @@ class ScheduleTryjobPipelineTest(testing.AppengineTestCase):
   def testGetBuildPropertiesWithCompileTargets(self):
     master_name = 'm'
     builder_name = 'b'
-    recipe = 'a/b/recipe'
     compile_targets = ['a.exe']
 
     expected_properties = {
-        'recipe': recipe,
+        'recipe': 'findit/chromium/compile',
         'good_revision': 1,
         'bad_revision': 2,
         'target_mastername': master_name,
@@ -47,7 +46,26 @@ class ScheduleTryjobPipelineTest(testing.AppengineTestCase):
     }
     try_job_pipeline = ScheduleTryJobPipeline()
     properties = try_job_pipeline._getBuildProperties(
-        recipe, master_name, builder_name, 1, 2, compile_targets)
+        master_name, builder_name, 1, 2, 'compile', compile_targets, None)
+
+    self.assertEqual(properties, expected_properties)
+
+  def testGetBuildPropertiesForTestFailure(self):
+    master_name = 'm'
+    builder_name = 'b'
+    targeted_tests = {'a':[]}
+
+    expected_properties = {
+        'recipe': 'findit/chromium/test',
+        'good_revision': 1,
+        'bad_revision': 2,
+        'target_mastername': master_name,
+        'target_testername': 'b',
+        'tests': targeted_tests
+    }
+    try_job_pipeline = ScheduleTryJobPipeline()
+    properties = try_job_pipeline._getBuildProperties(
+        master_name, builder_name, 1, 2, 'test', None, targeted_tests)
 
     self.assertEqual(properties, expected_properties)
 
@@ -75,8 +93,39 @@ class ScheduleTryjobPipelineTest(testing.AppengineTestCase):
     try_job_pipeline = ScheduleTryJobPipeline()
     try_job_id = try_job_pipeline.run(
         master_name, builder_name, build_number, good_revision, bad_revision,
-        [])
+        'compile', None)
 
     try_job = WfTryJob.Get(master_name, builder_name, build_number)
     self.assertEqual('1', try_job_id)
     self.assertEqual('1', try_job.compile_results[-1]['try_job_id'])
+
+
+  def testSuccessfullyScheduleNewTryJobForTest(self):
+    master_name = 'm'
+    builder_name = 'b'
+    build_number = 223
+    good_revision = 'rev1'
+    bad_revision = 'rev2'
+
+    responses = [
+        {
+            'build': {
+                'id': '1',
+                'url': 'url',
+                'status': 'SCHEDULED',
+            }
+        }
+    ]
+    self._Mock_GetTrybotForWaterfallBuilder(master_name, builder_name)
+    self._Mock_TriggerTryJobs(responses)
+
+    WfTryJob.Create(master_name, builder_name, build_number).put()
+
+    try_job_pipeline = ScheduleTryJobPipeline()
+    try_job_id = try_job_pipeline.run(
+        master_name, builder_name, build_number, good_revision, bad_revision,
+        'test', None)
+
+    try_job = WfTryJob.Get(master_name, builder_name, build_number)
+    self.assertEqual('1', try_job_id)
+    self.assertEqual('1', try_job.test_results[-1]['try_job_id'])