cli: convert outputformat %{external_bugs} to full URLs

crobinso · crobinso · commit 238f055af371 · 2018-03-05T18:27:52.000-05:00
Resolves: #65
diff --git a/bugzilla/_cli.py b/bugzilla/_cli.py
@@ -713,6 +713,14 @@ def bug_field(matchobj):
                 val += ("\n* %s - %s:\n%s\n" % (c['time'],
                          c.get("creator", c.get("author", "")), c['text']))
 
+        elif fieldname == "external_bugs":
+            val = ""
+            for e in getattr(b, "external_bugs", []):
+                url = e["type"]["full_url"].replace("%id%", e["ext_bz_bug_id"])
+                if not val:
+                    val += "\n"
+                val += "External bug: %s\n" % url
+
         elif fieldname == "__unicode__":
             val = b.__unicode__()
         else:
diff --git a/tests/ro_functional.py b/tests/ro_functional.py
@@ -337,3 +337,10 @@ def testExtraFields(self):
         bug = bz.getbug(720773, include_fields=["summary"])
         self.assertTrue("summary" in dir(bug))
         self.assertTrue("comments" not in dir(bug))
+
+    def testExternalBugsOutput(self):
+        out = self.clicomm('query --bug_id 989253 '
+            '--outputformat="%{external_bugs}"')
+        expect = ("http://bugzilla.gnome.org/show_bug.cgi?id=703421\n" +
+            "External bug: https://bugs.launchpad.net/bugs/1203576")
+        self.assertTrue(expect in out)
