fix representations variables in federation tests

set entity fetch and batch entity fetch separately
wundergraph · Oct 2, 2023 · e5a588c · e5a588c
1 parent 06565ae
commit e5a588c
Show file tree

Hide file tree

Showing 4 changed files with 76 additions and 11 deletions.
diff --git a/v2/pkg/engine/datasource/graphql_datasource/graphql_datasource.go b/v2/pkg/engine/datasource/graphql_datasource/graphql_datasource.go
@@ -379,11 +379,11 @@ func (p *Planner) requiresSerialFetch() bool {
 }
 
 func (p *Planner) requiresEntityFetch() bool {
-	return p.dataSourcePlannerConfig.HasRequiredFields()
+	return p.dataSourcePlannerConfig.HasRequiredFields() && p.dataSourcePlannerConfig.PathType == plan.PlannerPathObject
 }
 
 func (p *Planner) requiresEntityBatchFetch() bool {
-	return p.requiresEntityFetch() && p.dataSourcePlannerConfig.PathType != plan.PlannerPathObject
+	return p.dataSourcePlannerConfig.HasRequiredFields() && p.dataSourcePlannerConfig.PathType != plan.PlannerPathObject
 }
 
 func (p *Planner) ConfigureSubscription() plan.SubscriptionConfiguration {