violet-poolController-api 0.0.12__tar.gz → 0.0.16__tar.gz

{violet_poolcontroller_api-0.0.12 → violet_poolcontroller_api-0.0.16}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: violet-poolController-api
-Version: 0.0.12
+Version: 0.0.16
 Summary: Asynchronous Python client for the Violet Pool Controller.
 Home-page: https://github.com/Xerolux/violet-poolController-api
 Author: Basti (Xerolux)
@@ -16,11 +16,11 @@ Classifier: Topic :: Home Automation
 Requires-Python: >=3.12
 Description-Content-Type: text/markdown
 License-File: LICENSE
-Requires-Dist: aiohttp>=3.9.0
+Requires-Dist: aiohttp>=3.11.0
 Provides-Extra: test
-Requires-Dist: aioresponses>=0.7.6; extra == "test"
-Requires-Dist: pytest>=8.0; extra == "test"
-Requires-Dist: pytest-asyncio>=0.23; extra == "test"
+Requires-Dist: aioresponses>=0.7.7; extra == "test"
+Requires-Dist: pytest>=8.3; extra == "test"
+Requires-Dist: pytest-asyncio>=0.24; extra == "test"
 Dynamic: author
 Dynamic: home-page
 Dynamic: license-file

{violet_poolcontroller_api-0.0.12 → violet_poolcontroller_api-0.0.16}/pyproject.toml

@@ -1,35 +1,46 @@
-[build-system]
-requires = ["setuptools>=61.0.0", "wheel"]
-build-backend = "setuptools.build_meta"
-
-[project]
-name = "violet-poolController-api"
-version = "0.0.12"
-authors = [
-  { name="Basti (Xerolux)", email="git@xerolux.de" },
-]
-description = "Asynchronous Python client for the Violet Pool Controller."
-readme = "README.md"
-requires-python = ">=3.12"
-license = { text = "AGPL-3.0-or-later" }
-classifiers = [
-    "Programming Language :: Python :: 3",
-    "Operating System :: OS Independent",
-    "Intended Audience :: Developers",
-    "Topic :: Home Automation",
-]
-dependencies = [
-    "aiohttp>=3.9.0",
-]
-
-[project.optional-dependencies]
-test = [
-    "aioresponses>=0.7.6",
-    "pytest>=8.0",
-    "pytest-asyncio>=0.23",
-]
-
-[project.urls]
-"Homepage" = "https://github.com/Xerolux/violet-poolController-api"
-"Bug Tracker" = "https://github.com/Xerolux/violet-poolController-api/issues"
-"License" = "https://www.gnu.org/licenses/agpl-3.0.html"
+[build-system]
+requires = ["setuptools>=61.0.0", "wheel"]
+build-backend = "setuptools.build_meta"
+
+[project]
+name = "violet-poolController-api"
+version = "0.0.16"
+authors = [
+  { name="Basti (Xerolux)", email="git@xerolux.de" },
+]
+description = "Asynchronous Python client for the Violet Pool Controller."
+readme = "README.md"
+requires-python = ">=3.12"
+license = { text = "AGPL-3.0-or-later" }
+classifiers = [
+    "Programming Language :: Python :: 3",
+    "Operating System :: OS Independent",
+    "Intended Audience :: Developers",
+    "Topic :: Home Automation",
+]
+dependencies = [
+    "aiohttp>=3.11.0",
+]
+
+[project.optional-dependencies]
+test = [
+    "aioresponses>=0.7.7",
+    "pytest>=8.3",
+    "pytest-asyncio>=0.24",
+]
+
+[project.urls]
+"Homepage" = "https://github.com/Xerolux/violet-poolController-api"
+"Bug Tracker" = "https://github.com/Xerolux/violet-poolController-api/issues"
+"License" = "https://www.gnu.org/licenses/agpl-3.0.html"
+
+[tool.pytest.ini_options]
+asyncio_mode = "auto"
+
+[tool.ruff]
+line-length = 100
+target-version = "0.0.16"
+
+[tool.ruff.lint]
+select = ["E", "F", "W", "I", "UP"]
+ignore = ["E501"]

{violet_poolcontroller_api-0.0.12 → violet_poolcontroller_api-0.0.16}/setup.py

