[autotest] Fix unittest -- blacklist ap_configurator_test.py

ap_configurator_test depends on having selenium installed, which would
mean requiring everyone who develops on autotest to have selenium
installed. This seems excessive. Blacklisting the test from running as
part of unittest_suite.py.

BUG=chromium-os:37989
TEST=Unit test no longer runs as part of utils/unittest_suite.py

Change-Id: I07a7029dbb3b9c65fb57d30113d0fbc6826545d4
Reviewed-on: https://gerrit.chromium.org/gerrit/43220
Reviewed-by: Dan Shi <dshi@chromium.org>
Reviewed-by: Kris Rambish <krisr@chromium.org>
Commit-Queue: Aviv Keshet <akeshet@chromium.org>
Reviewed-by: Aviv Keshet <akeshet@chromium.org>
Tested-by: Aviv Keshet <akeshet@chromium.org>
diff --git a/utils/unittest_suite.py b/utils/unittest_suite.py
index 78a9602..09221c0 100755
--- a/utils/unittest_suite.py
+++ b/utils/unittest_suite.py
@@ -66,9 +66,11 @@
     'logging_manager_test.py',
     ))
 
-# This particular KVM autotest test is not a unittest
+
 SKIP = set((
+    # This particular KVM autotest test is not a unittest
     'guest_test.py',
+    'ap_configurator_test.py'
     ))
 
 LONG_TESTS = (REQUIRES_DJANGO |
@@ -84,7 +86,9 @@
 ROOT = os.path.abspath(os.path.join(os.path.dirname(__file__), '..'))
 
 
-class TestFailure(Exception): pass
+class TestFailure(Exception):
+    """Exception type for any test failure."""
+    pass
 
 
 def run_test(mod_names, options):
@@ -111,6 +115,11 @@
 
 
 def scan_for_modules(start, options):
+    """Scan folders and find all test modules.
+    @param start: The absolute directory to look for tests under.
+    @param options: optparse options.
+    @return a list of modules to be executed.
+    """
     modules = []
 
     skip_tests = SKIP
@@ -185,6 +194,7 @@
 
 
 def main():
+    """Entry point for unittest_suite.py"""
     options, args = parser.parse_args()
     if args:
         options.module_list = args