commit 34924cd4f02f6ae3dadf03a39b74d255b659f5a4
author Andrii Shyshkalov <tandrii@chromium.org> Wed Mar 15 17:08:32 2017 +0100
committer Commit Bot <commit-bot@chromium.org> Wed Mar 15 17:19:32 2017 +0000
tree b2b944157db8fd164058fcc3991acaffdfdacdd3
parent b7cb65a58df8732f96a9b7ef9fcd9f3368b4c7ce

git cl creds-check: add pretty printing of existing creds.

BUG=689543

Change-Id: I085f3e086059ecf3b57a8e0cd0d716adc54f8644
Reviewed-on: https://chromium-review.googlesource.com/454816
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Michael Achenbach <machenbach@chromium.org>

tests/git_cl_test.py

diff --git a/tests/git_cl_test.py b/tests/git_cl_test.py
index d112f0b..588c016 100755
--- a/tests/git_cl_test.py
+++ b/tests/git_cl_test.py
@@ -2933,6 +2933,38 @@
     self.assertEqual(cl.GetDescription(), 'desc1')  # cache hit.
     self.assertEqual(cl.GetDescription(force=True), 'desc2')
 
+  def test_print_current_creds(self):
+    class CookiesAuthenticatorMock(object):
+      def __init__(self):
+        self.gitcookies = {
+            'host.googlesource.com': ('user', 'pass'),
+            'host-review.googlesource.com': ('user', 'pass'),
+        }
+        self.netrc = self
+        self.netrc.hosts = {
+            'github.com': ('user2', None, 'pass2'),
+            'host2.googlesource.com': ('user3', None, 'pass'),
+        }
+    self.mock(git_cl.gerrit_util, 'CookiesAuthenticator',
+              CookiesAuthenticatorMock)
+    self.mock(sys, 'stdout', StringIO.StringIO())
+    git_cl._GitCookiesChecker().print_current_creds(include_netrc=True)
+    self.assertEqual(list(sys.stdout.getvalue().splitlines()), [
+        '                        Host\t User\t Which file',
+        '============================\t=====\t===========',
+        'host-review.googlesource.com\t user\t.gitcookies',
+        '       host.googlesource.com\t user\t.gitcookies',
+        '      host2.googlesource.com\tuser3\t     .netrc',
+    ])
+    sys.stdout.buf = ''
+    git_cl._GitCookiesChecker().print_current_creds(include_netrc=False)
+    self.assertEqual(list(sys.stdout.getvalue().splitlines()), [
+        '                        Host\tUser\t Which file',
+        '============================\t====\t===========',
+        'host-review.googlesource.com\tuser\t.gitcookies',
+        '       host.googlesource.com\tuser\t.gitcookies',
+    ])
+
   def _common_creds_check_mocks(self):
     def exists_mock(path):
       dirname = os.path.dirname(path)
@@ -2948,6 +2980,8 @@
 
   def test_creds_check_gitcookies_not_configured(self):
     self._common_creds_check_mocks()
+    self.mock(git_cl._GitCookiesChecker, 'get_hosts_with_creds',
+              lambda _, include_netrc: [])
     self.calls = [
       ((['git', 'config', '--global', 'http.cookiefile'],), CERR1),
       (('os.path.exists', '~/.netrc'), True),
@@ -2966,6 +3000,8 @@
 
   def test_creds_check_gitcookies_configured_custom_broken(self):
     self._common_creds_check_mocks()
+    self.mock(git_cl._GitCookiesChecker, 'get_hosts_with_creds',
+              lambda _, include_netrc: [])
     self.calls = [
       ((['git', 'config', '--global', 'http.cookiefile'],),
        '/custom/.gitcookies'),
@@ -2983,6 +3019,7 @@
         sys.stdout.getvalue(),
         'However, your configured .gitcookies file is missing.')
 
+
 if __name__ == '__main__':
   logging.basicConfig(
       level=logging.DEBUG if '-v' in sys.argv else logging.ERROR)