fix: testing and comments

Author: Lash-L

roborock/web_api.py

@@ -33,16 +33,16 @@
 
 class RoborockApiClient:
     _LOGIN_RATES = [
-        Rate(1, Duration.SECOND),  # 1 requests per hour
-        Rate(3, Duration.MINUTE),  # 3 requests per hour
-        Rate(10, Duration.HOUR),  # 10 requests per hour
-        Rate(20, Duration.DAY),  # 20 requests per day
+        Rate(1, Duration.SECOND),
+        Rate(3, Duration.MINUTE),
+        Rate(10, Duration.HOUR),
+        Rate(20, Duration.DAY),
     ]
     _HOME_DATA_RATES = [
-        Rate(1, Duration.SECOND),  # 1 requests per hour
-        Rate(5, Duration.MINUTE),  # 5 requests per hour
-        Rate(15, Duration.HOUR),  # 15 requests per hour
-        Rate(40, Duration.DAY),  # 40 requests per day
+        Rate(1, Duration.SECOND),
+        Rate(5, Duration.MINUTE),
+        Rate(15, Duration.HOUR),
+        Rate(40, Duration.DAY),
     ]
 
     _login_limiter = Limiter(_LOGIN_RATES)
@@ -387,10 +387,10 @@ async def get_home_data_v3(self, user_data: UserData) -> HomeData:
         home_request = PreparedRequest(
             rriot.r.a,
             {
-                "Authorization": self._get_hawk_authentication(rriot, "/v3/user/homes/" + str(home_id)),
+                "Authorization": self._get_hawk_authentication(rriot, "/v3/user/homes/" + home_id),
             },
         )
-        home_response = await home_request.request("get", "/v3/user/homes/" + str(home_id))
+        home_response = await home_request.request("get", "/v3/user/homes/" + home_id)
         if not home_response.get("success"):
             raise RoborockException(home_response)
         home_data = home_response.get("result")

tests/conftest.py

@@ -264,6 +264,16 @@ def mock_rest() -> aioresponses:
         yield mocked
 
 
+@pytest.fixture(autouse=True)
+def skip_rate_limit():
+    """Don't rate limit tests as they aren't actually hitting the api."""
+    with (
+        patch("roborock.web_api.RoborockApiClient._login_limiter.try_acquire"),
+        patch("roborock.web_api.RoborockApiClient._home_data_limiter.try_acquire"),
+    ):
+        yield
+
+
 @pytest.fixture(name="mock_create_local_connection")
 def create_local_connection_fixture(request_handler: RequestHandler) -> Generator[None, None, None]:
     """Fixture that overrides the transport creation to wire it up to the mock socket."""