[Findit] Rerun if the regression range is different.

appengine/findit/crash/test/findit_for_client_test.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.
 
 import copy
 import json
 
 from google.appengine.api import app_identity
 
+from crash import detect_regression_range
 from crash import findit_for_client
 from crash import findit_for_chromecrash
+from crash.chromecrash_parser import ChromeCrashParser
 from crash.test.crash_testcase import CrashTestCase
 from crash.type_enums import CrashClient
 from model.crash.fracas_crash_analysis import FracasCrashAnalysis
 
 
 class FinditForClientTest(CrashTestCase):
 
   def testCheckPolicyUnsupportedClient(self):
     pass_check, _ = findit_for_client.CheckPolicyForClient(
         {'signature': 'sig'}, '1', 'sig', 'unsupported_client',
@@ skipping 74 matching lines @@
     platform = 'linux'
     stack_trace = 'stack_trace'
     crash_identifiers = {
       'chrome_version': chrome_version,
       'signature': signature,
       'channel': 'canary',
       'platform': platform,
       'process_type': 'browser',
     }
     customized_data = {'channel': 'canary'}
+    regression_range = ['53.0.1235.1', '53.0.1235.1']
 
     analysis = FracasCrashAnalysis.Create(crash_identifiers)
 
     findit_for_client.ResetAnalysis(
         analysis, chrome_version, signature, CrashClient.FRACAS, platform,
-        stack_trace, customized_data)
+        stack_trace, customized_data, regression_range)
 
     analysis = FracasCrashAnalysis.Get(crash_identifiers)
     self.assertEqual(analysis.crashed_version, chrome_version)
     self.assertEqual(analysis.signature, signature)
     self.assertEqual(analysis.platform, platform)
     self.assertEqual(analysis.stack_trace, stack_trace)
     self.assertEqual(analysis.channel, customized_data['channel'])
+    self.assertEqual(analysis.regression_range, regression_range)
 
   def testCreateAnalysisForClient(self):
     crash_identifiers = {'signature': 'sig'}
     self.assertIsNotNone(findit_for_client.CreateAnalysisForClient(
         crash_identifiers, CrashClient.FRACAS))
 
   def testCreateAnalysisForUnsupportedClientId(self):
     crash_identifiers = {'signature': 'sig'}
     self.assertIsNone(findit_for_client.CreateAnalysisForClient(
         crash_identifiers, 'unsupported_id'))
@@ skipping 78 matching lines @@
         return expected_result, expected_tags
     self.mock(findit_for_chromecrash, 'FinditForChromeCrash',
               _MockFinditForChromeCrash)
 
     analysis = FracasCrashAnalysis.Create({'signature': 'sig'})
     analysis.client_id = CrashClient.FRACAS
 
     result, tags = findit_for_client.FindCulprit(analysis)
     self.assertEqual(result, expected_result)
     self.assertEqual(tags, expected_tags)
+
+    analysis.client_id = 'unsupported_client'
+    result, tags = findit_for_client.FindCulprit(analysis)
+    self.assertEqual(result, expected_result)
+    self.assertEqual(tags, expected_tags)
+
+
+  def testGetRegressionRange(self):
+    expected_regression_range = ['51.0.1233.1', '51.0.1233.2']
+    def _MockDetectRegressionRange(*_):
+      return expected_regression_range
+
+    self.mock(detect_regression_range, 'DetectRegressionRange',
+              _MockDetectRegressionRange)
+    historical_metadata = [{'chrome_version': '51.0.1233.0', 'cpm': 0},
+                           {'chrome_version': '51.0.1233.1', 'cpm': 0},
+                           {'chrome_version': '51.0.1233.2', 'cpm': 0.89}]
+
+    regression_range = findit_for_client.GetRegressionRange(
+        CrashClient.FRACAS, {'historical_metadata': historical_metadata})
+    self.assertEqual(regression_range, expected_regression_range)
+
+    regression_range = findit_for_client.GetRegressionRange(
+        'unsupported_client', {'historical_metadata': historical_metadata})
+    self.assertEqual(regression_range, None)