[Findit] Capture versionized metadata for master_flake_analysis

appengine/findit/handlers/flake/test/check_flake_test.py

Index: appengine/findit/handlers/flake/test/check_flake_test.py
diff --git a/appengine/findit/handlers/flake/test/check_flake_test.py b/appengine/findit/handlers/flake/test/check_flake_test.py
index 742c5247c9dbe91098a9450368ac825ff4fa6e45..005934a74d99256ea3273e8f011ae04f6fc2ed99 100644
--- a/appengine/findit/handlers/flake/test/check_flake_test.py
+++ b/appengine/findit/handlers/flake/test/check_flake_test.py
@@ -19,15 +19,6 @@ class CheckFlakeTest(wf_testcase.WaterfallTestCase):
       ('/waterfall/check-flake', check_flake.CheckFlake),
   ], debug=True)
 
-  def _CreateAndSaveMasterFlakeAnalysis(
-      self, master_name, builder_name, build_number,
-      step_name, test_name, status):
-    analysis = MasterFlakeAnalysis.Create(
-        master_name, builder_name, build_number, step_name, test_name)
-    analysis.status = status
-    analysis.put()
-    return analysis
-
   def testCorpUserCanScheduleANewAnalysis(self):
     master_name = 'm'
     builder_name = 'b'
@@ -74,14 +65,13 @@ class CheckFlakeTest(wf_testcase.WaterfallTestCase):
     step_name = 's'
     test_name = 't'
     success_rate = .9
-    status = analysis_status.PENDING
 
-    master_flake_analysis = self._CreateAndSaveMasterFlakeAnalysis(
-        master_name, builder_name, build_number, step_name,
-        test_name, status)
+    master_flake_analysis = MasterFlakeAnalysis.Create(
+        master_name, builder_name, build_number, step_name, test_name)
+    master_flake_analysis.status = analysis_status.PENDING
     master_flake_analysis.build_numbers.append(int(build_number))
-    master_flake_analysis.success_rates.append(success_rate)
-    master_flake_analysis.put()
+    master_flake_analysis.pass_rates.append(success_rate)
+    master_flake_analysis.Save()
 
     response = self.test_app.get('/waterfall/check-flake', params={
         'master_name': master_name,
@@ -91,9 +81,8 @@ class CheckFlakeTest(wf_testcase.WaterfallTestCase):
         'test_name': test_name,
         'format': 'json'})
 
-    self.assertEquals(200, response.status_int)
-    expected_check_flake_result ={
-        'success_rates': [[int(build_number), success_rate]],
+    expected_check_flake_result = {
+        'pass_rates': [[int(build_number), success_rate]],
         'analysis_status': STATUS_TO_DESCRIPTION.get(
             master_flake_analysis.status),
         'master_name': master_name,
@@ -103,4 +92,6 @@ class CheckFlakeTest(wf_testcase.WaterfallTestCase):
         'test_name': test_name,
         'suspected_flake_build_number': None
     }
+
+    self.assertEquals(200, response.status_int)
     self.assertEqual(expected_check_flake_result, response.json_body)