PyPlumIO 0.5.23__tar.gz → 0.5.25__tar.gz

{pyplumio-0.5.23 → pyplumio-0.5.25}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: PyPlumIO
-Version: 0.5.23
+Version: 0.5.25
 Summary: PyPlumIO is a native ecoNET library for Plum ecoMAX controllers.
 Author-email: Denis Paavilainen <denpa@denpa.pro>
 License: MIT License

{pyplumio-0.5.23 → pyplumio-0.5.25}/PyPlumIO.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: PyPlumIO
-Version: 0.5.23
+Version: 0.5.25
 Summary: PyPlumIO is a native ecoNET library for Plum ecoMAX controllers.
 Author-email: Denis Paavilainen <denpa@denpa.pro>
 License: MIT License

{pyplumio-0.5.23 → pyplumio-0.5.25}/pyplumio/_version.py

@@ -12,5 +12,5 @@ __version__: str
 __version_tuple__: VERSION_TUPLE
 version_tuple: VERSION_TUPLE
 
-__version__ = version = '0.5.23'
-__version_tuple__ = version_tuple = (0, 5, 23)
+__version__ = version = '0.5.25'
+__version_tuple__ = version_tuple = (0, 5, 25)

{pyplumio-0.5.23 → pyplumio-0.5.25}/pyplumio/helpers/data_types.py

@@ -181,7 +181,7 @@ class String(DataType):
 
     def unpack(self, data: bytes) -> None:
         """Unpack the data."""
-        self._value = data.split(b"\0", 1)[0].decode()
+        self._value = data.split(b"\0", 1)[0].decode("utf-8", "replace")
         self._size = len(self.value) + 1
 
 

{pyplumio-0.5.23 → pyplumio-0.5.25}/pyplumio/helpers/parameter.py

@@ -113,9 +113,18 @@ class Parameter(ABC):
 
     def _call_relational_method(self, method_to_call: str, other: Any) -> Any:
         """Call a specified relational method."""
-        handler = getattr(self.values.value, method_to_call)
-        other = other.value if isinstance(other, ParameterValues) else other
-        return handler(_normalize_parameter_value(other))
+        if isinstance(other, Parameter):
+            other = other.values
+
+        if isinstance(other, ParameterValues):
+            handler = getattr(self.values, method_to_call)
+            return handler(other)
+
+        if isinstance(other, (int, float, bool)) or other in (STATE_OFF, STATE_ON):
+            handler = getattr(self.values.value, method_to_call)
+            return handler(_normalize_parameter_value(other))
+        else:
+            return NotImplemented
 
     def __int__(self) -> int:
         """Return an integer representation of parameter's value."""
@@ -161,6 +170,13 @@ class Parameter(ABC):
         """Compare if parameter value is less that other."""
         return self._call_relational_method("__lt__", other)
 
+    def __copy__(self) -> Parameter:
+        """Create a copy of parameter."""
+        values = type(self.values)(
+            self.values.value, self.values.min_value, self.values.max_value
+        )
+        return type(self)(self.device, self.description, values)
+
     async def set(self, value: Any, retries: int = SET_RETRIES) -> bool:
         """Set a parameter value."""
         if (value := _normalize_parameter_value(value)) == self.values.value:

{pyplumio-0.5.23 → pyplumio-0.5.25}/tests/helpers/test_data_types.py

@@ -229,6 +229,12 @@ def test_string() -> None:
     assert data_type == data_types.String.from_bytes(buffer)
     assert data_type == "test"
 
+    # Test with unknown unicode char.
+    buffer = b"test\xd8\x00"
+    data_type = data_types.String.from_bytes(buffer)
+    assert data_type.value == "test�"
+    assert data_type.size == 6
+
 
 def test_var_bytes() -> None:
     """Test a variable bytes data type."""

{pyplumio-0.5.23 → pyplumio-0.5.25}/tests/helpers/test_parameter.py

@@ -1,5 +1,6 @@
 """Contains tests for the parameter helper class."""
 
+from copy import copy
 from unittest.mock import Mock, patch
 
 import pytest
@@ -171,21 +172,32 @@ def test_switch_update(switch: Switch) -> None:
 
 def test_number_relational(number: Number):
     """Test number relational methods."""
+    new_number = copy(number)
+    assert number == new_number
     assert (number - 1) == 0
+    new_values = ParameterValues(value=1, min_value=0, max_value=5)
+    number.update(new_values)
+    assert number == new_values
     assert (number + 1) == 2
     assert (number * 5) == 5
     assert (number / 1) == 1
     assert (number // 1) == 1
+    assert number.__eq__("cola") is NotImplemented
 
 
 def test_switch_relational(switch: Switch):
     """Test switch relational methods."""
+    new_switch = copy(switch)
+    assert switch == new_switch
+    new_values = ParameterValues(value=1, min_value=0, max_value=1)
     assert (switch + 1) == 1
-    switch.update(ParameterValues(value=1, min_value=0, max_value=0))
+    switch.update(new_values)
+    assert switch == new_values
     assert (switch - 1) == 0
     assert (switch * 0) == 0
     assert (switch / 1) == 1
     assert (switch // 1) == 1
+    assert switch.__eq__("cola") is NotImplemented
 
 
 def test_number_compare(number: Number) -> None: