Merge branch 'fw-13-4'

Author: ziima

recuair_cli/main.py

@@ -149,7 +149,8 @@ async def post_request(client: httpx.AsyncClient, device: str, data: dict[str, A
     except httpx.HTTPError as error:
         _LOGGER.debug("Error encountered: %s", error)
         raise RecuairError(f"Error from device {device}: {error}") from error
-    if response.status_code != HTTPStatus.MOVED_PERMANENTLY:
+    # MOVED_PERMANENTLY was returned by firmware 12 (end of 2023).
+    if response.status_code not in (HTTPStatus.MOVED_PERMANENTLY, HTTPStatus.SEE_OTHER):
         raise RecuairError(f"Unknown error from device {device}, status code {response.status_code}")
     # XXX: When invalid request is send, Recuair returns status page :-/
     _LOGGER.debug("Response [%s]: %s", response, response.text)

recuair_cli/tests/test_main.py

@@ -1,3 +1,4 @@
+from http import HTTPStatus
 from pathlib import Path
 from unittest import IsolatedAsyncioTestCase, TestCase
 from unittest.mock import call, patch
@@ -83,7 +84,15 @@ async def test_error(self):
 class PostRequestTest(IsolatedAsyncioTestCase):
     async def test(self):
         with respx.mock() as rsps:
-            rsps.post("http://example/", data={"answer": "42"}).mock(Response(301))
+            rsps.post("http://example/", data={"answer": "42"}).mock(Response(HTTPStatus.SEE_OTHER))
+
+            async with httpx.AsyncClient() as client:
+                await post_request(client, "example", {"answer": "42"})
+
+    async def test_fw_12(self):
+        # Test post to firmware 12
+        with respx.mock() as rsps:
+            rsps.post("http://example/", data={"answer": "42"}).mock(Response(HTTPStatus.MOVED_PERMANENTLY))
 
             async with httpx.AsyncClient() as client:
                 await post_request(client, "example", {"answer": "42"})