Test Autofill's ability to merge duplicate profiles and...

chrome/test/functional/autofill.py

Index: chrome/test/functional/autofill.py
===================================================================
--- chrome/test/functional/autofill.py	(revision 74022)
+++ chrome/test/functional/autofill.py	(working copy)
@@ -93,9 +93,9 @@
     self.assertEqual([expected_credit_card],
                      self.GetAutoFillProfile()['credit_cards'])
 
-  def testAutofillCrowdSourcing(self):
-    """Test able to send POST request of web form to crowd source server.
-    Require a loop of 1000 submits as the source server only collects 1% of
+  def testAutofillCrowdsourcing(self):
+    """Test able to send POST request of web form to toolbar server.
+    Require a loop of 1000 submits as the toolbar server only collects 1% of
     the data posted."""
     # HTML file needs to be run from a specific http:// url to be able to verify
     # the results a few days later by visiting the same url.
@@ -124,6 +124,35 @@
                              'window.domAutomationController.send("done")',
                              0, 0)
 
+  def testMergeDuplicateProfilesInAutofill(self):
+    """Test Autofill ability to merge duplicate profiles and throw away junk
+    profiles. The goal is to submit a couple hundre profiles and see the merge
+    outcome."""
+    # HTML file needs to be run from a http:// url.
+    url = self.GetHttpURLForDataPath(
+        os.path.join('autofill', 'dup-profiles-test.html'))
+    # Add and set Autofill profiles.
+    file_path = os.path.join(self.DataDir(), 'autofill',
+                             'dataset_duplicate-profiles.txt')
+    profiles = self.EvalDataFrom(file_path)
+    self.FillAutoFillProfile(profiles=profiles)
+    for i in range(len(self.GetAutoFillProfile()['profiles'])):

[Nirnimesh, 2011/02/07 22:20:02]

Why not:
for profile in self.GetAutoFillProfile()['profiles']:

[dyu1, 2011/02/07 23:06:59]

The second for loop wont work as profile will be a dictionary value versus an
int.

On 2011/02/07 22:20:02, Nirnimesh wrote:

[Nirnimesh, 2011/02/07 23:26:19]

for profile in self.GetAutoFillProfile()['profiles']:
  # |profile| is a dictionary
  # then..

  for key, val in profile.iteritems():
    ..

+      self.NavigateToURL(url, 0, 0)

[Nirnimesh, 2011/02/07 22:20:02]

remove 0, 0

[dyu1, 2011/02/07 23:06:59]

Done.

+      for key, value in self.GetAutoFillProfile()['profiles'][i].iteritems():
+        script = 'document.getElementById("%s").value = "%s"; ' \
+                 'window.domAutomationController.send("done")' % (key, value)
+        self.ExecuteJavascript(script, 0, 0)
+      self.ExecuteJavascript('document.getElementById("merge_dup").submit();'
+                             'window.domAutomationController.send("done")',
+                             0, 0)
+    print self.GetAutoFillProfile()['profiles']

[Nirnimesh, 2011/02/07 22:20:02]

Remove

[dyu1, 2011/02/07 23:06:59]

Done.

+    import pickle
+    # Write profile dictionary to a file.

[Nirnimesh, 2011/02/07 22:20:02]

Why?

[dyu1, 2011/02/07 23:06:59]

Retrieve a data dump to compare against original dictionary list to make sure
number are not the same.

On 2011/02/07 22:20:02, Nirnimesh wrote:

+    profile_dict = self.GetAutoFillProfile()['profiles']
+    output = open('merged-profile.txt', 'wb')
+    pickle.dump(profile_dict, output)
+    output.close()

[Nirnimesh, 2011/02/07 22:20:02]

What's the objective of this test? Where do you test it?

[dyu1, 2011/02/07 23:06:59]

Final test is a manual test to make sure the profiles are merged properly. This
is a large enough feature where we need a test such as this but wont be run on
the continuous build.

On 2011/02/07 22:20:02, Nirnimesh wrote:

 
+
 if __name__ == '__main__':
   pyauto_functional.Main()