git cl status for Gerrit.

git_cl.py

 #!/usr/bin/env python
 # Copyright (c) 2012 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.
 
 # Copyright (C) 2008 Evan Martin <martine@danga.com>
 
 """A git-command for integrating reviews on Rietveld and Gerrit."""
 
 from distutils.version import LooseVersion
@@ skipping 1564 matching lines @@
         print(
             'You aren\'t using Rietveld at the moment, but Gerrit.\n'
             'Using Rietveld in your PRESUBMIT scripts won\'t work.\n'
             'Please, either change your PRESUBIT to not use rietveld_obj.%s,\n'
             'or use Rietveld for codereview.\n'
             'See also http://crbug.com/579160.' % attr)
         raise NotImplementedError()
     return ThisIsNotRietveldIssue()
 
   def GetStatus(self):
-    # TODO(tandrii)
-    raise NotImplementedError()
+    """Apply a rough heuristic to give a simple summary of an issue's review
+    or CQ status, assuming adherence to a common workflow.
+
+    Returns None if no issue for this branch, or one of the following keywords:
+      * 'error'   - error from review tool (including deleted issues)
+      * 'unsent'  - no reviewers added
+      * 'waiting' - waiting for review
+      * 'reply'   - waiting for owner to reply to review
+      * 'lgtm'    - Code-Review +2 from at least one approved reviewer
+      * 'commit'  - in the commit queue
+      * 'closed'  - abandoned
+    """
+    if not self.GetIssue():
+      return None
+
+    try:
+      data = self._GetChangeDetail(['DETAILED_LABELS', 'CURRENT_REVISION'])
+    except httplib.HTTPException:
+      return 'error'
+
+    if data['status'] == 'ABANDONED':
+      return 'abandoned'

[ukai, 2016/03/28 02:19:51]

'closed' ?
(or add 'abondoned' in docstring above?)

[tandrii(chromium), 2016/03/28 18:57:21]

Done.

+
+    cq_label = data['labels'].get('Commit-Queue', {})
+    if cq_label:
+      # Vote value is a stringified integer, which we expect from 0 to 2.
+      vote_value = cq_label.get('value', '0')
+      vote_text = cq_label.get('values', {}).get(vote_value, '')
+      if vote_text.lower() == 'Commit'.lower():

[ukai, 2016/03/28 02:19:51]

if vote.text.lower() == 'commit':
?

[tandrii(chromium), 2016/03/28 18:57:21]

Done.

+        return 'commit'
+
+    lgtm_label = data['labels'].get('Code-Review', {})
+    if lgtm_label:
+      if 'rejected' in lgtm_label:
+        return 'not lgtm'

[ukai, 2016/03/28 02:19:51]

'waiting' ?
(or add 'not lgtm' in docstring above?)

[tandrii(chromium), 2016/03/28 18:57:21]

Done.

+      if 'approved' in lgtm_label:
+        return 'lgtm'
+
+    if not data.get('reviewers', {}).get('REVIEWER', []):
+      return 'unsent'
+
+    messages = data.get('messages', [])
+    if messages:
+      owner = data['owner'].get('_account_id')
+      last_message_author = messages[-1].get('author', {}).get('_account_id')
+      if owner != last_message_author:
+        # Some reply from non-owner.
+        return 'reply'
+
+    return 'waiting'
 
   def GetMostRecentPatchset(self):
-    data = gerrit_util.GetChangeDetail(self._GetGerritHost(), self.GetIssue(),
-                                       ['CURRENT_REVISION'])
+    data = self._GetChangeDetail(['CURRENT_REVISION'])
     return data['revisions'][data['current_revision']]['_number']
 
   def FetchDescription(self):
-    data = gerrit_util.GetChangeDetail(self._GetGerritHost(), self.GetIssue(),
-                                       ['COMMIT_FOOTERS', 'CURRENT_REVISION'])
+    data = self._GetChangeDetail(['COMMIT_FOOTERS', 'CURRENT_REVISION'])
     return data['revisions'][data['current_revision']]['commit_with_footers']
 
   def UpdateDescriptionRemote(self, description):
     # TODO(tandrii)
     raise NotImplementedError()
 
   def CloseIssue(self):
     gerrit_util.AbandonChange(self._GetGerritHost(), self.GetIssue(), msg='')
 
 
+  def _GetChangeDetail(self, options):
+    return gerrit_util.GetChangeDetail(self._GetGerritHost(), self.GetIssue(),
+                                       options)
+
+
 class ChangeDescription(object):
   """Contains a parsed form of the change description."""
   R_LINE = r'^[ \t]*(TBR|R)[ \t]*=[ \t]*(.*?)[ \t]*$'
   BUG_LINE = r'^[ \t]*(BUG)[ \t]*=[ \t]*(.*?)[ \t]*$'
 
   def __init__(self, description):
     self._description_lines = (description or '').strip().splitlines()
 
   @property               # www.logilab.org/ticket/89786
   def description(self):  # pylint: disable=E0202
@@ skipping 2653 matching lines @@
 if __name__ == '__main__':
   # These affect sys.stdout so do it outside of main() to simplify mocks in
   # unit testing.
   fix_encoding.fix_encoding()
   colorama.init()
   try:
     sys.exit(main(sys.argv[1:]))
   except KeyboardInterrupt:
     sys.stderr.write('interrupted\n')
     sys.exit(1)