ISSUE #?

* Fix bug related with setter.

Author: Sergio García Prado

packages/core/minos-microservice-aggregate/minos/aggregate/entities/refs/models.py

@@ -89,6 +89,19 @@ def __getitem__(self, item: str) -> Any:
             except Exception:
                 raise exc
 
+    def __setattr__(self, key: str, value: Any) -> None:
+        try:
+            return super().__setattr__(key, value)
+        except AttributeError as exc:
+            if key == "uuid":
+                self.data = value
+                return
+
+            try:
+                setattr(self.data, key, value)
+            except Exception:
+                raise exc
+
     def __getattr__(self, item: str) -> Any:
         try:
             return super().__getattr__(item)
@@ -183,14 +196,6 @@ def uuid(self) -> UUID:
             return self.data
         return self.data.uuid
 
-    @uuid.setter
-    def uuid(self, value: UUID) -> None:
-        """Set the uuid that identifies the ``Model``.
-
-        :return: This method does not return anything.
-        """
-        self.data = value
-
     @property
     def data_cls(self) -> Optional[type]:
         """Get data class if available.

packages/core/minos-microservice-aggregate/minos/aggregate/events/models.py

@@ -76,6 +76,15 @@ def __getitem__(self, item: str) -> Any:
                     raise exc
             raise exc
 
+    def __getattr__(self, item: str) -> Any:
+        try:
+            return super().__getattr__(item)
+        except AttributeError as exc:
+            try:
+                return self[item]
+            except Exception:
+                raise exc
+
     def get_one(self, name: str, return_diff: bool = False) -> Union[FieldDiff, Any, list[FieldDiff], list[Any]]:
         """Get first field diff with given name.
 

packages/core/minos-microservice-common/minos/common/model/abc.py

@@ -240,13 +240,16 @@ def __setattr__(self, key: str, value: Any) -> None:
             object.__setattr__(self, key, value)
             return
 
+        if key not in self._fields:
+            raise AttributeError(f"{type(self).__name__!r} does not contain the {key!r} attribute.")
+
         try:
             self[key] = value
         except KeyError as exc:
             raise AttributeError(str(exc))
 
     def __getattr__(self, item: str) -> Any:
-        if item.startswith("_"):
+        if item.startswith("_") or item not in self._fields:
             raise AttributeError(f"{type(self).__name__!r} does not contain the {item!r} attribute.")
 
         try:

packages/core/minos-microservice-saga/minos/saga/context.py

@@ -37,12 +37,18 @@ def __delitem__(self, item: str) -> None:
         except KeyError:
             raise KeyError(f"{type(self).__name__!r} does not contain the {item!r} field")
 
+    def __setattr__(self, key: str, value: Any) -> None:
+        try:
+            super().__setattr__(key, value)
+        except AttributeError:
+            self[key] = value
+
     def __delattr__(self, item: str) -> None:
         if item.startswith("_"):
             super().__delattr__(item)
             return
 
         try:
-            self[item]
+            del self[item]
         except KeyError as exc:
             raise AttributeError(str(exc))

packages/core/minos-microservice-saga/tests/test_saga/test_context.py

@@ -57,7 +57,7 @@ def test_deleter_reserved_word(self):
         context["items"] = "foo"
         del context.items
         self.assertIsInstance(context.items, Callable)
-        self.assertEqual("foo", context.fields["items"].value)
+        self.assertNotIn("items", context.fields)
 
     def test_deleter_raises(self):
         with self.assertRaises(AttributeError):