@@ -1,7 +1,7 @@
 from setuptools import setup, find_packages
 setup(
     name="violet-poolController-api",
-    version="0.0.12",
+    version="0.0.16",
     author="Basti (Xerolux)",
     author_email="git@xerolux.de",
     description="Asynchronous Python client for the Violet Pool Controller.",
@@ -19,13 +19,13 @@ setup(
     ],
     python_requires=">=3.12",
     install_requires=[
-        "aiohttp>=3.9.0",
+        "aiohttp>=3.11.0",
     ],
     extras_require={
         "test": [
-            "aioresponses>=0.7.6",
-            "pytest>=8.0",
-            "pytest-asyncio>=0.23",
+            "aioresponses>=0.7.7",
+            "pytest>=8.3",
+            "pytest-asyncio>=0.24",
         ],
     },
     project_urls={

{violet_poolcontroller_api-0.0.12 → violet_poolcontroller_api-0.0.16}/tests/test_api.py

@@ -316,6 +316,81 @@ async def test_get_hardware_profile(mock_aioresponse, api_client):
     assert profile["extension_module_1"] is True
     assert profile["extension_module_2"] is False
 
+@pytest.mark.asyncio
+async def test_set_switch_state_ext1_relay(mock_aioresponse, api_client):
+    """Test set_switch_state sends correct URL for EXT1_2 ON, OFF, and AUTO."""
+    base = "http://192.168.1.100/setFunctionManually"
+
+    mock_aioresponse.get(f"{base}?EXT1_2,ON,0,0", body="OK", status=200)
+    result = await api_client.set_switch_state("EXT1_2", "ON")
+    assert result["success"] is True
+
+    mock_aioresponse.get(f"{base}?EXT1_2,OFF,0,0", body="OK", status=200)
+    result = await api_client.set_switch_state("EXT1_2", "OFF")
+    assert result["success"] is True
+
+    mock_aioresponse.get(f"{base}?EXT1_2,AUTO,0,0", body="OK", status=200)
+    result = await api_client.set_switch_state("EXT1_2", "AUTO")
+    assert result["success"] is True
+
+
+@pytest.mark.asyncio
+async def test_module_alive_on_zero_count(mock_aioresponse, api_client):
+    """EXT1 module must be detected when alive_count key is present but value is 0.
+
+    The alive counter starts at 0 immediately after a controller restart.  The
+    old code required value > 0, which caused EXT1_* readings to be filtered
+    and the relay switch to appear broken right after a restart.
+    """
+    url = "http://192.168.1.100/getReadings?ALL"
+    mock_aioresponse.get(url, payload={"getReadings": {
+        "PUMPSTATE": "2",
+        "SYSTEM_ext1module_alive_count": "0",
+        "EXT1_1": 0,
+        "EXT1_2": 0,
+    }}, status=200)
+
+    result = await api_client.get_readings()
+    assert "EXT1_2" in result, (
+        "EXT1_2 must not be filtered when SYSTEM_ext1module_alive_count is present, "
+        "even if the counter is still 0 after a restart"
+    )
+
+
+@pytest.mark.asyncio
+async def test_ext1_readings_not_filtered_when_detected(mock_aioresponse, api_client):
+    """EXT1_* readings are included when extension_module_1 is detected."""
+    url = "http://192.168.1.100/getReadings?ALL"
+    mock_aioresponse.get(url, payload={"getReadings": {
+        "PUMPSTATE": "2",
+        "SYSTEM_ext1module_alive_count": "12345",
+        "EXT1_1": 1,
+        "EXT1_2": 0,
+        "EXT1_3": 0,
+    }}, status=200)
+
+    result = await api_client.get_readings()
+    assert "EXT1_1" in result
+    assert "EXT1_2" in result
+    assert "EXT1_3" in result
+
+
+@pytest.mark.asyncio
+async def test_ext1_readings_filtered_when_not_detected(mock_aioresponse, api_client):
+    """EXT1_* readings are stripped when extension_module_1 key is absent."""
+    url = "http://192.168.1.100/getReadings?ALL"
+    mock_aioresponse.get(url, payload={"getReadings": {
+        "PUMPSTATE": "2",
+        # No SYSTEM_ext1module_alive_count → module not connected
+        "EXT1_1": 0,
+        "EXT1_2": 0,
+    }}, status=200)
+
+    result = await api_client.get_readings()
+    assert "EXT1_1" not in result
+    assert "EXT1_2" not in result
+
+
 @pytest.mark.asyncio
 async def test_get_hardware_profile_standalone_dosing(mock_aioresponse, standalone_api_client):
     """Test get_hardware_profile with a standalone dosing configuration."""

{violet_poolcontroller_api-0.0.12 → violet_poolcontroller_api-0.0.16}/violet_poolController_api.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: violet-poolController-api
-Version: 0.0.12
+Version: 0.0.16
 Summary: Asynchronous Python client for the Violet Pool Controller.
 Home-page: https://github.com/Xerolux/violet-poolController-api
 Author: Basti (Xerolux)
@@ -16,11 +16,11 @@ Classifier: Topic :: Home Automation
 Requires-Python: >=3.12
 Description-Content-Type: text/markdown
 License-File: LICENSE
-Requires-Dist: aiohttp>=3.9.0
+Requires-Dist: aiohttp>=3.11.0
 Provides-Extra: test
-Requires-Dist: aioresponses>=0.7.6; extra == "test"
-Requires-Dist: pytest>=8.0; extra == "test"
-Requires-Dist: pytest-asyncio>=0.23; extra == "test"
+Requires-Dist: aioresponses>=0.7.7; extra == "test"
+Requires-Dist: pytest>=8.3; extra == "test"
+Requires-Dist: pytest-asyncio>=0.24; extra == "test"
 Dynamic: author
 Dynamic: home-page
 Dynamic: license-file

violet_poolcontroller_api-0.0.16/violet_poolController_api.egg-info/requires.txt

@@ -0,0 +1,6 @@
+aiohttp>=3.11.0
+
+[test]
+aioresponses>=0.7.7
+pytest>=8.3
+pytest-asyncio>=0.24