[Findit] Adding build model for flake try jobs

appengine/findit/model/wf_try_job.py

Index: appengine/findit/model/wf_try_job.py
diff --git a/appengine/findit/model/wf_try_job.py b/appengine/findit/model/wf_try_job.py
index 4736529bb6363e7d17faafccccf2339a95ebd3da..424ac2b73c11672cde4072607691b64e0eab62f4 100644
--- a/appengine/findit/model/wf_try_job.py
+++ b/appengine/findit/model/wf_try_job.py
@@ -5,10 +5,10 @@
 from google.appengine.ext import ndb
 
 from model.base_build_model import BaseBuildModel
-from model import analysis_status
+from model.base_try_job import BaseTryJob
 
 
-class WfTryJob(BaseBuildModel):
+class WfTryJob(BaseTryJob):
   """Represents a try job results for a failed build.
 
   'Wf' is short for waterfall.
@@ -19,21 +19,19 @@ class WfTryJob(BaseBuildModel):
   # A list of dict containing results and urls of each try job for test.
   test_results = ndb.JsonProperty(indexed=False, compressed=True)
 
-  # The status of the try job.
-  status = ndb.IntegerProperty(
-      default=analysis_status.PENDING, indexed=False)
-
   # A list of try job IDs associated with each try job for collecting metadata.
   try_job_ids = ndb.JsonProperty(indexed=False, compressed=True)

[stgao, 2016/12/16 23:55:54]

Do we still need this?

 
+  # Arguments number differs from overridden method - pylint: disable=W0221
   @staticmethod
-  def _CreateKey(master_name, builder_name, build_number):  # pragma: no cover
+  def _CreateKey(master_name, builder_name, build_number):
     return ndb.Key('WfTryJob',
                    BaseBuildModel.CreateBuildId(
                        master_name, builder_name, build_number))
 
+  # Arguments number differs from overridden method - pylint: disable=W0221
   @staticmethod
-  def Create(master_name, builder_name, build_number):  # pragma: no cover
+  def Create(master_name, builder_name, build_number):
     try_job = WfTryJob(
         key=WfTryJob._CreateKey(master_name, builder_name, build_number))
     try_job.compile_results = try_job.compile_results or []
@@ -42,15 +40,6 @@ class WfTryJob(BaseBuildModel):
     return try_job
 
   @staticmethod
-  def Get(master_name, builder_name, build_number):  # pragma: no cover
+  def Get(master_name, builder_name, build_number):
     return WfTryJob._CreateKey(
         master_name, builder_name, build_number).get()
-
-  @property
-  def completed(self):
-    return self.status in (
-        analysis_status.COMPLETED, analysis_status.ERROR)
-
-  @property
-  def failed(self):
-    return self.status == analysis_status.ERROR