From 3f4d1d4e3953b87625ab5b90d6c2186193f29811 Mon Sep 17 00:00:00 2001 From: Dom Sekotill Date: Thu, 2 May 2024 01:01:41 +0100 Subject: [PATCH] Fix some minor styling problems --- doc/docstring.py | 2 +- kilter/service/options.py | 4 ++-- tests/test_body_accessor.py | 1 - tests/test_header_accessor.py | 3 --- tests/test_options.py | 6 +++--- 5 files changed, 6 insertions(+), 10 deletions(-) diff --git a/doc/docstring.py b/doc/docstring.py index 47ec62a..50ed51b 100644 --- a/doc/docstring.py +++ b/doc/docstring.py @@ -212,7 +212,7 @@ def dot_import_first(module: ModuleType, cls: type|None, name: str) -> tuple[obj if cls is None: raise return ( - (cls, module, f"{module.__name__}.{cls.__name__}") if name == "self" else \ + (cls, module, f"{module.__name__}.{cls.__name__}") if name == "self" else (getattr(cls, name), cls, f"{module.__name__}.{cls.__name__}.{name}") ) diff --git a/kilter/service/options.py b/kilter/service/options.py index 4e8f3ab..9ef417f 100644 --- a/kilter/service/options.py +++ b/kilter/service/options.py @@ -1,4 +1,4 @@ -# Copyright 2023 Dominik Sekotill +# Copyright 2023-2024 Dominik Sekotill # # This Source Code Form is subject to the terms of the Mozilla Public # License, v. 2.0. If a copy of the MPL was not distributed with this @@ -194,7 +194,7 @@ def examine_recipients( can_respond: bool = False, can_add: bool = False, can_remove: bool = False, - include_rejected: bool= False, + include_rejected: bool = False, with_parameters: bool = False, ) -> Decorator: """ diff --git a/tests/test_body_accessor.py b/tests/test_body_accessor.py index 51d2f0f..c82d925 100644 --- a/tests/test_body_accessor.py +++ b/tests/test_body_accessor.py @@ -65,7 +65,6 @@ class HeaderAccessorTests(AsyncTestCase): async for chunk in body: result2 += chunk - async with trio.open_nursery() as tg: tg.start_soon(test_filter) await trio.testing.wait_all_tasks_blocked() diff --git a/tests/test_header_accessor.py b/tests/test_header_accessor.py index afaca39..03b3396 100644 --- a/tests/test_header_accessor.py +++ b/tests/test_header_accessor.py @@ -67,7 +67,6 @@ class HeaderAccessorTests(AsyncTestCase): async for header in headers: result2.append(header.name) - async with trio.open_nursery() as tg: tg.start_soon(test_filter) await trio.testing.wait_all_tasks_blocked() @@ -100,7 +99,6 @@ class HeaderAccessorTests(AsyncTestCase): assert session.phase == Phase.BODY result.append(header.name) - async with trio.open_nursery() as tg: tg.start_soon(test_filter) await trio.testing.wait_all_tasks_blocked() @@ -127,7 +125,6 @@ class HeaderAccessorTests(AsyncTestCase): assert session.phase == Phase.BODY result.append(header.name) - async with trio.open_nursery() as tg: tg.start_soon(test_filter) await trio.testing.wait_all_tasks_blocked() diff --git a/tests/test_options.py b/tests/test_options.py index a0c6d6a..e142a6d 100644 --- a/tests/test_options.py +++ b/tests/test_options.py @@ -108,11 +108,11 @@ class Tests(TestCase): flags = options.get_flags(filtr) assert flags.set_options == SKIP|NR_CONNECT, \ - f"{flags.set_options!r} is not SKIP" + f"{flags.set_options!r} is not SKIP" assert flags.unset_options == HEADER_LEADING_SPACE|SKIP, \ - f"{flags.unset_options!r} is not HEADER_LEADING_SPACE" + f"{flags.unset_options!r} is not HEADER_LEADING_SPACE" assert flags.set_actions == ADD_HEADERS|CHANGE_HEADERS, \ - f"{flags.set_actions!r} is not ADD_HEADERS" + f"{flags.set_actions!r} is not ADD_HEADERS" def test_responds_to_connect(self) -> None: """ -- GitLab