Following review

Signed-off-by: Simone Orru <simone.orru@secomind.com>
astarte-platform · Sep 28, 2023 · 4d9ae77 · 4d9ae77
1 parent 0f694d6
commit 4d9ae77
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/astarte/device/interface.py b/astarte/device/interface.py
@@ -124,7 +124,8 @@ def __init__(self, interface_definition: dict):
             raise InterfaceFileDecodeError(f"Invalid aggregation type for interface {self.name}.")
 
         if (self.type == "properties") and (self.aggregation == "object"):
-            raise InterfaceFileDecodeError("Properties can only be objects.")
+            raise InterfaceFileDecodeError(
+                "Invalid aggregation type 'object', properties can only be 'individual'.")
 
         self.mappings: list[Mapping] = []
         endpoints = []

diff --git a/astarte/device/mapping.py b/astarte/device/mapping.py
@@ -186,7 +186,7 @@ def __init__(self, mapping_definition: dict, is_datastream: bool):
                 "Fields 'reliability' and 'explicit_timestamp' have no meaning for properties."
             )
         if ("allow_unset" in mapping_definition) and is_datastream:
-            raise InterfaceFileDecodeError("Field 'allow_unset' have no meaning for datastreams.")
+            raise InterfaceFileDecodeError("Field 'allow_unset' has no meaning for datastreams.")
 
     def validate_path(self, path: str):
         """