Standardize skip_files to test_.+; rename support to test_support.

appengine/auth_service/tests/frontend_handlers_test.py

 #!/usr/bin/env python
 # Copyright 2014 The Swarming Authors. All rights reserved.
 # Use of this source code is governed by the Apache v2.0 license that can be
 # found in the LICENSE file.
 
 import logging
 import os
 import sys
 import unittest
 
 APP_DIR = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
 
 import test_env
 test_env.setup_test_env()
 
 # From components/third_party/
 import webtest
 
 from components import auth_testing
 from components import template
-
-from support import test_case
+from test_support import test_case
 
 from common import importer
 from common import replication
-

[vadimsh, 2015/03/19 06:01:01]

eh, I like new lines between groups of imports, you don't. We'll keep making
this bikesheddings over and over.

[M-A, 2015/03/19 14:50:10]

This is because this confuses the difference between
stdlib/externallib/locallib/. Normally, you'd have:

import sys  # stdlib
...

from components import bar  # externallib
...

from server import foo  # locallib
...

For tests, this muddy up because of the test setup. But when externallib has
multiple subgroups, each become a separated with the same separator (an empty
line) than separating the 3 overall sections.

But yes it is bikeshedding, so I odn't mind too much as long as they are all the
same.

 from frontend import handlers
 
 
 GOOD_IMPORTER_CONFIG = [
   {
     'domain': 'example.com',
     'systems': ['ldap'],
     'url': 'http://example.com/stuff.tar.gz',
   },
   {
@@ skipping 63 matching lines @@
     self.assertEqual([], importer.read_config())
 
 
 if __name__ == '__main__':
   if '-v' in sys.argv:
     unittest.TestCase.maxDiff = None
     logging.basicConfig(level=logging.DEBUG)
   else:
     logging.basicConfig(level=logging.FATAL)
   unittest.main()