Fix TOU status showing inverted values

TOU status was always appearing as OFF regardless of actual state.

Root cause: tou_status_to_python converter was using incorrect encoding:
- Old (incorrect): 0 = OFF, 1 = ON
- Correct: 1 = OFF, 2 = ON (standard OnOffFlag encoding)

When device sent 2 (ON), converter returned False. When device sent 1 (OFF),
it incorrectly returned True.

Changes:
- Updated tou_status_to_python() to use OnOffFlag encoding (1=OFF, 2=ON)
- Fixed all TOU status converter tests to match correct encoding
- Added documentation noting this uses standard OnOffFlag encoding

All 378 tests pass.

Author: eman

src/nwp500/converters.py

@@ -65,7 +65,8 @@ def device_bool_from_python(value: bool) -> int:
 def tou_status_to_python(value: Any) -> bool:
     """Convert Time of Use status to Python bool.
 
-    Device representation: 0 = Off/False, 1 = On/True
+    Device representation: 1 = Off/False, 2 = On/True
+    (Uses standard OnOffFlag encoding)
 
     Args:
         value: Device TOU status value.
@@ -74,12 +75,12 @@ def tou_status_to_python(value: Any) -> bool:
         Python boolean.
 
     Example:
-        >>> tou_status_to_python(1)
+        >>> tou_status_to_python(2)
         True
-        >>> tou_status_to_python(0)
+        >>> tou_status_to_python(1)
         False
     """
-    return bool(value == 1)
+    return bool(value == 2)
 
 
 def tou_override_to_python(value: Any) -> bool:

tests/test_model_converters.py

@@ -93,48 +93,49 @@ def test_off_value_variations(self, off_value):
 class TestTouStatusConverter:
     """Test tou_status_to_python converter.
 
-    TOU (Time of Use) status encoding converts device state to boolean.
-    Device: 1 = Enabled (True), anything else = Disabled (False)
-    NOTE: String values are NOT converted to int before comparison.
+    TOU (Time of Use) status encoding uses standard OnOffFlag:
+    Device: 1 = OFF/False, 2 = ON/True
     """
 
     def test_tou_disabled(self):
-        """TOU disabled state: 0 = False."""
-        result = tou_status_to_python(0)
+        """TOU disabled state: 1 = False."""
+        result = tou_status_to_python(1)
         assert isinstance(result, bool)
         assert result is False
 
     def test_tou_enabled(self):
-        """TOU enabled state: 1 = True."""
-        result = tou_status_to_python(1)
+        """TOU enabled state: 2 = True."""
+        result = tou_status_to_python(2)
         assert isinstance(result, bool)
         assert result is True
 
     def test_string_disabled(self):
-        """String '0' = TOU disabled."""
-        assert tou_status_to_python("0") is False
+        """String '1' is not equal to int 2, so returns False."""
+        assert tou_status_to_python("1") is False
 
     def test_string_enabled(self):
-        """String '1' is not equal to int 1, so returns False."""
-        # tou_status_to_python uses: bool(value == 1)
-        # String "1" != int 1, so result is False
-        assert tou_status_to_python("1") is False
+        """String '2' is not equal to int 2, so returns False."""
+        # tou_status_to_python uses: bool(value == 2)
+        # String "2" != int 2, so result is False
+        assert tou_status_to_python("2") is False
 
     def test_invalid_value(self):
-        """Value other than 1 is treated as False."""
-        assert tou_status_to_python(2) is False
+        """Value other than 2 is treated as False."""
+        assert tou_status_to_python(0) is False
         assert tou_status_to_python(3) is False
         assert tou_status_to_python(-1) is False
 
-    @pytest.mark.parametrize("enabled_value", [1, 1.0])
+    @pytest.mark.parametrize("enabled_value", [2, 2.0])
     def test_enabled_variations(self, enabled_value):
-        """Test numeric variations of enabled (value == 1)."""
-        # Only numeric 1 and float 1.0 equal int 1
+        """Test numeric variations of enabled (value == 2)."""
+        # Only numeric 2 and float 2.0 equal int 2
         assert tou_status_to_python(enabled_value) is True
 
-    @pytest.mark.parametrize("disabled_value", [0, "0", 0.0, 2, 3, -1, "1"])
+    @pytest.mark.parametrize(
+        "disabled_value", [0, "0", 0.0, 1, 3, -1, "1", "2"]
+    )
     def test_disabled_variations(self, disabled_value):
-        """Test various representations of disabled (value != 1)."""
+        """Test various representations of disabled (value != 2)."""
         assert tou_status_to_python(disabled_value) is False