Skip to content

Commit d073e5b

Browse files
CopilotLash-L
andcommitted
fix: Remove max_delay parameter and update test fixtures
Co-authored-by: Lash-L <20257911+Lash-L@users.noreply.github.com>
1 parent 6fd78d9 commit d073e5b

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

roborock/web_api.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ class RoborockApiClient:
6262
Rate(40, Duration.DAY),
6363
]
6464

65-
_login_limiter = Limiter(_LOGIN_RATES, max_delay=1000)
65+
_login_limiter = Limiter(_LOGIN_RATES)
6666
_home_data_limiter = Limiter(_HOME_DATA_RATES)
6767

6868
def __init__(

tests/fixtures/web_api_fixtures.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,9 @@
1313
def skip_rate_limit() -> Generator[None, None, None]:
1414
"""Don't rate limit tests as they aren't actually hitting the api."""
1515
with (
16-
patch("roborock.web_api.RoborockApiClient._login_limiter.try_acquire"),
17-
patch("roborock.web_api.RoborockApiClient._home_data_limiter.try_acquire"),
16+
patch("roborock.web_api.RoborockApiClient._login_limiter.try_acquire", return_value=True),
17+
patch("roborock.web_api.RoborockApiClient._login_limiter.try_acquire_async", return_value=True),
18+
patch("roborock.web_api.RoborockApiClient._home_data_limiter.try_acquire", return_value=True),
1819
):
1920
yield
2021

0 commit comments

Comments
 (0)