Revert "Reuse named fragment in subgraphs even if they only apply par…

…tially (#2639)" This reverts commit d60349b.
apollographql · Jul 17, 2023 · 6083bea · 6083bea
1 parent 99f2f8a
commit 6083bea
Show file tree

Hide file tree

Showing 5 changed files with 83 additions and 876 deletions.
diff --git a/gateway-js/src/__tests__/buildQueryPlan.test.ts b/gateway-js/src/__tests__/buildQueryPlan.test.ts
@@ -744,21 +744,18 @@ describe('buildQueryPlan', () => {
 
     it(`should not get confused by a fragment spread multiple times`, () => {
       const operationString = `#graphql
-        fragment PriceAndCountry on Product {
+        fragment Price on Product {
           price
-          details {
-            country
-          }
         }
 
         query {
           topProducts {
             __typename
             ... on Book {
-              ...PriceAndCountry
+              ...Price
             }
             ... on Furniture {
-              ...PriceAndCountry
+              ...Price
             }
           }
         }
@@ -773,20 +770,16 @@ describe('buildQueryPlan', () => {
                       topProducts {
                         __typename
                         ... on Book {
-                          ...PriceAndCountry
+                          ...Price
                         }
                         ... on Furniture {
-                          ...PriceAndCountry
+                          ...Price
                         }
                       }
                     }
                     
-                    fragment PriceAndCountry on Product {
+                    fragment Price on Product {
                       price
-                      details {
-                        __typename
-                        country
-                      }
                     }
                   },
                 }