violet-poolController-api 0.0.19__tar.gz → 0.0.22__tar.gz

{violet_poolcontroller_api-0.0.19/violet_poolController_api.egg-info → violet_poolcontroller_api-0.0.22}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: violet-poolController-api
-Version: 0.0.19
+Version: 0.0.22
 Summary: Asynchronous Python client for the Violet Pool Controller.
 Home-page: https://github.com/Xerolux/violet-poolController-api
 Author: Basti (Xerolux)

{violet_poolcontroller_api-0.0.19 → violet_poolcontroller_api-0.0.22}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "violet-poolController-api"
-version = "0.0.19"
+version = "0.0.22"
 authors = [
   { name="Basti (Xerolux)", email="git@xerolux.de" },
 ]
@@ -45,13 +45,13 @@ asyncio_mode = "auto"
 
 [tool.ruff]
 line-length = 100
-target-version = "0.0.19"
+target-version = "0.0.22"
 
 [tool.ruff.lint]
 select = ["E", "F", "W", "I", "UP"]
 ignore = ["E501"]
 
 [tool.mypy]
-python_version = "0.0.19"
+python_version = "0.0.22"
 warn_return_any = true
 warn_unused_configs = true

{violet_poolcontroller_api-0.0.19 → violet_poolcontroller_api-0.0.22}/setup.py

