commit 892f2ceea5c9cb143790d79c3fd7ab533c8e1337
author Joanna Wang <jojwang@chromium.org> Tue Mar 14 21:39:47 2023 +0000
committer LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Mar 14 21:39:47 2023 +0000
tree 9d0104c4e2221a4bc1484be8ac6fa137e1f8ef6b
parent 8d8e3238c06f378d58468ba0b262672c6f016a9c

Fix git_cl tests for DOGFOOD_STACKED_CHANGES=1

Bug: 1422684
Change-Id: I0c772951cdce8bbe0b03b0008d26e80adea593d7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/4337899
Commit-Queue: Gavin Mak <gavinmak@google.com>
Auto-Submit: Joanna Wang <jojwang@chromium.org>
Reviewed-by: Gavin Mak <gavinmak@google.com>

git_cl.py

diff --git a/git_cl.py b/git_cl.py
index 5cb46e7..7ae990d 100755
--- a/git_cl.py
+++ b/git_cl.py
@@ -162,6 +162,11 @@
 _MAX_STACKED_BRANCHES_UPLOAD = 20
 
 
+# Environment variable to indicate if user is participating in the stcked
+# changes dogfood.
+DOGFOOD_STACKED_CHANGES_VAR = 'DOGFOOD_STACKED_CHANGES'
+
+
 # Repo prefixes that are enrolled in the stacked changes dogfood.
 DOGFOOD_STACKED_CHANGES_REPOS = [
     'chromium.googlesource.com/infra/',
@@ -4796,7 +4801,7 @@
     options.retry_failed = False
 
   remote = cl.GetRemoteUrl()
-  dogfood_stacked_changes = (os.environ.get('DOGFOOD_STACKED_CHANGES')
+  dogfood_stacked_changes = (os.environ.get(DOGFOOD_STACKED_CHANGES_VAR)
                              not in ['1', '0']
                              and any(repo in remote
                                      for repo in DOGFOOD_STACKED_CHANGES_REPOS))
@@ -4807,7 +4812,7 @@
           'File bugs at https://bit.ly/3Y6opoI')
 
   if options.squash and (dogfood_stacked_changes
-                         or os.environ.get('DOGFOOD_STACKED_CHANGES') == '1'):
+                         or os.environ.get(DOGFOOD_STACKED_CHANGES_VAR) == '1'):
     if options.dependencies:
       parser.error('--dependencies is not available for this workflow.')