build(deps): bump black from 19.10b0 to 20.8b1 (getsentry#801)

dependabot-preview[bot] · untitaker · sentry-bot · web-flow · commit c5b0098a5faf · 2020-08-29T21:24:40.000+02:00
* build(deps): bump black from 19.10b0 to 20.8b1 Bumps [black](https://github.com/psf/black) from 19.10b0 to 20.8b1. - [Release notes](https://github.com/psf/black/releases) - [Changelog](https://github.com/psf/black/blob/master/CHANGES.md) - [Commits](https://github.com/psf/black/commits) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> * add black action * always run black action, its a python-only project * attempt push * fix: Formatting Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com> Co-authored-by: Markus Unterwaditzer <markus-honeypot@unterwaditzer.net> Co-authored-by: sentry-bot <markus+ghbot@sentry.io>
diff --git a/.github/workflows/black.yml b/.github/workflows/black.yml
@@ -0,0 +1,25 @@
+name: black
+
+on: push
+
+jobs:
+  format:
+    runs-on: ubuntu-16.04
+    steps:
+      - uses: actions/checkout@v2
+      - uses: actions/setup-python@v2
+        with:
+          python-version: '3.x'
+
+      - name: Install Black
+        run: pip install -r linter-requirements.txt
+
+      - name: Run Black
+        run: black tests examples sentry_sdk
+
+      - name: Commit changes
+        run: |
+          git config --global user.name 'sentry-bot'
+          git config --global user.email 'markus+ghbot@sentry.io'
+          git commit -am "fix: Formatting"
+          git push
diff --git a/linter-requirements.txt b/linter-requirements.txt
@@ -1,4 +1,4 @@
-black==19.10b0
+black==20.8b1
 flake8==3.8.3
 flake8-import-order==0.18.1
 mypy==0.782
diff --git a/sentry_sdk/_functools.py b/sentry_sdk/_functools.py
@@ -28,14 +28,14 @@ def update_wrapper(
     # type: (Any, Any, Any, Any) -> Any
     """Update a wrapper function to look like the wrapped function
 
-       wrapper is the function to be updated
-       wrapped is the original function
-       assigned is a tuple naming the attributes assigned directly
-       from the wrapped function to the wrapper function (defaults to
-       functools.WRAPPER_ASSIGNMENTS)
-       updated is a tuple naming the attributes of the wrapper that
-       are updated with the corresponding attribute from the wrapped
-       function (defaults to functools.WRAPPER_UPDATES)
+    wrapper is the function to be updated
+    wrapped is the original function
+    assigned is a tuple naming the attributes assigned directly
+    from the wrapped function to the wrapper function (defaults to
+    functools.WRAPPER_ASSIGNMENTS)
+    updated is a tuple naming the attributes of the wrapper that
+    are updated with the corresponding attribute from the wrapped
+    function (defaults to functools.WRAPPER_UPDATES)
     """
     for attr in assigned:
         try:
@@ -57,10 +57,10 @@ def wraps(wrapped, assigned=WRAPPER_ASSIGNMENTS, updated=WRAPPER_UPDATES):
     # type: (Callable[..., Any], Any, Any) -> Callable[[Callable[..., Any]], Callable[..., Any]]
     """Decorator factory to apply update_wrapper() to a wrapper function
 
-       Returns a decorator that invokes update_wrapper() with the decorated
-       function as the wrapper argument and the arguments to wraps() as the
-       remaining arguments. Default arguments are as for update_wrapper().
-       This is a convenience function to simplify applying partial() to
-       update_wrapper().
+    Returns a decorator that invokes update_wrapper() with the decorated
+    function as the wrapper argument and the arguments to wraps() as the
+    remaining arguments. Default arguments are as for update_wrapper().
+    This is a convenience function to simplify applying partial() to
+    update_wrapper().
     """
     return partial(update_wrapper, wrapped=wrapped, assigned=assigned, updated=updated)
diff --git a/sentry_sdk/hub.py b/sentry_sdk/hub.py
@@ -315,8 +315,7 @@ def capture_event(
         **scope_args  # type: Dict[str, Any]
     ):
         # type: (...) -> Optional[str]
-        """Captures an event. Alias of :py:meth:`sentry_sdk.Client.capture_event`.
-        """
+        """Captures an event. Alias of :py:meth:`sentry_sdk.Client.capture_event`."""
         client, top_scope = self._stack[-1]
         scope = _update_scope(top_scope, scope, scope_args)
         if client is not None:
diff --git a/sentry_sdk/integrations/__init__.py b/sentry_sdk/integrations/__init__.py
@@ -27,8 +27,7 @@ def _generate_default_integrations_iterator(integrations, auto_enabling_integrat
 
     def iter_default_integrations(with_auto_enabling_integrations):
         # type: (bool) -> Iterator[Type[Integration]]
-        """Returns an iterator of the default integration classes:
-        """
+        """Returns an iterator of the default integration classes:"""
         from importlib import import_module
 
         if with_auto_enabling_integrations:
diff --git a/sentry_sdk/integrations/asgi.py b/sentry_sdk/integrations/asgi.py
@@ -124,7 +124,8 @@ async def _run_app(self, scope, callback):
 
                 if ty in ("http", "websocket"):
                     transaction = Transaction.continue_from_headers(
-                        dict(scope["headers"]), op="{}.server".format(ty),
+                        dict(scope["headers"]),
+                        op="{}.server".format(ty),
                     )
                 else:
                     transaction = Transaction(op="asgi.server")
diff --git a/sentry_sdk/integrations/aws_lambda.py b/sentry_sdk/integrations/aws_lambda.py
@@ -227,11 +227,15 @@ def inner(*args, **kwargs):
 
                 return inner  # type: ignore
 
-            lambda_bootstrap.LambdaRuntimeClient.post_invocation_result = _wrap_post_function(
-                lambda_bootstrap.LambdaRuntimeClient.post_invocation_result
+            lambda_bootstrap.LambdaRuntimeClient.post_invocation_result = (
+                _wrap_post_function(
+                    lambda_bootstrap.LambdaRuntimeClient.post_invocation_result
+                )
             )
-            lambda_bootstrap.LambdaRuntimeClient.post_invocation_error = _wrap_post_function(
-                lambda_bootstrap.LambdaRuntimeClient.post_invocation_error
+            lambda_bootstrap.LambdaRuntimeClient.post_invocation_error = (
+                _wrap_post_function(
+                    lambda_bootstrap.LambdaRuntimeClient.post_invocation_error
+                )
             )
 
 
diff --git a/sentry_sdk/integrations/excepthook.py b/sentry_sdk/integrations/excepthook.py
@@ -14,7 +14,8 @@
     from types import TracebackType
 
     Excepthook = Callable[
-        [Type[BaseException], BaseException, TracebackType], Any,
+        [Type[BaseException], BaseException, TracebackType],
+        Any,
     ]
 
 
diff --git a/sentry_sdk/utils.py b/sentry_sdk/utils.py
@@ -883,7 +883,7 @@ class ServerlessTimeoutWarning(Exception):
 
 class TimeoutThread(threading.Thread):
     """Creates a Thread which runs (sleeps) for a time duration equal to
-       waiting_time and raises a custom ServerlessTimeout exception.
+    waiting_time and raises a custom ServerlessTimeout exception.
     """
 
     def __init__(self, waiting_time, configured_timeout):
diff --git a/tests/integrations/flask/test_flask.py b/tests/integrations/flask/test_flask.py
@@ -247,7 +247,9 @@ def test_flask_session_tracking(sentry_init, capture_envelopes, app):
     sentry_init(
         integrations=[flask_sentry.FlaskIntegration()],
         release="demo-release",
-        _experiments=dict(auto_session_tracking=True,),
+        _experiments=dict(
+            auto_session_tracking=True,
+        ),
     )
 
     @app.route("/")
diff --git a/tests/integrations/gcp/test_gcp.py b/tests/integrations/gcp/test_gcp.py
@@ -204,8 +204,10 @@ def inner(code, timeout="10s", subprocess_kwargs=()):
         function_call_response = api_request.execute()
 
         # STEP : Fetch logs of invoked function
-        log_name = "projects/{}/logs/cloudfunctions.googleapis.com%2Fcloud-functions".format(
-            project_id
+        log_name = (
+            "projects/{}/logs/cloudfunctions.googleapis.com%2Fcloud-functions".format(
+                project_id
+            )
         )
         project_name = "projects/{}".format(project_id)
         body = {"resourceNames": [project_name], "filter": log_name}
diff --git a/tests/integrations/logging/test_logging.py b/tests/integrations/logging/test_logging.py
@@ -80,7 +80,10 @@ def test_logging_stack(sentry_init, capture_events):
     logger.error("first", exc_info=True)
     logger.error("second")
 
-    event_with, event_without, = events
+    (
+        event_with,
+        event_without,
+    ) = events
 
     assert event_with["level"] == "error"
     assert event_with["threads"]["values"][0]["stacktrace"]["frames"]
diff --git a/tests/integrations/stdlib/test_subprocess.py b/tests/integrations/stdlib/test_subprocess.py
@@ -118,7 +118,10 @@ def test_subprocess_basic(
 
     capture_message("hi")
 
-    transaction_event, message_event, = events
+    (
+        transaction_event,
+        message_event,
+    ) = events
 
     assert message_event["message"] == "hi"
 
diff --git a/tests/test_transport.py b/tests/test_transport.py
@@ -168,7 +168,9 @@ def test_complex_limits_without_data_category(
         dict(dsn="http://foobar@{}/123".format(httpserver.url[len("http://") :]))
     )
     httpserver.serve_content(
-        "hm", response_code, headers={"X-Sentry-Rate-Limits": "4711::organization"},
+        "hm",
+        response_code,
+        headers={"X-Sentry-Rate-Limits": "4711::organization"},
     )
 
     client.capture_event({"type": "transaction"})
diff --git a/tests/utils/test_general.py b/tests/utils/test_general.py
@@ -128,32 +128,44 @@ def test_parse_invalid_dsn(dsn):
 
 @pytest.mark.parametrize("empty", [None, []])
 def test_in_app(empty):
-    assert handle_in_app_impl(
-        [{"module": "foo"}, {"module": "bar"}],
-        in_app_include=["foo"],
-        in_app_exclude=empty,
-    ) == [{"module": "foo", "in_app": True}, {"module": "bar"}]
-
-    assert handle_in_app_impl(
-        [{"module": "foo"}, {"module": "bar"}],
-        in_app_include=["foo"],
-        in_app_exclude=["foo"],
-    ) == [{"module": "foo", "in_app": True}, {"module": "bar"}]
-
-    assert handle_in_app_impl(
-        [{"module": "foo"}, {"module": "bar"}],
-        in_app_include=empty,
-        in_app_exclude=["foo"],
-    ) == [{"module": "foo", "in_app": False}, {"module": "bar", "in_app": True}]
+    assert (
+        handle_in_app_impl(
+            [{"module": "foo"}, {"module": "bar"}],
+            in_app_include=["foo"],
+            in_app_exclude=empty,
+        )
+        == [{"module": "foo", "in_app": True}, {"module": "bar"}]
+    )
+
+    assert (
+        handle_in_app_impl(
+            [{"module": "foo"}, {"module": "bar"}],
+            in_app_include=["foo"],
+            in_app_exclude=["foo"],
+        )
+        == [{"module": "foo", "in_app": True}, {"module": "bar"}]
+    )
+
+    assert (
+        handle_in_app_impl(
+            [{"module": "foo"}, {"module": "bar"}],
+            in_app_include=empty,
+            in_app_exclude=["foo"],
+        )
+        == [{"module": "foo", "in_app": False}, {"module": "bar", "in_app": True}]
+    )
 
 
 def test_iter_stacktraces():
-    assert set(
-        iter_event_stacktraces(
-            {
-                "threads": {"values": [{"stacktrace": 1}]},
-                "stacktrace": 2,
-                "exception": {"values": [{"stacktrace": 3}]},
-            }
+    assert (
+        set(
+            iter_event_stacktraces(
+                {
+                    "threads": {"values": [{"stacktrace": 1}]},
+                    "stacktrace": 2,
+                    "exception": {"values": [{"stacktrace": 3}]},
+                }
+            )
         )
-    ) == {1, 2, 3}
+        == {1, 2, 3}
+    )

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-black==19.10b0`
	`1`	`+black==20.8b1`
`2`	`2`	`flake8==3.8.3`
`3`	`3`	`flake8-import-order==0.18.1`
`4`	`4`	`mypy==0.782`
Original file line number	Diff line number	Diff line change
`@@ -124,7 +124,8 @@ async def _run_app(self, scope, callback):`
`124`	`124`
`125`	`125`	`if ty in ("http", "websocket"):`
`126`	`126`	`transaction = Transaction.continue_from_headers(`
`127`		`- dict(scope["headers"]), op="{}.server".format(ty),`
	`127`	`+ dict(scope["headers"]),`
	`128`	`+ op="{}.server".format(ty),`
`128`	`129`	`)`
`129`	`130`	`else:`
`130`	`131`	`transaction = Transaction(op="asgi.server")`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,8 @@`
`14`	`14`	`from types import TracebackType`
`15`	`15`
`16`	`16`	`Excepthook = Callable[`
`17`		`- [Type[BaseException], BaseException, TracebackType], Any,`
	`17`	`+ [Type[BaseException], BaseException, TracebackType],`
	`18`	`+ Any,`
`18`	`19`	`]`
`19`	`20`
`20`	`21`