Skip to content

Commit 154ab12

Browse files
committed
remove unnecessary duplicated fields from defer tests
Replicates graphql/graphql-js@24b9761
1 parent 7f4f04d commit 154ab12

File tree

1 file changed

+6
-12
lines changed

1 file changed

+6
-12
lines changed

tests/execution/test_defer.py

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -349,7 +349,6 @@ async def can_defer_fragments_containing_scalar_types():
349349
}
350350
}
351351
fragment NameFragment on Hero {
352-
id
353352
name
354353
}
355354
"""
@@ -359,9 +358,7 @@ async def can_defer_fragments_containing_scalar_types():
359358
assert result == [
360359
{"data": {"hero": {"id": "1"}}, "hasNext": True},
361360
{
362-
"incremental": [
363-
{"data": {"id": "1", "name": "Luke"}, "path": ["hero"]}
364-
],
361+
"incremental": [{"data": {"name": "Luke"}, "path": ["hero"]}],
365362
"hasNext": False,
366363
},
367364
]
@@ -507,12 +504,11 @@ async def can_defer_a_fragment_within_an_already_deferred_fragment():
507504
"""
508505
query HeroNameQuery {
509506
hero {
510-
id
511507
...TopFragment @defer(label: "DeferTop")
512508
}
513509
}
514510
fragment TopFragment on Hero {
515-
name
511+
id
516512
...NestedFragment @defer(label: "DeferNested")
517513
}
518514
fragment NestedFragment on Hero {
@@ -525,7 +521,7 @@ async def can_defer_a_fragment_within_an_already_deferred_fragment():
525521
result = await complete(document)
526522

527523
assert result == [
528-
{"data": {"hero": {"id": "1"}}, "hasNext": True},
524+
{"data": {"hero": {}}, "hasNext": True},
529525
{
530526
"incremental": [
531527
{
@@ -540,7 +536,7 @@ async def can_defer_a_fragment_within_an_already_deferred_fragment():
540536
"label": "DeferNested",
541537
},
542538
{
543-
"data": {"name": "Luke"},
539+
"data": {"id": "1"},
544540
"path": ["hero"],
545541
"label": "DeferTop",
546542
},
@@ -555,7 +551,6 @@ async def can_defer_a_fragment_that_is_also_not_deferred_with_deferred_first():
555551
"""
556552
query HeroNameQuery {
557553
hero {
558-
id
559554
...TopFragment @defer(label: "DeferTop")
560555
...TopFragment
561556
}
@@ -568,7 +563,7 @@ async def can_defer_a_fragment_that_is_also_not_deferred_with_deferred_first():
568563
result = await complete(document)
569564

570565
assert result == [
571-
{"data": {"hero": {"id": "1", "name": "Luke"}}, "hasNext": True},
566+
{"data": {"hero": {"name": "Luke"}}, "hasNext": True},
572567
{
573568
"incremental": [
574569
{
@@ -587,7 +582,6 @@ async def can_defer_a_fragment_that_is_also_not_deferred_with_non_deferred_first
587582
"""
588583
query HeroNameQuery {
589584
hero {
590-
id
591585
...TopFragment
592586
...TopFragment @defer(label: "DeferTop")
593587
}
@@ -600,7 +594,7 @@ async def can_defer_a_fragment_that_is_also_not_deferred_with_non_deferred_first
600594
result = await complete(document)
601595

602596
assert result == [
603-
{"data": {"hero": {"id": "1", "name": "Luke"}}, "hasNext": True},
597+
{"data": {"hero": {"name": "Luke"}}, "hasNext": True},
604598
{
605599
"incremental": [
606600
{

0 commit comments

Comments
 (0)