git_cl: description fetching from code review servers.

tests/git_cl_test.py

Index: tests/git_cl_test.py
diff --git a/tests/git_cl_test.py b/tests/git_cl_test.py
index a78c7383c5dc992a6906de3cbae6d36d50dbd4ad..831d90a63b15752a18391a9a1ebdc14adf0e1739 100755
--- a/tests/git_cl_test.py
+++ b/tests/git_cl_test.py
@@ -1377,6 +1377,44 @@ class TestGitCl(TestCase):
     # super tedious.
     self.assertEqual(0, git_cl.main(['set-commit', '-d']))
 
+  def test_description_display(self):
+    real_out = sys.stdout

[tandrii(chromium), 2016/04/19 09:42:04]

remove this

+    out = StringIO.StringIO()
+    self.mock(git_cl.sys, 'stdout', out)
+    self.mock(git_cl.Changelist, 'GetDescription',
+              lambda *args: 'foobar')
+
+    self.calls = [
+        ((['git', 'symbolic-ref', 'HEAD'],), 'feature'),
+        ((['git', 'config', 'branch.feature.rietveldissue'],), '123123'),
+        ((['git', 'config', 'rietveld.autoupdate'],), ''),
+        ((['git', 'config', 'rietveld.server'],), ''),
+        ((['git', 'config', 'rietveld.server'],), ''),
+    ]
+    self.assertEqual(0, git_cl.main(['description', '-d']))
+    self.assertEqual('foobar\n', out.getvalue())
+
+  def test_description_rietveld(self):
+    real_out = sys.stdout

[tandrii(chromium), 2016/04/19 09:42:04]

ditto

+    out = StringIO.StringIO()
+    self.mock(git_cl.sys, 'stdout', out)
+    self.mock(git_cl.Changelist, 'GetDescription',
+              lambda *args: 'foobar')
+
+    self.calls = [
+        ((['git', 'symbolic-ref', 'HEAD'],), 'feature'),
+        ((['git', 'config', 'branch.feature.rietveldissue'],), '123123'),

[tandrii(chromium), 2016/04/19 09:42:04]

this is wrong. This means that coincidentally local branch has already issue
123123 associated. This call should never be made if '-i' flag is given.

+        ((['git', 'config', 'rietveld.autoupdate'],), ''),
+        ((['git', 'config', 'rietveld.server'],), ''),
+        ((['git', 'config', 'rietveld.server'],), ''),
+    ]
+    self.assertEqual(0, git_cl.main([
+        'description', '-d', '-i', '123123', '-s', 'https://code.review.org'

[tandrii(chromium), 2016/04/19 09:42:04]

ah, this is outdated test.

[martiniss, 2016/04/25 22:33:06]

Yup. Tests should be good to go now.

+        '--rietveld']))
+    self.assertEqual('foobar\n', out.getvalue())
+
+
+
 
 if __name__ == '__main__':
   git_cl.logging.basicConfig(