[Password manager Python tests] Remove the option to disable 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 100 matching lines @@
         self.passwords_tree = ElementTree.parse(passwords_path).getroot()
       else:
         raise Exception("Error: |passwords_path| needs to be provided if"
             "|chrome_path| is provided, otherwise the tests could not be run")
     # Password internals page.
     self.internals_page = "chrome://password-manager-internals/"
     # The Website window.
     self.website_window = None
     # The WebsiteTests list.
     self.websitetests = []
-    # The enabled WebsiteTests list.
-    self.working_tests = []
-    # The disabled WebsiteTests list.
-    self.disabled_tests = []
     # Map messages to the number of their appearance in the log.
     self.message_count = dict()
     self.message_count[MESSAGE_ASK] = 0
     self.message_count[MESSAGE_SAVE] = 0
     # The tests needs two tabs to work. A new tab is opened with the first
     # GoTo. This is why we store here whether or not it's the first time to
     # execute GoTo.
     self.first_go_to = True
     # List of all tests results.
     self.tests_results = []
 
-  def AddWebsiteTest(self, websitetest, disabled=False):
+  def AddWebsiteTest(self, websitetest):
     """Adds a WebsiteTest to the testing Environment.
 
     Args:
       websitetest: The WebsiteTest instance to be added.
-      disabled: Whether test is disabled.
     """
     websitetest.environment = self
     if hasattr(self, "driver"):
       websitetest.driver = self.driver
     if hasattr(self, "passwords_tree") and self.passwords_tree is not None:
       if not websitetest.username:
         username_tag = (
             self.passwords_tree.find(
                 ".//*[@name='%s']/username" % websitetest.name))
         if username_tag.text:
           websitetest.username = username_tag.text
       if not websitetest.password:
         password_tag = (
             self.passwords_tree.find(
                 ".//*[@name='%s']/password" % websitetest.name))
         if password_tag.text:
           websitetest.password = password_tag.text
     self.websitetests.append(websitetest)
-    if disabled:
-      self.disabled_tests.append(websitetest.name)
-    else:
-      self.working_tests.append(websitetest.name)
 
   def ClearCache(self, clear_passwords):
     """Clear the browser cookies. If |clear_passwords| is true, clear all the
     saved passwords too.
 
     Args:
       clear_passwords : Clear all the passwords if the bool value is true.
     """
     logging.info("\nClearCache\n")
     self.driver.get("chrome://settings/clearBrowserData")
@@ skipping 135 matching lines @@
           the prompt are going to be run.
 
     Raises:
       Exception: An exception is raised if the tests fail.
     """
     if prompt_test:
       self.PromptTestList(self.websitetests)
     else:
       self.TestList(self.websitetests)
 
-  def DisabledTests(self, prompt_test):
-    """Runs the tests on all the disabled WebsiteTests.
-
-    Args:
-      prompt_test: If True, tests caring about showing the save-password
-          prompt are going to be run, otherwise tests which don't care about
-          the prompt are going to be executed.
-
-    Raises:
-      Exception: An exception is raised if the tests fail.
-    """
-    self.Test(self.disabled_tests, prompt_test)
-
-  def WorkingTests(self, prompt_test):
-    """Runs the tests on all the enabled WebsiteTests.
-
-    Args:
-      prompt_test: If True, tests caring about showing the save-password
-          prompt are going to be run, otherwise tests which don't care about
-          the prompt are going to be executed.
-
-    Raises:
-      Exception: An exception is raised if the tests fail.
-    """
-    self.Test(self.working_tests, prompt_test)
-
   def Test(self, tests, prompt_test):
     """Runs the tests on websites named in |tests|.
 
     Args:
       tests: A list of the names of the WebsiteTests that are going to be
           tested.
       prompt_test: If True, tests caring about showing the save-password
           prompt are going to be run, otherwise tests which don't care about
           the prompt are going to be executed.
 
@@ skipping 60 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()