[Findit] Send targeted_tests to try-job in build_parameter instend of build_property (Findit side).

appengine/findit/common/waterfall/buildbucket_client.py

 # Copyright 2015 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.
 
 """"Serves as a client for selected APIs in Buildbucket."""
 
 import collections
 import json
 import logging
 
@@ skipping 16 matching lines @@
   name, while Findit uses shortened master name (tryserver.chromium.linux).
   """
   prefix = 'master.'
   if master_name.startswith(prefix):
     return master_name
   return '%s%s' % (prefix, master_name)
 
 
 class TryJob(collections.namedtuple(
     'TryJobNamedTuple',
-    ('master_name', 'builder_name', 'revision', 'properties', 'tags'))):
+    ('master_name', 'builder_name', 'revision', 'properties', 'tags', 'tests'))

[stgao, 2016/05/20 05:15:48]

How about making this generic enough for other change too? Now it is the list of
tests, but later we might have to move other properties or add some new ones
too.

+):
   """Represents a try-job to be triggered through Buildbucket.
 
   Tag for "user_agent" should not be set, as it will be added automatically.
   """
 
   def ToBuildbucketRequest(self):
     parameters_json = {
         'builder_name': self.builder_name,
         'properties': self.properties,
     }
+    if self.tests:
+      parameters_json['tests'] = self.tests
     if self.revision:
       parameters_json['changes'] = [
           {
               'author': {
                   'email': _ROLE_EMAIL,
               },
               'revision': self.revision,
           },
       ]
 
@@ skipping 126 matching lines @@
     else:
       error_content = {
           'error': {
               'reason': status_code,
               'message': content
           }
       }
       json_results.append(error_content)
 
   return _ConvertFuturesToResults(json_results)