commit ae3586b5380f0c00448c5a056528419b03ddf9b1
author Edward Lesmes <ehmaldonado@google.com> Mon Mar 23 21:21:14 2020 +0000
committer LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon Mar 23 21:21:14 2020 +0000
tree 7b4a3a6fbb79aa0d8e0837da6a9dd013bd326429
parent 10d9c49f23c8ba5e868ea50d049cf773e591a2a6

depot_tools: Use gclient_utils.AskForData instead of raw_input.

gclient_utils.AskForData will use raw_input on Python 2 and input on Python 3.

Bug: 1063976
Change-Id: I93c059c4e4454d01787716b5a0a2641ad5253f53
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2116370
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
Reviewed-by: Anthony Polito <apolito@google.com>
Auto-Submit: 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 0be5847..1cf2880 100755
--- a/tests/git_cl_test.py
+++ b/tests/git_cl_test.py
@@ -1218,7 +1218,7 @@
     mock.patch(
         'git_cl.GenerateGerritChangeId', return_value=change_id).start()
     mock.patch(
-        'git_cl.ask_for_data',
+        'gclient_utils.AskForData',
         lambda prompt: self._mocked_call('ask_for_data', prompt)).start()
 
     self.mockGit.config['gerrit.host'] = 'true'
@@ -1854,7 +1854,7 @@
 
   def _test_gerrit_ensure_authenticated_common(self, auth):
     mock.patch(
-        'git_cl.ask_for_data',
+        'gclient_utils.AskForData',
         lambda prompt: self._mocked_call('ask_for_data', prompt)).start()
     mock.patch('git_cl.gerrit_util.CookiesAuthenticator',
               CookiesAuthenticatorMockFactory(hosts_with_creds=auth)).start()
@@ -2253,7 +2253,7 @@
         'git_cl.gclient_utils.rm_file_or_tree',
         lambda path: self._mocked_call(['rm_file_or_tree', path])).start()
     mock.patch(
-        'git_cl.ask_for_data',
+        'gclient_utils.AskForData',
         lambda prompt: self._mocked_call('ask_for_data', prompt)).start()
     return git_cl.Changelist(issue=123)
 
@@ -2405,7 +2405,7 @@
       # git cl also checks for existence other files not relevant to this test.
       return None
     mock.patch(
-        'git_cl.ask_for_data',
+        'gclient_utils.AskForData',
         lambda prompt: self._mocked_call('ask_for_data', prompt)).start()
     mock.patch('os.path.exists', exists_mock).start()