Remove __setitem__ from PyRFLX package API

ref #783
AdaCore · Sep 28, 2021 · eb851df · eb851df
1 parent e832daf
commit eb851df
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 8 deletions.
diff --git a/rflx/pyrflx/package.py b/rflx/pyrflx/package.py
@@ -28,9 +28,6 @@ def set_message(self, key: str, value: MessageValue) -> None:
     def __getitem__(self, key: str) -> MessageValue:
         return self.new_message(key)
 
-    def __setitem__(self, key: str, value: MessageValue) -> None:
-        self.set_message(key, value)
-
     def __iter__(self) -> Iterator[MessageValue]:
         return self.__messages.values().__iter__()
 

diff --git a/tests/unit/pyrflx_test.py b/tests/unit/pyrflx_test.py
@@ -58,11 +58,6 @@ def test_package_iterator(tlv_package: Package) -> None:
     assert [m.name for m in tlv_package] == ["Message"]
 
 
-def test_package_set_item(tlv_package: Package) -> None:
-    msg = Message("TLV::Msg", [], {})
-    tlv_package["Msg"] = MessageValue(msg)
-
-
 def test_pyrflx_iterator(pyrflx_: PyRFLX) -> None:
     assert {p.name for p in pyrflx_} == {
         "Ethernet",