|
7 | 7 | SlidingWindowRateLimiter, |
8 | 8 | ) |
9 | 9 |
|
| 10 | + |
10 | 11 | @pytest.fixture |
11 | 12 | def mock_redis_from_url(): |
12 | 13 | with patch("redis.asyncio.from_url") as mocker: |
@@ -48,49 +49,49 @@ async def test_fail_open_allows_requests(): |
48 | 49 | assert allowed is True |
49 | 50 | assert retry_after is None |
50 | 51 |
|
51 | | -def test_get_rate_limiter_none_strategy(mock_redis_from_url): |
| 52 | +def test_get_rate_limiter_none_strategy(mock_redis_from_url): # noqa: ARG001 |
52 | 53 | rl = get_rate_limiter(strategy="none", redis_url="redis://localhost:6379/0", fail_open=True) |
53 | 54 | assert rl is None |
54 | 55 |
|
55 | 56 |
|
56 | | -def test_get_rate_limiter_empty_strategy(mock_redis_from_url): |
| 57 | +def test_get_rate_limiter_empty_strategy(mock_redis_from_url): # noqa: ARG001 |
57 | 58 | rl = get_rate_limiter(strategy="", redis_url="redis://localhost:6379/0", fail_open=True) |
58 | 59 | assert rl is None |
59 | 60 |
|
60 | 61 |
|
61 | | -def test_get_rate_limiter_null_strategy(mock_redis_from_url): |
| 62 | +def test_get_rate_limiter_null_strategy(mock_redis_from_url): # noqa: ARG001 |
62 | 63 | rl = get_rate_limiter(strategy="null", redis_url="redis://localhost:6379/0", fail_open=True) |
63 | 64 | assert rl is None |
64 | 65 |
|
65 | 66 |
|
66 | | -def test_get_rate_limiter_no_strategy(mock_redis_from_url): |
| 67 | +def test_get_rate_limiter_no_strategy(mock_redis_from_url): # noqa: ARG001 |
67 | 68 | rl = get_rate_limiter(strategy=None, redis_url="redis://localhost:6379/0", fail_open=True) |
68 | 69 | assert rl is None |
69 | 70 |
|
70 | 71 |
|
71 | | -def test_get_rate_limiter_no_redis_url(mock_redis_from_url): |
| 72 | +def test_get_rate_limiter_no_redis_url(mock_redis_from_url): # noqa: ARG001 |
72 | 73 | rl = get_rate_limiter(strategy="sliding_window", redis_url="", fail_open=True) |
73 | 74 | assert rl is None |
74 | 75 |
|
75 | 76 |
|
76 | | -def test_get_rate_limiter_sliding_window(mock_redis_from_url): |
| 77 | +def test_get_rate_limiter_sliding_window(mock_redis_from_url): # noqa: ARG001 |
77 | 78 | rl = get_rate_limiter(strategy="sliding_window", redis_url="redis://x", fail_open=True) |
78 | 79 | assert isinstance(rl, SlidingWindowRateLimiter) |
79 | 80 |
|
80 | 81 |
|
81 | | -def test_get_rate_limiter_sliding_window_dash(mock_redis_from_url): |
| 82 | +def test_get_rate_limiter_sliding_window_dash(mock_redis_from_url): # noqa: ARG001 |
82 | 83 | rl = get_rate_limiter(strategy="sliding-window", redis_url="redis://x", fail_open=False) |
83 | 84 | assert isinstance(rl, SlidingWindowRateLimiter) |
84 | 85 | assert rl.get_fail_open() is False |
85 | 86 |
|
86 | 87 |
|
87 | | -def test_get_rate_limiter_fail_open_coercion(mock_redis_from_url): |
| 88 | +def test_get_rate_limiter_fail_open_coercion(mock_redis_from_url): # noqa: ARG001 |
88 | 89 | rl = get_rate_limiter(strategy="sliding_window", redis_url="redis://x", fail_open=None) |
89 | 90 | assert isinstance(rl, SlidingWindowRateLimiter) |
90 | 91 | assert rl.get_fail_open() is False # default fallback |
91 | 92 |
|
92 | 93 |
|
93 | | -def test_rate_limiter_unknown_strategy(mock_redis_from_url): |
| 94 | +def test_rate_limiter_unknown_strategy(mock_redis_from_url): # noqa: ARG001 |
94 | 95 | with pytest.raises(ValueError) as e: |
95 | 96 | get_rate_limiter(strategy="weird_strategy", redis_url="redis://x", fail_open=True) |
96 | 97 | assert "Unknown rate limiter strategy" in str(e.value) |
0 commit comments