Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4465)

Unified Diff: chrome/test/functional/passwords.py

Issue 6994005: Fix password automation hooks to get pyauto test testSavePassword working. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Disabling a single pyauto passwords test. Created 9 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/test/functional/passwords.py
diff --git a/chrome/test/functional/passwords.py b/chrome/test/functional/passwords.py
index 3b39918a1bad74242ab6cc63276b950318458e47..1d2e60a2928188ea064b19a3a306b8e544619d9f 100644
--- a/chrome/test/functional/passwords.py
+++ b/chrome/test/functional/passwords.py
@@ -54,7 +54,13 @@ class PasswordTest(pyauto.PyUITest):
'https://www.example.com/', 'https://www.example.com/login',
'username', 'password', 'https://www.example.com/login/')
self.assertTrue(self.AddSavedPassword(password1))
- self.assertEquals(self.GetSavedPasswords(), [password1])
+ # There may be multiple logins for the same 'signon_realm', since other
Nirnimesh 2011/05/10 19:25:52 Wouldn't tests begin with a clean profile?
dennis_jeffrey 2011/05/17 01:40:21 Yes, this is true in Windows and ChromeOS (the onl
+ # tests in this file use them. We need to look for the particular login
+ # used by this test when verifiying.
+ saved_password = [x for x in self.GetSavedPasswords()
+ if x['signon_realm'] == 'https://www.example.com/' and
+ x['username_value'] == 'user@example.com']
+ self.assertEquals(saved_password, [password1])
def testRemovePasswords(self):
"""Verify that saved passwords can be removed."""

Powered by Google App Engine
This is Rietveld 408576698