[Predator] Run predator.

appengine/findit/crash/culprit.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.
 
 from collections import namedtuple
 
 
 class Culprit(namedtuple('Culprit',
     ['project', 'components', 'cls', 'regression_range', 'algorithm'])):
   """The result of successfully identifying the culprit of a crash report.
@@ skipping 80 matching lines @@
     """
     # TODO(wrengr): will this auto-dropping of unnecessary fields cause
     # any issues for JSON serialization?
     result = {}
     result['found'] = (
         bool(self.project) or
         bool(self.components) or
         bool(self.cls) or
         bool(self.regression_range))
     if self.regression_range:
       result['regression_range'] = self.regression_range

[wrengr, 2016/11/01 22:07:52]

We should copy the self.regression_range into a list, since it may be (i.e.,
should be) a tuple but comparing tuples to lists fails in the unittests.

[Sharu Jiang, 2016/11/11 23:10:26]

Done in rebase.

-    if self.project is not None:
+    if self.project:
       result['suspected_project'] = self.project
-    if self.components is not None:
+    if self.components:
       result['suspected_components'] = self.components
-    if self.cls is not None:
+    if self.cls:
       result['suspected_cls'] = [cl.ToDict() for cl in self.cls]
 
     tags = {
       'found_suspects': bool(self.cls),
       'has_regression_range': bool(self.regression_range),
       'found_project': bool(self.project),
       'found_components': bool(self.components),
       'solution': self.algorithm,
     }
 
     return result, tags