commit 1ee78cdaac80d5dde7f630e6497b7b0829e38114
author Andrii Shyshkalov <tandrii@google.com> Thu Mar 12 01:31:53 2020 +0000
committer LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Mar 12 01:31:53 2020 +0000
tree dcf35ecf4aba82842fdcb99efa1e98c41e198612
parent 4c7c4b4bc6e8f294d611679364a821508e21b6bd

git cl: use short URLs in `git cl status`.

This also applies to git map-branches.

R=ehmaldonado

Change-Id: Ica9bc92660716c15cbdd6bfd77b93f08bd0d8f9b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2099329
Commit-Queue: Andrii Shyshkalov <tandrii@google.com>
Auto-Submit: Andrii Shyshkalov <tandrii@google.com>
Reviewed-by: Edward Lesmes <ehmaldonado@chromium.org>

tests/git_cl_test.py

diff --git a/tests/git_cl_test.py b/tests/git_cl_test.py
index 85ab724..0aa29d9 100755
--- a/tests/git_cl_test.py
+++ b/tests/git_cl_test.py
@@ -249,6 +249,18 @@
     self.assertEqual(
         set([(changes[0], 'waiting'), (changes[1], 'error')]), actual)
 
+  def test_get_issue_url(self):
+    cl = git_cl.Changelist(issue=123)
+    cl._gerrit_server = 'https://example.com'
+    self.assertEqual(cl.GetIssueURL(), 'https://example.com/123')
+    self.assertEqual(cl.GetIssueURL(short=True), 'https://example.com/123')
+
+    cl = git_cl.Changelist(issue=123)
+    cl._gerrit_server = 'https://chromium-review.googlesource.com'
+    self.assertEqual(cl.GetIssueURL(),
+                     'https://chromium-review.googlesource.com/123')
+    self.assertEqual(cl.GetIssueURL(short=True), 'https://crrev.com/c/123')
+
   def test_set_preserve_tryjobs(self):
     d = git_cl.ChangeDescription('Simple.')
     d.set_preserve_tryjobs()