Refactoring and paralelizing Python password manager tests.

components/test/data/password_manager/automated_tests/environment.py

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """The testing Environment class."""
 
 import logging
 import shutil
 import sys
 import time
@@ skipping 69 matching lines @@
           # Add the handler to the root logger.
           logging.getLogger('').addHandler(console)
 
       elif log_to_console:
         logging.basicConfig(level=numeric_level)
 
     # Cleaning the chrome testing profile folder.
     try:
       shutil.rmtree(profile_path)
     except Exception, e:
-      # The tests execution can continue, but this make them less stable.
-      logging.error("Error: Could not wipe the chrome profile directory (%s). \
-          This affects the stability of the tests. Continuing to run tests."
-          % e)
+      pass

[melandory, 2015/02/06 08:37:46]

I disagree with removing of this log message: it's needed to catch wrong setup
of profile folder.

And i think I know why you are removing it, it's because you get this exception
every time you run tests. It's happening because of line 175  run_tests.py
environment = Environment('', '', '', None, False)
this dummy environment serves only one purpose -- get tests names (line 190). I
think it's better to tackle the problem that we need to know test name before
creating real environment, instead of removing consequence.

[dvadym, 2015/02/06 11:02:26]

Now each test run has own temporary profile directory. So this is not a problem
anymore. And yeah, it failed each time, since it can't remove non-existing
temporary folder :).

     # If |chrome_path| is not defined, this means that we are in the dashboard
     # website, and we just need to get the list of all websites. In this case,
     # we don't need to initilize the webdriver.
     if chrome_path:
       options = Options()
       self.enable_automatic_password_saving = enable_automatic_password_saving
       if enable_automatic_password_saving:
         options.add_argument("enable-automatic-password-saving")
       # Chrome path.
       options.binary_location = chrome_path
@@ skipping 314 matching lines @@
       except Exception as e:
         successful = False
         error = e.message
       self.tests_results.append(TestResult(websitetest.name, "prompt",
                                            successful, error))
 
   def Quit(self):
     """Closes the tests."""
     # Close the webdriver.
     self.driver.quit()