From ec6423cba5a1935eff28eb16945a726917a884b4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Luk=C3=A1=C5=A1=20Kucharczyk?= Date: Sat, 20 Jun 2026 21:33:25 +0200 Subject: [PATCH] style(session): apply ruff format to issue #53 changes Co-Authored-By: Claude Opus 4.8 --- games/views/session.py | 15 ++++----------- tests/test_rendered_pages.py | 2 +- tests/test_session_endpoints.py | 4 +--- 3 files changed, 6 insertions(+), 15 deletions(-) diff --git a/games/views/session.py b/games/views/session.py index ea01fa2..3fa7724 100644 --- a/games/views/session.py +++ b/games/views/session.py @@ -49,16 +49,12 @@ class SessionRowData(TypedDict): cell_data: list[Node] -def session_row_data( - session: Session, device_list, csrf_token: str -) -> SessionRowData: +def session_row_data(session: Session, device_list, csrf_token: str) -> SessionRowData: """Canonical session-list row. Single source of truth shared by list_sessions and the htmx finish/reset fragments.""" row_selector = f"#session-row-{session.pk}" end_url = reverse("games:list_sessions_end_session", args=[session.pk]) - reset_url = reverse( - "games:list_sessions_reset_session_start", args=[session.pk] - ) + reset_url = reverse("games:list_sessions_reset_session_start", args=[session.pk]) actions = ButtonGroup( [ { @@ -202,8 +198,7 @@ def list_sessions(request: HttpRequest, search_string: str = "") -> HttpResponse "Actions", ], "rows": [ - session_row_data(session, device_list, csrf_token) - for session in sessions + session_row_data(session, device_list, csrf_token) for session in sessions ], } content = paginated_table_content( @@ -313,9 +308,7 @@ def _row_with_navbar(request: HttpRequest, session: Session) -> HttpResponse: counts = model_counts(request) fragment = Fragment( session_row(session, device_list, get_token(request)), - NavbarPlaytime( - counts["today_played"], counts["last_7_played"], oob=True - ), + NavbarPlaytime(counts["today_played"], counts["last_7_played"], oob=True), ) return HttpResponse(str(fragment)) diff --git a/tests/test_rendered_pages.py b/tests/test_rendered_pages.py index 9912f7b..89668d6 100644 --- a/tests/test_rendered_pages.py +++ b/tests/test_rendered_pages.py @@ -130,7 +130,7 @@ class RenderedPagesTest(TestCase): self.assertIn("submit_and_redirect", html) self.assertIn("Submit & Create Purchase", html) # & correctly escaped self.assertIn("submit_and_create_session", html) - self.assertIn("Submit & Create Session", html) # & correctly escaped + self.assertIn("Submit & Create Session", html) # & correctly escaped # Fields self-style: label + control carry their own classes (no #add-form # / form CSS in input.css). self.assertIn("mb-2.5 text-sm font-medium text-heading", html) # _LABEL_CLASS diff --git a/tests/test_session_endpoints.py b/tests/test_session_endpoints.py index 555de40..ccc84be 100644 --- a/tests/test_session_endpoints.py +++ b/tests/test_session_endpoints.py @@ -38,9 +38,7 @@ def test_reset_session_start_htmx_returns_row_no_refresh_header( auth_client, running_session ): original_start = running_session.timestamp_start - url = reverse( - "games:list_sessions_reset_session_start", args=[running_session.pk] - ) + url = reverse("games:list_sessions_reset_session_start", args=[running_session.pk]) response = auth_client.get(url, HTTP_HX_REQUEST="true") body = response.content.decode() assert response.status_code == 200