[Findit] Fixing analysis mismatch in check flake

appengine/findit/model/flake/master_flake_analysis.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 base64
 
 from google.appengine.ext import ndb
 
 from gae_libs.model.versioned_model import VersionedModel
 from model import result_status
 from model import triage_status
 from model.base_analysis import BaseAnalysis
 from model.base_build_model import BaseBuildModel
+from model.base_triaged_model import TriagedModel
 from model.flake.flake_swarming_task import FlakeSwarmingTaskData
-from model.base_triaged_model import TriagedModel
 
 
 class DataPoint(ndb.Model):
   build_number = ndb.IntegerProperty(indexed=False)
   pass_rate = ndb.FloatProperty(indexed=False)
 
 
 class MasterFlakeAnalysis(
     BaseAnalysis, BaseBuildModel, VersionedModel, TriagedModel):
   """Represents an analysis of a flaky test on a Waterfall test cycle."""
@@ skipping 18 matching lines @@
       return -1
     return self.algorithm_parameters.get('iterations_to_rerun')
 
   @staticmethod
   def _CreateAnalysisId(
       master_name, builder_name, build_number, step_name, test_name):
     encoded_test_name = base64.urlsafe_b64encode(test_name)
     return '%s/%s/%s/%s/%s' % (
         master_name, builder_name, build_number, step_name, encoded_test_name)
 
+  @staticmethod
+  def GetBuildConfigurationFromKey(master_flake_analysis_key):
+    """Extracts master_name and builder_name from key."""
+    if not master_flake_analysis_key:
+      return None, None
+
+    components = master_flake_analysis_key.pairs()[0][1].split('/')
+    master_name = components[0]
+    builder_name = components[1]
+    return master_name, builder_name
+
   # Arguments number differs from overridden method - pylint: disable=W0221
   @classmethod
   def Create(cls, master_name, builder_name, build_number, step_name,
              test_name):  # pragma: no cover.
     return super(MasterFlakeAnalysis, cls).Create(
         MasterFlakeAnalysis._CreateAnalysisId(
             master_name, builder_name, build_number, step_name, test_name))
 
   # Arguments number differs from overridden method - pylint: disable=W0221
   @classmethod
@@ skipping 85 matching lines @@
   # Findit's automatic analysis upon detection, or Findit API.
   triggering_source = ndb.IntegerProperty(default=None, indexed=True)
 
   # Who triggered the analysis. Used for differentiating between manual and
   # automatic runs, and determining the most active users to gather feedback.
   triggering_user_email = ndb.StringProperty(default=None, indexed=False)
 
   # Overall conclusion of analysis result for the flake. Found untriaged, Found
   # Correct, etc. used to filter what is displayed on the check flake dashboard.
   result_status = ndb.IntegerProperty(indexed=True)