Refactoring and paralelizing Python password manager tests.

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

Index: components/test/data/password_manager/automated_tests/environment.py
diff --git a/components/test/data/password_manager/automated_tests/environment.py b/components/test/data/password_manager/automated_tests/environment.py
index 86685b30057499ffd314ea4d28b654d27cb0a4fa..71eca92129228c40e6d6ed4f27d6e4a65addaa0a 100644
--- a/components/test/data/password_manager/automated_tests/environment.py
+++ b/components/test/data/password_manager/automated_tests/environment.py
@@ -87,10 +87,7 @@ class Environment:
     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.