git-cl: Better warning message for legacy buckets.

Bug: 976104
Change-Id: Ide06f73faf7a9ce6aa2d7f47deffc0c505dcdb14
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1835017
Auto-Submit: Edward Lesmes <ehmaldonado@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@google.com>
Reviewed-by: Anthony Polito <apolito@google.com>
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>

git_cl.py

@@ -423,19 +423,23 @@ def _get_bucket_map(changelist, options, option_parser):
       'Please specify the bucket, e.g. "-B luci.chromium.try".')
 
 
-def _parse_bucket(bucket):
+def _parse_bucket(raw_bucket):
-  if '/' in bucket:
+  legacy = True
-    return tuple(bucket.split('/', 1))
+  project = bucket = None
-  # Legacy buckets.
+  if '/' in raw_bucket:
-  print('WARNING Please specify buckets as <project>/<bucket>.')
+    legacy = False
+    project, bucket = raw_bucket.split('/', 1)
   # Assume luci.<project>.<bucket>.
-  if bucket.startswith('luci.'):
+  elif raw_bucket.startswith('luci.'):
-    return tuple(bucket[len('luci.'):].split('.', 1))
+    project, bucket = raw_bucket[len('luci.'):].split('.', 1)
   # Otherwise, assume prefix is also the project name.
-  if '.' in bucket:
+  elif '.' in raw_bucket:
-    project = bucket.split('.')[0]
+    project = raw_bucket.split('.')[0]
+    bucket = raw_bucket
+  # Legacy buckets.
+  if legacy:
+    print('WARNING Please use %s/%s to specify the bucket.' % (project, bucket))
   return project, bucket
-  return None, None
 
 
 def _trigger_try_jobs(auth_config, changelist, buckets, options, patchset):

tests/git_cl_test.py

@@ -3321,8 +3321,9 @@ class CMDTryTestCase(unittest.TestCase):
       self.assertEqual(
           test_case['result'], git_cl._parse_bucket(test_case['bucket']))
       if test_case.get('has_warning'):
-        self.assertIn(
+        expected_warning = 'WARNING Please use %s/%s to specify the bucket' % (
-            'WARNING Please specify buckets', git_cl.sys.stdout.getvalue())
+            test_case['result'])
+        self.assertIn(expected_warning, git_cl.sys.stdout.getvalue())
 
 
 class CMDUploadTestCase(unittest.TestCase):