Upgrade add-trailing-comma to v2.0.1

Addresses asottile/add-trailing-comma#99

.pre-commit-config.yaml

@@ -1,7 +1,7 @@
 ---
 repos:
 - repo: git://github.com/asottile/add-trailing-comma
-  rev: v0.7.1
+  rev: v2.0.1
   hooks:
   - id: add-trailing-comma
 

cheroot/errors.py

@@ -29,7 +29,7 @@ def plat_specific_errors(*errnames):
     depending on the specific platform (OS). This function will return
     the list of numeric values for a given list of potential names.
     """
-    missing_attr = set([None, ])
+    missing_attr = set([None])
     unique_nums = set(getattr(errno, k, None) for k in errnames)
     return list(unique_nums - missing_attr)
 

cheroot/server.py

@@ -1620,12 +1620,16 @@ def clear_stats(self):
                 [w['Work Time'](w) for w in s['Worker Threads'].values()], 0,
             ),
             'Read Throughput': lambda s: (not s['Enabled']) and -1 or sum(
-                [w['Bytes Read'](w) / (w['Work Time'](w) or 1e-6)
-                 for w in s['Worker Threads'].values()], 0,
+                [
+                    w['Bytes Read'](w) / (w['Work Time'](w) or 1e-6)
+                    for w in s['Worker Threads'].values()
+                ], 0,
             ),
             'Write Throughput': lambda s: (not s['Enabled']) and -1 or sum(
-                [w['Bytes Written'](w) / (w['Work Time'](w) or 1e-6)
-                 for w in s['Worker Threads'].values()], 0,
+                [
+                    w['Bytes Written'](w) / (w['Work Time'](w) or 1e-6)
+                    for w in s['Worker Threads'].values()
+                ], 0,
             ),
             'Worker Threads': {},
         }
@@ -2134,7 +2138,9 @@ def get_ssl_adapter_class(name='builtin'):
         try:
             adapter = getattr(mod, attr_name)
         except AttributeError:
-            raise AttributeError("'%s' object has no attribute '%s'"
-                                 % (mod_path, attr_name))
+            raise AttributeError(
+                "'%s' object has no attribute '%s'"
+                % (mod_path, attr_name),
+            )
 
     return adapter

cheroot/test/helper.py

@@ -108,8 +108,10 @@ def assertEqualDates(self, dt1, dt2, seconds=None):
         else:
             diff = dt2 - dt1
         if not diff < datetime.timedelta(seconds=seconds):
-            raise AssertionError('%r and %r are not within %r seconds.' %
-                                 (dt1, dt2, seconds))
+            raise AssertionError(
+                '%r and %r are not within %r seconds.' %
+                (dt1, dt2, seconds),
+            )
 
 
 class Request:
@@ -155,9 +157,13 @@ def __call__(self, environ, start_response):
             resp.status = '404 Not Found'
         else:
             output = handler(req, resp)
-            if (output is not None
-                and not any(resp.status.startswith(status_code)
-                            for status_code in ('204', '304'))):
+            if (
+                output is not None
+                and not any(
+                    resp.status.startswith(status_code)
+                    for status_code in ('204', '304')
+                )
+            ):
                 resp.body = output
                 try:
                     resp.headers.setdefault('Content-Length', str(len(output)))

cheroot/test/test_cli.py

@@ -77,9 +77,9 @@ def main(self):
 @pytest.mark.parametrize(
     ('app_name', 'app_method'),
     (
-        (None, 'application',),
-        ('application', 'application',),
-        ('main', 'main',),
+        (None, 'application'),
+        ('application', 'application'),
+        ('main', 'main'),
     ),
 )
 def test_Aplication_resolve(app_name, app_method, wsgi_app):

cheroot/test/test_conn.py

@@ -45,8 +45,10 @@ def content():
     def upload(req, resp):
         """Process file upload and render thank."""
         if not req.environ['REQUEST_METHOD'] == 'POST':
-            raise AssertionError("'POST' != request.method %r" %
-                                 req.environ['REQUEST_METHOD'])
+            raise AssertionError(
+                "'POST' != request.method %r" %
+                req.environ['REQUEST_METHOD'],
+            )
         return "thanks for '%s'" % req.environ['wsgi.input'].read()
 
     def custom_204(req, resp):
@@ -805,11 +807,13 @@ def test_No_Message_Body(test_client):
 
 
 @pytest.mark.xfail(
-    reason=unwrap(trim("""
+    reason=unwrap(
+        trim("""
         Headers from earlier request leak into the request
         line for a subsequent request, resulting in 400
         instead of 413. See cherrypy/cheroot#69 for details.
-        """)),
+        """),
+    ),
 )
 def test_Chunked_Encoding(test_client):
     """Test HTTP uploads with chunked transfer-encoding."""

cheroot/test/test_ssl.py

@@ -40,7 +40,7 @@
     IS_WINDOWS
     # pylint: disable=unsupported-membership-test
     and b'Microsoft Windows Server 2016 Datacenter' in subprocess.check_output(
-        ('systeminfo', ),
+        ('systeminfo',),
     )
 )
 IS_LIBRESSL_BACKEND = ssl.OPENSSL_VERSION.startswith('LibreSSL')
@@ -149,7 +149,7 @@ def tls_ca_certificate_pem_path(ca):
 def tls_certificate(ca):
     """Provide a leaf certificate via fixture."""
     interface, host, port = _get_conn_data(ANY_INTERFACE_IPV4)
-    return ca.issue_server_cert(ntou(interface), )
+    return ca.issue_server_cert(ntou(interface))
 
 
 @pytest.fixture
@@ -359,7 +359,7 @@ def test_tls_client_auth(
         )
         if not six.PY2:
             if IS_MACOS and IS_PYPY and adapter_type == 'pyopenssl':
-                expected_substrings = ('tlsv1 alert unknown ca', )
+                expected_substrings = ('tlsv1 alert unknown ca',)
         if (
                 tls_verify_mode in (
                     ssl.CERT_REQUIRED,
@@ -433,7 +433,7 @@ def test_ssl_env(
         'idna.core.ulabel',
         return_value=ntob('127.0.0.1'),
     ):
-        client_cert = ca.issue_cert(ntou('127.0.0.1'),)
+        client_cert = ca.issue_cert(ntou('127.0.0.1'))
 
     with client_cert.private_key_and_cert_chain_pem.tempfile() as cl_pem:
         tls_adapter_cls = get_ssl_adapter_class(name=adapter_type)
@@ -612,8 +612,10 @@ def test_http_over_https_error(
         fqdn = '[{}]'.format(fqdn)
 
     expect_fallback_response_over_plain_http = (
-        (adapter_type == 'pyopenssl'
-         and (IS_ABOVE_OPENSSL10 or not six.PY2))
+        (
+            adapter_type == 'pyopenssl'
+            and (IS_ABOVE_OPENSSL10 or not six.PY2)
+        )
         or PY27
     ) or (
         IS_GITHUB_ACTIONS_WORKFLOW

docs/conf.py

@@ -31,7 +31,7 @@
 CONDITIONAL_EXTENSIONS = ()
 
 if IS_SPELLCHECK_BUILD:
-    CONDITIONAL_EXTENSIONS += ('sphinxcontrib.spelling', )
+    CONDITIONAL_EXTENSIONS += ('sphinxcontrib.spelling',)
 
 
 extensions = [

docs/scm_tag_titles_ext.py

@@ -36,7 +36,7 @@ def _get_scm_timestamp_for(commitish, *, scm=None):
 
     try:
         ts = subprocess.check_output(
-            _SCM_COMMANDS[scm] + (commitish, ),
+            _SCM_COMMANDS[scm] + (commitish,),
             stderr=subprocess.DEVNULL,
             text=True,
         ).strip()