Skip to content

Commit 85c1285

Browse files
committed
Add missing filter call in complete_awaitable_value()
1 parent 8f8102f commit 85c1285

File tree

3 files changed

+5
-6
lines changed

3 files changed

+5
-6
lines changed

src/graphql/execution/execute.py

+1
Original file line numberDiff line numberDiff line change
@@ -1142,6 +1142,7 @@ async def complete_awaitable_value(
11421142
)
11431143
error = located_error(raw_error, field_nodes, path.as_list())
11441144
handle_field_error(error, return_type, errors)
1145+
self.filter_subsequent_payloads(path, async_payload_record)
11451146
completed = None
11461147
return completed
11471148

tests/execution/test_subscribe.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -488,7 +488,7 @@ async def resolves_to_an_error_if_variables_were_wrong_type():
488488
],
489489
)
490490

491-
assert result.errors[0].original_error is None
491+
assert result.errors[0].original_error
492492

493493

494494
# Once a subscription returns a valid AsyncIterator, it can still yield errors.

tests/execution/test_variables.py

+3-5
Original file line numberDiff line numberDiff line change
@@ -27,11 +27,11 @@
2727
)
2828

2929

30-
def faulty_parse_value(value: str) -> str:
30+
def faulty_parse_value(_value: str) -> str:
3131
raise TestFaultyScalarGraphQLError
3232

3333

34-
def faulty_parse_literal(ast: ValueNode, _variables=None) -> str:
34+
def faulty_parse_literal(_ast: ValueNode, _variables=None) -> str:
3535
raise TestFaultyScalarGraphQLError
3636

3737

@@ -733,9 +733,7 @@ def reports_error_for_array_passed_into_string_input():
733733
],
734734
)
735735

736-
errors = result.errors
737-
assert errors
738-
assert errors[0].original_error
736+
assert result.errors[0].original_error
739737

740738
def reports_error_for_non_provided_variables_for_non_nullable_inputs():
741739
# Note: this test would typically fail validation before

0 commit comments

Comments
 (0)