Remove PV_ARRAY component category

The PV_ARRAY component category is from the early
MicrogridData days and it is no longer.

Signed-off-by: Daniel Zullo <[email protected]>

Author: daniel-zullo-frequenz

src/frequenz/sdk/microgrid/_graph.py

@@ -896,7 +896,6 @@ def _validate_leaf_components(self) -> None:
                 component_category={
                     ComponentCategory.BATTERY,
                     ComponentCategory.EV_CHARGER,
-                    ComponentCategory.PV_ARRAY,
                 }
             )
         )

src/frequenz/sdk/microgrid/component/_component.py

@@ -97,11 +97,6 @@ class ComponentCategory(Enum):
     CHP = components_pb.ComponentCategory.COMPONENT_CATEGORY_CHP
     """CHP component."""
 
-    # Types not yet supported by the API but which can be inferred
-    # from available graph info.
-    PV_ARRAY = 1000001
-    """PV array component."""
-
 
 def _component_category_from_protobuf(
     component_category: components_pb.ComponentCategory.ValueType,

tests/microgrid/test_graph.py

@@ -1286,13 +1286,6 @@ def test__validate_leaf_components(self) -> None:
         ):
             graph._validate_leaf_components()
 
-        graph._graph.clear()
-        graph._graph.add_node(5, **asdict(Component(5, ComponentCategory.PV_ARRAY)))
-        with pytest.raises(
-            gr.InvalidGraphError, match="Leaf components without graph predecessors"
-        ):
-            graph._validate_leaf_components()
-
         # successors present for at least one leaf node
         graph._graph.clear()
         graph._graph.add_nodes_from(
@@ -1323,20 +1316,6 @@ def test__validate_leaf_components(self) -> None:
         ):
             graph._validate_leaf_components()
 
-        graph._graph.clear()
-        graph._graph.add_nodes_from(
-            [
-                (1, asdict(Component(1, ComponentCategory.GRID))),
-                (4, asdict(Component(4, ComponentCategory.EV_CHARGER))),
-                (5, asdict(Component(5, ComponentCategory.PV_ARRAY))),
-            ]
-        )
-        graph._graph.add_edges_from([(1, 5), (5, 4)])
-        with pytest.raises(
-            gr.InvalidGraphError, match="Leaf components with graph successors"
-        ):
-            graph._validate_leaf_components()
-
         # all leaf nodes have at least one predecessor
         # and no successors
         graph._graph.clear()
@@ -1346,10 +1325,9 @@ def test__validate_leaf_components(self) -> None:
                 (2, asdict(Component(2, ComponentCategory.METER))),
                 (3, asdict(Component(3, ComponentCategory.BATTERY))),
                 (4, asdict(Component(4, ComponentCategory.EV_CHARGER))),
-                (5, asdict(Component(5, ComponentCategory.PV_ARRAY))),
             ]
         )
-        graph._graph.add_edges_from([(1, 2), (1, 3), (1, 4), (1, 5)])
+        graph._graph.add_edges_from([(1, 2), (1, 3), (1, 4)])
         graph._validate_leaf_components()
 
     def test_graph_correction(self) -> None:

tests/microgrid/test_microgrid_api.py

@@ -35,7 +35,6 @@ def components(self) -> List[List[Component]]:
                 Component(1, ComponentCategory.GRID),
                 Component(4, ComponentCategory.METER),
                 Component(5, ComponentCategory.METER),
-                Component(6, ComponentCategory.PV_ARRAY),
                 Component(7, ComponentCategory.METER),
                 Component(8, ComponentCategory.INVERTER),
                 Component(9, ComponentCategory.BATTERY),
@@ -68,7 +67,6 @@ def connections(self) -> List[List[Connection]]:
             [
                 Connection(1, 4),
                 Connection(1, 5),
-                Connection(5, 6),
                 Connection(1, 7),
                 Connection(7, 8),
                 Connection(8, 9),