8000 Reformat and add a script to the makefile · graphql-python/graphql-ws@94694e8 · GitHub
[go: up one dir, main page]

Skip to content

Commit 94694e8

Browse files
committed
Reformat and add a script to the makefile
1 parent 0501fa9 commit 94694e8

File tree

6 files changed

+14
-10
lines changed

6 files changed

+14
-10
lines changed

Makefile

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,9 +50,12 @@ clean-test: ## remove test and coverage artifacts
5050
lint: ## check style with flake8
5151
flake8 graphql_ws tests
5252

53+
format:
54+
black ./graphql_ws ./tests
55+
5356
test: ## run tests quickly with the default Python
5457
py.test
55-
58+
5659

5760
test-all: ## run tests on every Python version with tox
5861
tox

graphql_ws/base.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ def remove_operation(self, op_id):
4141

4242
def unsubscribe(self, op_id):
4343
async_iterator = self.remove_operation(op_id)
44-
if hasattr(async_iterator, 'dispose'):
44+
if hasattr(async_iterator, "dispose"):
4545
async_iterator.dispose()
4646
return async_iterator
4747

graphql_ws/django/consumers.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77

88

99
class GraphQLSubscriptionConsumer(AsyncJsonWebsocketConsumer):
10-
1110
async def connect(self):
1211
self.connection_context = None
1312
if WS_PROTOCOL in self.scope["subprotocols"]:

graphql_ws/django/routing.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,10 @@
1515
channels_version_tuple = get_version_tuple(channels_version)
1616

1717

18-
if channels_version_tuple > (3, 0 , 0):
19-
websocket_urlpatterns = [path("subscriptions", GraphQLSubscriptionConsumer.as_asgi())]
18+
if channels_version_tuple > (3, 0, 0):
19+
websocket_urlpatterns = [
20+
path("subscriptions", GraphQLSubscriptionConsumer.as_asgi())
21+
]
2022
else:
2123
websocket_urlpatterns = [path("subscriptions", GraphQLSubscriptionConsumer)]
2224

tests/test_base.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ def test_observer_data():
9191
send_error=send_error,
9292
send_message=send_message,
9393
)
94-
observer.on_next('data')
94+
observer.on_next("data")
9595
assert send_result.called
9696
assert not send_error.called
9797

@@ -107,6 +107,6 @@ def test_observer_exception():
107107
send_error=send_error,
108108
send_message=send_message,
109109
)
110-
observer.on_next(TypeError('some bad message'))
110+
observer.on_next(TypeError("some bad message"))
111111
assert send_error.called
112112
assert not send_result.called

tests/test_base_async.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -83,18 +83,18 @@ async def test_resolver_with_promise(server):
8383
connection_context=None, op_id=1, execution_result=result
8484
)
8585
assert server.send_message.called
86-
assert result.data == {'test': [1, 2]}
86+
assert result.data == {"test": [1, 2]}
8787

8888

8989
async def test_resolver_with_nested_promise(server):
9090
server.send_message = AsyncMock()
9191
result = mock.Mock()
9292
inner = promise.Promise(lambda resolve, reject: resolve(2))
93-
outer = promise.Promise(lambda resolve, reject: resolve({'in': inner}))
93+
outer = promise.Promise(lambda resolve, reject: resolve({"in": inner}))
9494
result.data = {"test": [1, outer]}
9595
result.errors = None
9696
await server.send_execution_result(
9797
connection_context=None, op_id=1, execution_result=result
9898
)
9999
assert server.send_message.called
100-
assert result.data == {'test': [1, {'in': 2}]}
100+
assert result.data == {"test": [1, {"in": 2}]}

0 commit comments

Comments
 (0)
0