@@ -1,7 +1,7 @@
 from setuptools import setup, find_packages
 setup(
     name="violet-poolController-api",
-    version="0.0.19",
+    version="0.0.22",
     author="Basti (Xerolux)",
     author_email="git@xerolux.de",
     description="Asynchronous Python client for the Violet Pool Controller.",

{violet_poolcontroller_api-0.0.19 → violet_poolcontroller_api-0.0.22}/tests/test_api.py

@@ -214,12 +214,12 @@ async def test_set_config_sanitizes_payload_before_request(
     api_client: VioletPoolAPI,
     monkeypatch: pytest.Monkeypatch,
 ) -> None:
-    """Test set_config applies sanitizer before sending JSON payload."""
+    """Test set_config applies sanitizer before sending form-encoded payload."""
     captured: dict[str, object] = {}
 
     async def fake_request(endpoint: str, **kwargs: Any) -> str:  # noqa: ANN401
         captured["endpoint"] = endpoint
-        captured["json_payload"] = kwargs.get("json_payload")
+        captured["data"] = kwargs.get("data")
         return "OK"
 
     monkeypatch.setattr(api_client, "_request", fake_request)
@@ -228,7 +228,7 @@ async def test_set_config_sanitizes_payload_before_request(
 
     assert result["success"] is True
     assert result["response"] == "OK"
-    assert captured["json_payload"] == {"poolmode": "A<mode>", "speed": 3.7}
+    assert captured["data"] == {"poolmode": "A<mode>", "speed": 3.7}
 
 
 @pytest.mark.asyncio
@@ -694,10 +694,9 @@ async def test_set_all_dmx_scenes_alloff(
     mock_aioresponse: aioresponses,
     api_client: VioletPoolAPI,
 ) -> None:
-    """Test set_all_dmx_scenes sends ALLOFF to all 12 DMX scenes."""
-    for i in range(1, 13):
-        url = f"http://192.168.1.100/setFunctionManually?DMX_SCENE{i},ALLOFF,0,0"
-        mock_aioresponse.get(url, body="OK", status=200)
+    """Test set_all_dmx_scenes sends a single ALLOFF request to DMX_SCENE1."""
+    url = "http://192.168.1.100/setFunctionManually?DMX_SCENE1,ALLOFF,0,0"
+    mock_aioresponse.get(url, body="OK", status=200)
 
     result = await api_client.set_all_dmx_scenes("ALLOFF")
 
@@ -777,8 +776,8 @@ async def test_set_device_temperature(
     api_client: VioletPoolAPI,
 ) -> None:
     """Test set_device_temperature sends correct target temperature."""
-    url = "http://192.168.1.100/setTargetValues?target=HEATER_TARGET_TEMP&value=28.0"
-    mock_aioresponse.get(url, body="OK", status=200)
+    url = "http://192.168.1.100/setConfig"
+    mock_aioresponse.post(url, body="OK", status=200)
 
     result = await api_client.set_device_temperature("HEATER", 28.0)
 
@@ -791,8 +790,8 @@ async def test_set_ph_target(
     api_client: VioletPoolAPI,
 ) -> None:
     """Test set_ph_target sends correct pH target value."""
-    url = "http://192.168.1.100/setTargetValues?target=pH&value=7.2"
-    mock_aioresponse.get(url, body="OK", status=200)
+    url = "http://192.168.1.100/setConfig"
+    mock_aioresponse.post(url, body="OK", status=200)
 
     result = await api_client.set_ph_target(7.2)
 
@@ -805,8 +804,8 @@ async def test_set_orp_target(
     api_client: VioletPoolAPI,
 ) -> None:
     """Test set_orp_target sends correct ORP target value."""
-    url = "http://192.168.1.100/setTargetValues?target=ORP&value=750"
-    mock_aioresponse.get(url, body="OK", status=200)
+    url = "http://192.168.1.100/setConfig"
+    mock_aioresponse.post(url, body="OK", status=200)
 
     result = await api_client.set_orp_target(750)
 
@@ -819,8 +818,8 @@ async def test_set_min_chlorine_level(
     api_client: VioletPoolAPI,
 ) -> None:
     """Test set_min_chlorine_level sends correct chlorine target value."""
-    url = "http://192.168.1.100/setTargetValues?target=MinChlorine&value=0.5"
-    mock_aioresponse.get(url, body="OK", status=200)
+    url = "http://192.168.1.100/setConfig"
+    mock_aioresponse.post(url, body="OK", status=200)
 
     result = await api_client.set_min_chlorine_level(0.5)
 
@@ -833,8 +832,8 @@ async def test_set_target_value(
     api_client: VioletPoolAPI,
 ) -> None:
     """Test set_target_value sends generic target value update."""
-    url = "http://192.168.1.100/setTargetValues?target=CUSTOM&value=42.0"
-    mock_aioresponse.get(url, body="OK", status=200)
+    url = "http://192.168.1.100/setConfig"
+    mock_aioresponse.post(url, body="OK", status=200)
 
     result = await api_client.set_target_value("CUSTOM", 42.0)
 

{violet_poolcontroller_api-0.0.19 → violet_poolcontroller_api-0.0.22/violet_poolController_api.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: violet-poolController-api
-Version: 0.0.19
+Version: 0.0.22
 Summary: Asynchronous Python client for the Violet Pool Controller.
 Home-page: https://github.com/Xerolux/violet-poolController-api
 Author: Basti (Xerolux)

{violet_poolcontroller_api-0.0.19 → violet_poolcontroller_api-0.0.22}/violet_poolcontroller_api/api.py

@@ -53,7 +53,6 @@ from .const_api import (
     API_SET_DOSING_PARAMETERS,
     API_SET_FUNCTION_MANUALLY,
     API_SET_OUTPUT_TESTMODE,
-    API_SET_TARGET_VALUES,
     API_TRIGGER_MANUAL_DOSING,
     DOSING_FUNCTIONS,
     DOSING_OUTPUT_INDEX,
@@ -775,7 +774,7 @@ class VioletPoolAPI:
         body = await self._request(
             API_SET_CONFIG,
             method="POST",
-            json_payload=sanitized_config,
+            data=sanitized_config,
         )
         return self._command_result(body)
 
@@ -866,7 +865,7 @@ class VioletPoolAPI:
         body = await self._request(
             API_RESTORE_CALIBRATION,
             method="POST",
-            json_payload={"sensor": sensor, "timestamp": timestamp},
+            data={"sensor": sensor, "timestamp": timestamp},
         )
         return self._command_result(body)
 
@@ -1040,13 +1039,16 @@ class VioletPoolAPI:
         )
 
     async def set_all_dmx_scenes(self, action: str) -> dict[str, Any]:
-        """Send the same command to all DMX scenes.
+        """Send a global DMX command that affects all scenes and the LIGHT output.
+
+        The controller treats ALLON/ALLOFF/ALLAUTO as global actions: a single
+        request to any DMX_SCENE key switches all 12 scenes and LIGHT at once.
 
         Args:
             action: The action to perform (ALLON, ALLOFF, ALLAUTO).
 
         Returns:
-            A dictionary with the combined command results.
+            A dictionary with the command result.
 
         Raises:
             VioletPoolAPIError: If the action is unsupported.
@@ -1056,28 +1058,7 @@ class VioletPoolAPI:
             msg = f"Unsupported DMX action: {action}"
             raise VioletPoolAPIError(msg)
 
-        tasks = []
-        for scene in range(1, 13):
-            key = f"DMX_SCENE{scene}"
-            tasks.append(self.set_switch_state(key, action))
-
-        # Run requests concurrently
-        raw_results = await asyncio.gather(*tasks, return_exceptions=True)
-
-        results: list[dict[str, Any]] = []
-        for res in raw_results:
-            if isinstance(res, Exception):
-                results.append(
-                    {"success": False, "response": f"{type(res).__name__}: {res!s}"},
-                )
-            elif isinstance(res, dict):
-                results.append(res)
-
-        success = all(result.get("success") is True for result in results)
-        response = ", ".join(
-            str(result.get("response", "")) for result in results if result.get("response")
-        )
-        return {"success": success, "response": response}
+        return await self.set_switch_state("DMX_SCENE1", action)
 
     async def set_light_color_pulse(self) -> dict[str, Any]:
         """Trigger the color pulse animation for the pool light.
@@ -1186,12 +1167,7 @@ class VioletPoolAPI:
             A dictionary with the command result.
 
         """
-        params = {"target": key, "value": value}
-        body = await self._request(
-            API_SET_TARGET_VALUES,
-            params=params,
-        )
-        return self._command_result(body)
+        return await self.set_config({key: value})
 
     async def set_dosing_parameters(
         self,
@@ -1209,7 +1185,7 @@ class VioletPoolAPI:
         body = await self._request(
             API_SET_DOSING_PARAMETERS,
             method="POST",
-            json_payload=dict(parameters),
+            data=dict(parameters),
         )
         return self._command_result(body)