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 8b00ac7e15647f8ff5a5f9aabf9cde1699feb363..3956234dbd3fa85608fef52a458bd6e97e5ae42a 100755
--- a/tests/git_cl_test.py
+++ b/tests/git_cl_test.py
@@ -1381,6 +1381,50 @@ class TestGitCl(TestCase):
     # super tedious.
     self.assertEqual(0, git_cl.main(['set-commit', '-d']))
 
+  def test_description_display(self):
+    out = StringIO.StringIO()
+    self.mock(git_cl.sys, 'stdout', out)
+
+    class MockChangelist():
+      def __init__(self, **kwargs):
+        pass
+      def GetIssue(self):
+        return 1
+      def GetDescription(self):
+        return 'foo'
+
+    self.mock(git_cl, 'Changelist', MockChangelist)
+
+    self.assertEqual(0, git_cl.main(['description', '-d']))
+    self.assertEqual('foo\n', out.getvalue())
+
+  def test_description_rietveld(self):
+    out = StringIO.StringIO()
+    self.mock(git_cl.sys, 'stdout', out)
+    self.mock(git_cl.Changelist, 'GetDescription',

[tandrii(chromium), 2016/04/26 10:34:20]

if you have diff tests for Gerrit/Rietveld, then actually mock FetchDescription
of the respective implementation classes. That said, I personally would just
have kept the Gerrit test below only :)

+              lambda *args: 'foobar')
+
+    self.calls = [
+        ((['git', 'config', 'rietveld.autoupdate'],), ''),
+        ((['git', 'config', 'rietveld.server'],), ''),
+        ((['git', 'config', 'rietveld.server'],), ''),
+    ]
+    self.assertEqual(0, git_cl.main([
+        'description', 'https://code.review.org/123123', '-d', '--rietveld']))
+    self.assertEqual('foobar\n', out.getvalue())
+
+  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())
+
+
+
 
 if __name__ == '__main__':
   git_cl.logging.basicConfig(