fix broken test

streamingfast · Oct 26, 2023 · 979144a · 979144a
1 parent 0a4289d
commit 979144a
Showing 1 changed file with 10 additions and 7 deletions.
diff --git a/pipeline/resolve.go b/pipeline/resolve.go
@@ -42,14 +42,17 @@ func BuildRequestDetails(
 		return nil, nil, err
 	}
 
-	graph, err := manifest.NewModuleGraph(request.Modules.Modules)
-	if err != nil {
-		return nil, nil, status.Errorf(grpccodes.InvalidArgument, "invalid modules: %s", err.Error())
-	}
+	moduleHasStatefulDependencies := true
+	if req.Modules != nil { // because of tests which do not define modules in the request. too annoying to add this to tests for now. (TODO)
+		graph, err := manifest.NewModuleGraph(request.Modules.Modules)
+		if err != nil {
+			return nil, nil, status.Errorf(grpccodes.InvalidArgument, "invalid modules: %s", err.Error())
+		}
 
-	moduleHasStatefulDependencies, err := graph.HasStatefulDependencies(request.OutputModule)
-	if err != nil {
-		return nil, nil, status.Errorf(grpccodes.InvalidArgument, "invalid output module: %s", err.Error())
+		moduleHasStatefulDependencies, err = graph.HasStatefulDependencies(request.OutputModule)
+		if err != nil {
+			return nil, nil, status.Errorf(grpccodes.InvalidArgument, "invalid output module: %s", err.Error())
+		}
 	}
 
 	linearHandoff, err := computeLinearHandoffBlockNum(request.ProductionMode, req.ResolvedStartBlockNum, request.StopBlockNum, getRecentFinalBlock, moduleHasStatefulDependencies)