Remove requirement for inverters to have successors

This is a quick fix to allow PV inverters to be present in the graph.

This will be partially reverted later by expecting battery inverters
to still have graph successors.

Signed-off-by: Sahas Subramanian <[email protected]>

Author: shsms

src/frequenz/sdk/microgrid/_graph.py

@@ -485,18 +485,6 @@ def _validate_intermediary_components(self) -> None:
                 f"{missing_predecessors}"
             )
 
-        missing_successors = list(
-            filter(
-                lambda c: sum(1 for _ in self.successors(c.component_id)) == 0,
-                intermediary_components,
-            )
-        )
-        if len(missing_successors) > 0:
-            raise InvalidGraphError(
-                "Intermediary components without graph successors: "
-                f"{missing_successors}"
-            )
-
     def _validate_junctions(self) -> None:
         """Check that junctions are configured correctly in the graph.
 

tests/microgrid/test_graph.py

@@ -1118,11 +1118,7 @@ def test__validate_intermediary_components(self) -> None:
             ]
         )
         graph._graph.add_edges_from([(1, 3)])
-        with pytest.raises(
-            gr.InvalidGraphError,
-            match="Intermediary components without graph successors",
-        ):
-            graph._validate_intermediary_components()
+        graph._validate_intermediary_components()
 
         graph._graph.clear()
 
@@ -1134,11 +1130,7 @@ def test__validate_intermediary_components(self) -> None:
             ]
         )
         graph._graph.add_edges_from([(1, 2), (2, 3)])
-        with pytest.raises(
-            gr.InvalidGraphError,
-            match="Intermediary components without graph successors",
-        ):
-            graph._validate_intermediary_components()
+        graph._validate_intermediary_components()
 
         # all intermediary nodes have at least one predecessor
         # and at least one successor