Rip out old code for finding reference test for an alert.

dashboard/dashboard/group_report.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.
 
 """Provides the web interface for a set of alerts and their graphs."""
 
 import json
 
 from google.appengine.ext import ndb
 
 from dashboard import alerts
 from dashboard import chart_handler
-from dashboard import list_tests
 from dashboard import update_test_suites
 from dashboard.common import request_handler
 from dashboard.common import utils
 from dashboard.models import anomaly
 from dashboard.models import page_state
 from dashboard.models import stoppage_alert
 
 # This is the max number of alerts to query at once. This is used in cases
 # when we may want to query more many more alerts than actually get displayed.
 _QUERY_LIMIT = 2000
@@ skipping 151 matching lines @@
       bug_id: An integer bug ID.
     """
     anomaly_dicts = alerts.AnomalyDicts(
         [a for a in alert_list if a.key.kind() == 'Anomaly'])
     stoppage_alert_dicts = alerts.StoppageAlertDicts(
         [a for a in alert_list if a.key.kind() == 'StoppageAlert'])
     alert_dicts = anomaly_dicts + stoppage_alert_dicts
 
     values = {
         'alert_list': alert_dicts[:_DISPLAY_LIMIT],
-        # This code for getting the subtests is supposed to be used to sort out
-        # which metrics are "core" vs "non-core". But it's extremely slow, and
-        # also doesn't seem to work very well. Turn it off for now:
-        # https://github.com/catapult-project/catapult/issues/2877
-        #'subtests': _GetSubTestsForAlerts(alert_dicts),
         'bug_id': bug_id,
         'test_suites': update_test_suites.FetchCachedTestSuites(),
         'selected_keys': selected_keys,
     }
     self.GetDynamicVariables(values)
 
     self.response.out.write(json.dumps(values))
 
 
 def _IsInt(x):
   """Returns True if the input can be parsed as an int."""
   try:
     int(x)
     return True
   except ValueError:
     return False
 
 
-def _GetSubTestsForAlerts(alert_list):
-  """Gets subtest dict for list of alerts."""
-  subtests = {}
-  for alert in alert_list:
-    bot_name = alert['master'] + '/' + alert['bot']
-    testsuite = alert['testsuite']
-    if bot_name not in subtests:
-      subtests[bot_name] = {}
-    if testsuite not in subtests[bot_name]:
-      subtests[bot_name][testsuite] = list_tests.GetSubTests(
-          testsuite, [bot_name])
-  return subtests
-
-
 def _GetOverlaps(anomalies, start, end):
   """Gets the minimum range for the list of anomalies.
 
   Args:
     anomalies: The list of anomalies.
     start: The start revision.
     end: The end revision.
 
   Returns:
     A list of anomalies.
   """
   return [a for a in anomalies
           if a.start_revision <= end and a.end_revision >= start]