Add --issue flag to git cl status.

tests/git_cl_test.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.
 
 """Unit tests for git_cl.py."""
 
 import os
 import StringIO
 import stat
@@ skipping 1514 matching lines @@
 
   def test_description_rietveld(self):
     out = StringIO.StringIO()
     self.mock(git_cl.sys, 'stdout', out)
     self.mock(git_cl.Changelist, 'GetDescription', lambda *args: 'foobar')
 
     self.assertEqual(0, git_cl.main([
         'description', 'https://code.review.org/123123', '-d', '--rietveld']))
     self.assertEqual('foobar\n', out.getvalue())
 
+  def test_StatusFieldOverrideIssueMissingArgs(self):
+    out = StringIO.StringIO()
+    self.mock(git_cl.sys, 'stderr', out)
+
+    try:
+      git_cl.main(['status', '--issue', '1'])

[tandrii(chromium), 2016/08/17 20:16:32]

add assert 0 :)

[iannucci, 2016/08/17 20:20:36]

done

+    except SystemExit as ex:
+      self.assertEqual(ex.code, 2)
+      self.assertRegexpMatches(out.getvalue(), r'--issue may only be specified')
+
+    out = StringIO.StringIO()
+    self.mock(git_cl.sys, 'stderr', out)
+
+    try:
+      git_cl.main(['status', '--issue', '1', '--rietveld'])

[tandrii(chromium), 2016/08/17 20:16:32]

ditto

[iannucci, 2016/08/17 20:20:36]

done

+    except SystemExit as ex:
+      self.assertEqual(ex.code, 2)
+      self.assertRegexpMatches(out.getvalue(), r'--issue may only be specified')
+
+  def test_StatusFieldOverrideIssueMissingArgs(self):
+    out = StringIO.StringIO()
+    self.mock(git_cl.sys, 'stdout', out)
+
+    def assertIssue(cl_self, *_args):
+      self.assertEquals(cl_self.issue, 1)
+      return 'foobar'
+
+    self.mock(git_cl.Changelist, 'GetDescription', assertIssue)
+    self.calls = [
+      ((['git', 'config', 'rietveld.autoupdate'],), ''),
+      ((['git', 'config', 'rietveld.server'],), ''),
+      ((['git', 'config', 'rietveld.server'],), ''),
+    ]
+    git_cl.main(['status', '--issue', '1', '--rietveld', '--field', 'desc'])
+    self.assertEqual(out.getvalue(), 'foobar\n')
+
   def test_description_gerrit(self):
     out = StringIO.StringIO()
     self.mock(git_cl.sys, 'stdout', out)
     self.mock(git_cl.Changelist, 'GetDescription', lambda *args: 'foobar')
 
     self.assertEqual(0, git_cl.main([
         'description', 'https://code.review.org/123123', '-d', '--gerrit']))
     self.assertEqual('foobar\n', out.getvalue())
 
   def test_description_set_raw(self):
@@ skipping 206 matching lines @@
         ((['exists', '/abs/git_repo_root/.git/hooks/commit-msg'],), True),
         ((['FileRead', '/abs/git_repo_root/.git/hooks/commit-msg'],),
          '...\n# From Gerrit Code Review\n...\nadd_ChangeId()\n'),
         (('Do you want to remove it now? [Yes/No]',), 'Yes'),
         ((['rm_file_or_tree', '/abs/git_repo_root/.git/hooks/commit-msg'],),
          ''),
     ]
     cl._codereview_impl._GerritCommitMsgHookCheck(offer_removal=True)
 
 
+

[tandrii(chromium), 2016/08/17 20:16:32]

delete

[iannucci, 2016/08/17 20:20:36]

done in later patchset :)

 if __name__ == '__main__':
   git_cl.logging.basicConfig(
       level=git_cl.logging.DEBUG if '-v' in sys.argv else git_cl.logging.ERROR)
   unittest.main()