[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

tests/test_cookiejar.py

@@ -177,9 +177,7 @@ async def test_constructor_with_expired(
     assert jar._loop is loop
 
 
-async def test_save_load(
-    tmp_path, loop, cookies_to_send, cookies_to_receive
-) -> None:
+async def test_save_load(tmp_path, loop, cookies_to_send, cookies_to_receive) -> None:
     file_path = pathlib.Path(str(tmp_path)) / "aiohttp.test.cookie"
 
     # export cookie jar

tests/test_helpers.py

@@ -75,16 +75,16 @@ def test_parse_mimetype(mimetype, expected) -> None:
 def test_guess_filename_with_file_object(tmp_path) -> None:
     file_path = tmp_path / "test_guess_filename"
     with file_path.open("w+b") as fp:
-        assert (helpers.guess_filename(fp, "no-throw") is not None)
+        assert helpers.guess_filename(fp, "no-throw") is not None
 
 
 def test_guess_filename_with_path(tmp_path) -> None:
     file_path = tmp_path / "test_guess_filename"
-    assert (helpers.guess_filename(file_path, "no-throw") is not None)
+    assert helpers.guess_filename(file_path, "no-throw") is not None
 
 
 def test_guess_filename_with_default() -> None:
-    assert (helpers.guess_filename(None, "no-throw") == "no-throw")
+    assert helpers.guess_filename(None, "no-throw") == "no-throw"
 
 
 # ------------------- BasicAuth -----------------------------------

tests/test_urldispatch.py

@@ -48,8 +48,7 @@ def fill_routes(router):
     def go():
         route1 = router.add_route("GET", "/plain", make_handler())
         route2 = router.add_route("GET", "/variable/{name}", make_handler())
-        resource = router.add_static("/static",
-                                     pathlib.Path(aiohttp.__file__).parent)
+        resource = router.add_static("/static", pathlib.Path(aiohttp.__file__).parent)
         return [route1, route2] + list(resource)
 
     return go
@@ -341,19 +340,17 @@ def test_route_dynamic(router) -> None:
 
 
 def test_add_static(router) -> None:
-    resource = router.add_static("/st",
-                                 pathlib.Path(aiohttp.__file__).parent,
-                                 name="static")
+    resource = router.add_static(
+        "/st", pathlib.Path(aiohttp.__file__).parent, name="static"
+    )
     assert router["static"] is resource
     url = resource.url_for(filename="/dir/a.txt")
     assert "/st/dir/a.txt" == str(url)
     assert len(resource) == 2
 
 
 def test_add_static_append_version(router) -> None:
-    resource = router.add_static("/st",
-                                 pathlib.Path(__file__).parent,
-                                 name="static")
+    resource = router.add_static("/st", pathlib.Path(__file__).parent, name="static")
     url = resource.url_for(filename="/data.unknown_mime_type", append_version=True)
     expect_url = (
         "/st/data.unknown_mime_type?" "v=aUsn8CHEhhszc81d28QmlcBW0KQpfS2F4trgQKhOYd8%3D"
@@ -362,10 +359,9 @@ def test_add_static_append_version(router) -> None:
 
 
 def test_add_static_append_version_set_from_constructor(router) -> None:
-    resource = router.add_static("/st",
-                                 pathlib.Path(__file__).parent,
-                                 append_version=True,
-                                 name="static")
+    resource = router.add_static(
+        "/st", pathlib.Path(__file__).parent, append_version=True, name="static"
+    )
     url = resource.url_for(filename="/data.unknown_mime_type")
     expect_url = (
         "/st/data.unknown_mime_type?" "v=aUsn8CHEhhszc81d28QmlcBW0KQpfS2F4trgQKhOYd8%3D"
@@ -374,19 +370,16 @@ def test_add_static_append_version_set_from_constructor(router) -> None:
 
 
 def test_add_static_append_version_override_constructor(router) -> None:
-    resource = router.add_static("/st",
-                                 pathlib.Path(__file__).parent,
-                                 append_version=True,
-                                 name="static")
+    resource = router.add_static(
+        "/st", pathlib.Path(__file__).parent, append_version=True, name="static"
+    )
     url = resource.url_for(filename="/data.unknown_mime_type", append_version=False)
     expect_url = "/st/data.unknown_mime_type"
     assert expect_url == str(url)
 
 
 def test_add_static_append_version_filename_without_slash(router) -> None:
-    resource = router.add_static("/st",
-                                 pathlib.Path(__file__).parent,
-                                 name="static")
+    resource = router.add_static("/st", pathlib.Path(__file__).parent, name="static")
     url = resource.url_for(filename="data.unknown_mime_type", append_version=True)
     expect_url = (
         "/st/data.unknown_mime_type?" "v=aUsn8CHEhhszc81d28QmlcBW0KQpfS2F4trgQKhOYd8%3D"
@@ -395,17 +388,13 @@ def test_add_static_append_version_filename_without_slash(router) -> None:
 
 
 def test_add_static_append_version_non_exists_file(router) -> None:
-    resource = router.add_static("/st",
-                                 pathlib.Path(__file__).parent,
-                                 name="static")
+    resource = router.add_static("/st", pathlib.Path(__file__).parent, name="static")
     url = resource.url_for(filename="/non_exists_file", append_version=True)
     assert "/st/non_exists_file" == str(url)
 
 
 def test_add_static_append_version_non_exists_file_without_slash(router) -> None:
-    resource = router.add_static("/st",
-                                 pathlib.Path(__file__).parent,
-                                 name="static")
+    resource = router.add_static("/st", pathlib.Path(__file__).parent, name="static")
     url = resource.url_for(filename="non_exists_file", append_version=True)
     assert "/st/non_exists_file" == str(url)
 
@@ -417,8 +406,9 @@ def test_add_static_append_version_follow_symlink(router, tmp_path) -> None:
     pathlib.Path(str(symlink_path)).symlink_to(str(symlink_target_path), True)
 
     # Register global static route:
-    resource = router.add_static("/st", str(tmp_path), follow_symlinks=True,
-                                 append_version=True)
+    resource = router.add_static(
+        "/st", str(tmp_path), follow_symlinks=True, append_version=True
+    )
 
     url = resource.url_for(filename="/append_version_symlink/data.unknown_mime_type")
 
@@ -429,17 +419,17 @@ def test_add_static_append_version_follow_symlink(router, tmp_path) -> None:
     assert expect_url == str(url)
 
 
-def test_add_static_append_version_not_follow_symlink(router,
-                                                      tmp_path) -> None:
+def test_add_static_append_version_not_follow_symlink(router, tmp_path) -> None:
     # Tests the access to a symlink, in static folder with apeend_version
-    symlink_path = tmp_path / 'append_version_symlink'
+    symlink_path = tmp_path / "append_version_symlink"
     symlink_target_path = pathlib.Path(__file__).parent
 
     pathlib.Path(str(symlink_path)).symlink_to(str(symlink_target_path), True)
 
     # Register global static route:
-    resource = router.add_static("/st", str(tmp_path), follow_symlinks=False,
-                                 append_version=True)
+    resource = router.add_static(
+        "/st", str(tmp_path), follow_symlinks=False, append_version=True
+    )
 
     filename = "/append_version_symlink/data.unknown_mime_type"
     url = resource.url_for(filename=filename)
@@ -475,8 +465,7 @@ def test_dynamic_not_match(router) -> None:
 
 
 async def test_static_not_match(router) -> None:
-    router.add_static("/pre", pathlib.Path(aiohttp.__file__).parent,
-                      name="name")
+    router.add_static("/pre", pathlib.Path(aiohttp.__file__).parent, name="name")
     resource = router["name"]
     ret = await resource.resolve(make_mocked_request("GET", "/another/path"))
     assert (None, set()) == ret
@@ -512,20 +501,17 @@ def test_contains(router) -> None:
 
 
 def test_static_repr(router) -> None:
-    router.add_static("/get", pathlib.Path(aiohttp.__file__).parent,
-                      name="name")
+    router.add_static("/get", pathlib.Path(aiohttp.__file__).parent, name="name")
     assert Matches(r"<StaticResource 'name' /get") == repr(router["name"])
 
 
 def test_static_adds_slash(router) -> None:
-    route = router.add_static("/prefix",
-                              pathlib.Path(aiohttp.__file__).parent)
+    route = router.add_static("/prefix", pathlib.Path(aiohttp.__file__).parent)
     assert "/prefix" == route._prefix
 
 
 def test_static_remove_trailing_slash(router) -> None:
-    route = router.add_static("/prefix/",
-                              pathlib.Path(aiohttp.__file__).parent)
+    route = router.add_static("/prefix/", pathlib.Path(aiohttp.__file__).parent)
     assert "/prefix" == route._prefix
 
 
@@ -789,9 +775,9 @@ def test_named_resources_abc(router) -> None:
 def test_named_resources(router) -> None:
     route1 = router.add_route("GET", "/plain", make_handler(), name="route1")
     route2 = router.add_route("GET", "/variable/{name}", make_handler(), name="route2")
-    route3 = router.add_static("/static",
-                               pathlib.Path(aiohttp.__file__).parent,
-                               name="route3")
+    route3 = router.add_static(
+        "/static", pathlib.Path(aiohttp.__file__).parent, name="route3"
+    )
     names = {route1.name, route2.name, route3.name}
 
     assert 3 == len(router.named_resources())
@@ -970,15 +956,13 @@ def test_static_route_points_to_file(router) -> None:
 
 
 async def test_404_for_static_resource(router) -> None:
-    resource = router.add_static("/st",
-                                 pathlib.Path(aiohttp.__file__).parent)
+    resource = router.add_static("/st", pathlib.Path(aiohttp.__file__).parent)
     ret = await resource.resolve(make_mocked_request("GET", "/unknown/path"))
     assert (None, set()) == ret
 
 
 async def test_405_for_resource_adapter(router) -> None:
-    resource = router.add_static("/st",
-                                 pathlib.Path(aiohttp.__file__).parent)
+    resource = router.add_static("/st", pathlib.Path(aiohttp.__file__).parent)
     ret = await resource.resolve(make_mocked_request("POST", "/st/abc.py"))
     assert (None, {"HEAD", "GET"}) == ret
 
@@ -993,14 +977,12 @@ async def test_check_allowed_method_for_found_resource(router) -> None:
 
 
 def test_url_for_in_static_resource(router) -> None:
-    resource = router.add_static("/static",
-                                 pathlib.Path(aiohttp.__file__).parent)
+    resource = router.add_static("/static", pathlib.Path(aiohttp.__file__).parent)
     assert URL("/static/file.txt") == resource.url_for(filename="file.txt")
 
 
 def test_url_for_in_static_resource_pathlib(router) -> None:
-    resource = router.add_static("/static",
-                                 pathlib.Path(aiohttp.__file__).parent)
+    resource = router.add_static("/static", pathlib.Path(aiohttp.__file__).parent)
     assert URL("/static/file.txt") == resource.url_for(
         filename=pathlib.Path("file.txt")
     )
@@ -1179,8 +1161,7 @@ def test_frozen_app_on_subapp(app) -> None:
 
 
 def test_set_options_route(router) -> None:
-    resource = router.add_static("/static",
-                                 pathlib.Path(aiohttp.__file__).parent)
+    resource = router.add_static("/static", pathlib.Path(aiohttp.__file__).parent)
     options = None
     for route in resource:
         if route.method == "OPTIONS":

tests/test_web_sendfile_functional.py

@@ -563,7 +563,7 @@ async def test_static_file_huge(aiohttp_client, tmp_path) -> None:
 
     # fill 20MB file
     with file_path.open("wb") as f:
-        for i in range(1024*20):
+        for i in range(1024 * 20):
             f.write((chr(i % 64 + 0x20) * 1024).encode())
 
     file_st = file_path.stat()