Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(366)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: appengine/findit/crash/culprit.py

Issue 2449853012: [Predator] Fix bug in min_distance after refactor and add back skip added/deleted deps. (Closed)
Patch Set: Add culprit test. Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « appengine/findit/crash/changelist_classifier.py ('k') | appengine/findit/crash/scorers/min_distance.py » ('j') | appengine/findit/crash/scorers/test/aggregated_scorer_test.py » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: appengine/findit/crash/culprit.py
diff --git a/appengine/findit/crash/culprit.py b/appengine/findit/crash/culprit.py
index b57e0fd098e8a63ce6170315ea7da3cd68a94c6a..6582f2b648246e5d4537199f229ba67d29a4a985 100644
--- a/appengine/findit/crash/culprit.py
+++ b/appengine/findit/crash/culprit.py
@@ -4,14 +4,11 @@
from collections import namedtuple
-# TODO(http://crbug.com/659346): We do call this code from various
-# unittests, just not from culprit_test.py; so we need to add some extra
-# unittests there.
+
class Culprit(namedtuple('Culprit',
- ['project', 'components', 'cls', 'regression_range', 'algorithm']
- )): # pragma: no cover
+ ['project', 'components', 'cls', 'regression_range', 'algorithm'])):
"""The result of successfully identifying the culprit of a crash report.
-
+
Args:
project (str): the most-suspected project
components (list of str): the suspected crbug components.
« no previous file with comments | « appengine/findit/crash/changelist_classifier.py ('k') | appengine/findit/crash/scorers/min_distance.py » ('j') | appengine/findit/crash/scorers/test/aggregated_scorer_test.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698