Run black -l 79 against all source files

This is a straight reformat run using black as is, with no edits
applied at all.

The black run will format code consistently, however in
some cases that are prevalent in SQLAlchemy code it produces
too-long lines.   The too-long lines will be resolved in the
following commit that will resolve all remaining flake8 issues
including shadowed builtins, long lines, import order, unused
imports, duplicate imports, and docstring issues.

Change-Id: I7eda77fed3d8e73df84b3651fd6cfcfe858d4dc9

1 files changed, 65 insertions, 39 deletions

diff --git a/lib/sqlalchemy/testing/plugin/pytestplugin.py b/lib/sqlalchemy/testing/plugin/pytestplugin.py
index da682ea00..fd0a48462 100644
--- a/lib/sqlalchemy/testing/plugin/pytestplugin.py
+++ b/lib/sqlalchemy/testing/plugin/pytestplugin.py
@@ -13,6 +13,7 @@ import os
 
 try:
     import xdist  # noqa
+
     has_xdist = True
 except ImportError:
     has_xdist = False
@@ -24,30 +25,42 @@ def pytest_addoption(parser):
     def make_option(name, **kw):
         callback_ = kw.pop("callback", None)
         if callback_:
+
             class CallableAction(argparse.Action):
-                def __call__(self, parser, namespace,
-                             values, option_string=None):
+                def __call__(
+                    self, parser, namespace, values, option_string=None
+                ):
                     callback_(option_string, values, parser)
+
             kw["action"] = CallableAction
 
         zeroarg_callback = kw.pop("zeroarg_callback", None)
         if zeroarg_callback:
+
             class CallableAction(argparse.Action):
-                def __init__(self, option_strings,
-                             dest, default=False,
-                             required=False, help=None):
-                        super(CallableAction, self).__init__(
-                            option_strings=option_strings,
-                            dest=dest,
-                            nargs=0,
-                            const=True,
-                            default=default,
-                            required=required,
-                            help=help)
-
-                def __call__(self, parser, namespace,
-                             values, option_string=None):
+                def __init__(
+                    self,
+                    option_strings,
+                    dest,
+                    default=False,
+                    required=False,
+                    help=None,
+                ):
+                    super(CallableAction, self).__init__(
+                        option_strings=option_strings,
+                        dest=dest,
+                        nargs=0,
+                        const=True,
+                        default=default,
+                        required=required,
+                        help=help,
+                    )
+
+                def __call__(
+                    self, parser, namespace, values, option_string=None
+                ):
                     zeroarg_callback(option_string, values, parser)
+
             kw["action"] = CallableAction
 
         group.addoption(name, **kw)
@@ -59,18 +72,18 @@ def pytest_addoption(parser):
 def pytest_configure(config):
     if hasattr(config, "slaveinput"):
         plugin_base.restore_important_follower_config(config.slaveinput)
-        plugin_base.configure_follower(
-            config.slaveinput["follower_ident"]
-        )
+        plugin_base.configure_follower(config.slaveinput["follower_ident"])
     else:
-        if config.option.write_idents and \
-                os.path.exists(config.option.write_idents):
+        if config.option.write_idents and os.path.exists(
+            config.option.write_idents
+        ):
             os.remove(config.option.write_idents)
 
     plugin_base.pre_begin(config.option)
 
-    plugin_base.set_coverage_flag(bool(getattr(config.option,
-                                               "cov_source", False)))
+    plugin_base.set_coverage_flag(
+        bool(getattr(config.option, "cov_source", False))
+    )
 
     plugin_base.set_skip_test(pytest.skip.Exception)
 
@@ -94,10 +107,12 @@ if has_xdist:
 
         node.slaveinput["follower_ident"] = "test_%s" % uuid.uuid4().hex[0:12]
         from sqlalchemy.testing import provision
+
         provision.create_follower_db(node.slaveinput["follower_ident"])
 
     def pytest_testnodedown(node, error):
         from sqlalchemy.testing import provision
+
         provision.drop_follower_db(node.slaveinput["follower_ident"])
 
 
@@ -114,19 +129,22 @@ def pytest_collection_modifyitems(session, config, items):
 
     rebuilt_items = collections.defaultdict(list)
     items[:] = [
-        item for item in
-        items if isinstance(item.parent, pytest.Instance)
-        and not item.parent.parent.name.startswith("_")]
+        item
+        for item in items
+        if isinstance(item.parent, pytest.Instance)
+        and not item.parent.parent.name.startswith("_")
+    ]
     test_classes = set(item.parent for item in items)
     for test_class in test_classes:
         for sub_cls in plugin_base.generate_sub_tests(
-                test_class.cls, test_class.parent.module):
+            test_class.cls, test_class.parent.module
+        ):
             if sub_cls is not test_class.cls:
                 list_ = rebuilt_items[test_class.cls]
 
                 for inst in pytest.Class(
-                        sub_cls.__name__,
-                        parent=test_class.parent.parent).collect():
+                    sub_cls.__name__, parent=test_class.parent.parent
+                ).collect():
                     list_.extend(inst.collect())
 
     newitems = []
@@ -139,23 +157,29 @@ def pytest_collection_modifyitems(session, config, items):
 
     # seems like the functions attached to a test class aren't sorted already?
     # is that true and why's that? (when using unittest, they're sorted)
-    items[:] = sorted(newitems, key=lambda item: (
-        item.parent.parent.parent.name,
-        item.parent.parent.name,
-        item.name
-    ))
+    items[:] = sorted(
+        newitems,
+        key=lambda item: (
+            item.parent.parent.parent.name,
+            item.parent.parent.name,
+            item.name,
+        ),
+    )
 
 
 def pytest_pycollect_makeitem(collector, name, obj):
     if inspect.isclass(obj) and plugin_base.want_class(obj):
         return pytest.Class(name, parent=collector)
-    elif inspect.isfunction(obj) and \
-            isinstance(collector, pytest.Instance) and \
-            plugin_base.want_method(collector.cls, obj):
+    elif (
+        inspect.isfunction(obj)
+        and isinstance(collector, pytest.Instance)
+        and plugin_base.want_method(collector.cls, obj)
+    ):
         return pytest.Function(name, parent=collector)
     else:
         return []
 
+
 _current_class = None
 
 
@@ -180,6 +204,7 @@ def pytest_runtest_setup(item):
             global _current_class
             class_teardown(item.parent.parent)
             _current_class = None
+
         item.parent.parent.addfinalizer(finalize)
 
     test_setup(item)
@@ -194,8 +219,9 @@ def pytest_runtest_teardown(item):
 
 
 def test_setup(item):
-    plugin_base.before_test(item, item.parent.module.__name__,
-                            item.parent.cls, item.name)
+    plugin_base.before_test(
+        item, item.parent.module.__name__, item.parent.cls, item.name
+    )
 
 
 def test_teardown(item):