Make data-class constructors kw-only

For these classes: `ReportRequest`, `Report`, `Proposal`.

Also update the uses of these classes that weren't already using kw
args.

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

Author: shsms

src/frequenz/sdk/actor/_power_managing/_base_classes.py

@@ -19,7 +19,7 @@
     from .. import power_distributing
 
 
-@dataclasses.dataclass(frozen=True)
+@dataclasses.dataclass(frozen=True, kw_only=True)
 class ReportRequest:
     """A request to start a reporting stream."""
 
@@ -42,7 +42,7 @@ def get_channel_name(self) -> str:
         return f"power_manager.report.{self.battery_ids=}.{self.priority=}"
 
 
-@dataclasses.dataclass(frozen=True)
+@dataclasses.dataclass(frozen=True, kw_only=True)
 class Report:
     """Current PowerManager report for a set of batteries."""
 
@@ -73,7 +73,7 @@ class Report:
     """
 
 
-@dataclasses.dataclass(frozen=True)
+@dataclasses.dataclass(frozen=True, kw_only=True)
 class Proposal:
     """A proposal for a battery to be charged or discharged."""
 

src/frequenz/sdk/actor/_power_managing/_matryoshka.py

@@ -194,7 +194,10 @@ def get_status(
         target_power = self._target_power.get(battery_ids)
         if system_bounds.inclusion_bounds is None:
             return Report(
-                target_power, None, system_bounds.exclusion_bounds, distribution_result
+                target_power=target_power,
+                inclusion_bounds=None,
+                exclusion_bounds=system_bounds.exclusion_bounds,
+                distribution_result=distribution_result,
             )
 
         lower_bound = system_bounds.inclusion_bounds.lower
@@ -216,7 +219,7 @@ def get_status(
             else:
                 break
         return Report(
-            target_power,
+            target_power=target_power,
             inclusion_bounds=timeseries.Bounds[Power](
                 lower=lower_bound, upper=upper_bound
             ),