From 70a59aff9ef8fd352ad55fc4c20c36db2998c79f Mon Sep 17 00:00:00 2001 From: Arun Babu Neelicattu Date: Thu, 22 Feb 2024 01:58:30 +0100 Subject: [PATCH] pre-commit: update and fix linting issues --- .pre-commit-config.yaml | 8 ++++---- src/aiographql/client/request.py | 10 ++++++---- tests/conftest.py | 7 ++++--- tests/test_headers.py | 5 +---- 4 files changed, 15 insertions(+), 15 deletions(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 68f9fd9..3256235 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -1,19 +1,19 @@ repos: - repo: https://github.com/ambv/black - rev: 22.12.0 + rev: 24.2.0 hooks: - id: black - repo: https://github.com/pycqa/flake8 - rev: 6.0.0 + rev: 7.0.0 hooks: - id: flake8 additional_dependencies: [ flake8-bugbear ] - repo: https://github.com/timothycrosley/isort - rev: 5.11.4 + rev: 5.13.2 hooks: - id: isort - repo: https://github.com/pre-commit/pre-commit-hooks - rev: v4.4.0 + rev: v4.5.0 hooks: - id: trailing-whitespace - id: end-of-file-fixer diff --git a/src/aiographql/client/request.py b/src/aiographql/client/request.py index 3f37a05..987ba74 100644 --- a/src/aiographql/client/request.py +++ b/src/aiographql/client/request.py @@ -92,9 +92,11 @@ def __post_init__( object.__setattr__( self, "request", - GraphQLRequest(query=self.request) - if isinstance(self.request, str) - else self.request.copy( - headers=headers, operation=operation, variables=variables + ( + GraphQLRequest(query=self.request) + if isinstance(self.request, str) + else self.request.copy( + headers=headers, operation=operation, variables=variables + ) ), ) diff --git a/tests/conftest.py b/tests/conftest.py index e8091c4..998d01e 100644 --- a/tests/conftest.py +++ b/tests/conftest.py @@ -53,9 +53,10 @@ def city_name(): @pytest.fixture def subscription_query(city_name): + # noqa: B907 return f""" subscription {{ - city(where: {{name: {{_eq: "{city_name}"}}}}) {{ + city(where: {{name: {{_eq: "{city_name!r}"}}}}) {{ name id }} @@ -110,14 +111,14 @@ async def mutation_city(client, headers, city_name): affected_rows }} }} - """ + """ # noqa: B907 delete_mutation = f""" mutation {{ delete_city(where: {{name: {{_eq: "{city_name}"}}}}) {{ affected_rows }} }} - """ + """ # noqa: B907 request = GraphQLRequest(query=delete_mutation) _ = await client.query(request=request, headers=headers) diff --git a/tests/test_headers.py b/tests/test_headers.py index de4ed0a..f4b2b0d 100644 --- a/tests/test_headers.py +++ b/tests/test_headers.py @@ -48,7 +48,4 @@ async def test_post_headers(server, headers, client, query_city): async def test_no_headers(server, client, query_city): graphql_request = GraphQLRequest(query=query_city) with pytest.raises(GraphQLIntrospectionException): - response = await client.post(graphql_request) - assert isinstance(response, GraphQLResponse) - assert not response.data - assert response.errors + await client.post(graphql_request)