Skip to content

Commit 1510dae

Browse files
[PR #9921/e9637a92 backport][3.11] Freeze app for url dispatcher benchmarks (#9923)
**This is a backport of PR #9921 as merged into master (e9637a9).** It better reflects the real usage scenario, runners do app freezing before the start. Co-authored-by: Andrew Svetlov <[email protected]>
1 parent 30a6fbc commit 1510dae

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

tests/test_benchmarks_web_urldispatcher.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,7 @@ async def handler(request: web.Request) -> NoReturn:
4545

4646
app = web.Application()
4747
app.router.add_route("GET", "/", handler)
48+
app.freeze()
4849
request = _mock_request(method="GET", path="/")
4950

5051
async def run_url_dispatcher_benchmark() -> None:
@@ -65,6 +66,7 @@ def test_resolve_static_root_route(
6566

6667
app = web.Application()
6768
app.router.add_static("/", pathlib.Path(aiohttp.__file__).parent)
69+
app.freeze()
6870
request = _mock_request(method="GET", path="/")
6971

7072
async def run_url_dispatcher_benchmark() -> None:
@@ -89,6 +91,7 @@ async def handler(request: web.Request) -> NoReturn:
8991
app = web.Application()
9092
for count in range(250):
9193
app.router.add_route("GET", f"/api/server/dispatch/{count}/update", handler)
94+
app.freeze()
9295
request = _mock_request(method="GET", path="/api/server/dispatch/1/update")
9396

9497
async def run_url_dispatcher_benchmark() -> None:
@@ -112,6 +115,7 @@ async def handler(request: web.Request) -> NoReturn:
112115
app = web.Application()
113116
for count in range(250):
114117
app.router.add_route("GET", f"/api/server/dispatch/{count}/update", handler)
118+
app.freeze()
115119

116120
requests = [
117121
_mock_request(method="GET", path=f"/api/server/dispatch/{count}/update")
@@ -147,6 +151,7 @@ async def handler(request: web.Request) -> NoReturn:
147151
]
148152
for url in urls:
149153
app.router.add_route("GET", url, handler)
154+
app.freeze()
150155

151156
requests = [_mock_request(method="GET", path=url) for url in urls]
152157

@@ -172,6 +177,7 @@ async def handler(request: web.Request) -> NoReturn:
172177
for count in range(250):
173178
app.router.add_route("GET", f"/api/server/other/{count}/update", handler)
174179
app.router.add_route("GET", "/api/server/dispatch/{customer}/update", handler)
180+
app.freeze()
175181

176182
requests = [
177183
_mock_request(method="GET", path=f"/api/server/dispatch/{customer}/update")

0 commit comments

Comments
 (0)