tryserver recipe_module: Add get_tags.

recipe_modules/git_cl/api.py

 # Copyright 2016 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.
 
 from recipe_engine import recipe_api
 
 class GitClApi(recipe_api.RecipeApi):
   def __call__(self, subcmd, args, name=None, **kwargs):
     if not name:
       name = 'git_cl ' + subcmd
     if 'cwd' not in kwargs:
       kwargs['cwd'] = (self.c and self.c.repo_location) or None
 
     return self.m.step(
         name, [self.package_repo_resource('git_cl.py')] + args, **kwargs)
 
-  def get_description(self, **kwargs):
-    return self('description', ['-d'], stdout=self.m.raw_io.output(), **kwargs)
+  def get_description(self, patch=None, codereview=None, **kwargs):
+    args = ['-d']
+    if patch or codereview:
+      assert patch and codereview, "Both patch and codereview must be provided"

[iannucci, 2016/05/12 00:25:52]

why make them optional and defaulted to None then?

[martiniss, 2016/05/13 19:23:56]

If one of them is provided, the other must be provided, but if neither are, then
it's fine... Should I do something different? Maybe update the assert?

[iannucci, 2016/05/23 19:52:33]

Acknowledged.

+      args.append(patch)
+      args.append('--%s' % codereview)
 
-  def set_description(self, description, **kwargs):
+    return self('description', args, stdout=self.m.raw_io.output(), **kwargs)
+
+  def set_description(self, description, patch=None, codereview=None, **kwargs):
+    args = ['-n', '-']
+    if patch or codereview:
+      assert patch and codereview, "Both patch and codereview must be provided"
+      args.append(patch)
+      args.append('--%s' % codereview)
+
     return self(
-        'description', ['-n', '-'],
-        stdout=self.m.raw_io.output(),
+        'description', args, stdout=self.m.raw_io.output(),
         stdin=self.m.raw_io.input(data=description),
         name='git_cl set description', **kwargs)