Skip to content

Commit 8e6ee98

Browse files
committed
Use default argument of max() and change test name
1 parent 957b39f commit 8e6ee98

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

rest_framework/views.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -363,7 +363,7 @@ def check_throttles(self, request):
363363
if duration is not None
364364
]
365365

366-
duration = max(durations) if durations else None
366+
duration = max(durations, default=None)
367367
self.throttled(request, duration)
368368

369369
def determine_version(self, request, *args, **kwargs):

tests/test_throttling.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,7 @@ def test_request_throttling_multiple_throttles(self):
159159
assert response.status_code == 429
160160
assert int(response['retry-after']) == 58
161161

162-
def test_handle_negative_throttle_value(self):
162+
def test_throttle_rate_change_negative(self):
163163
self.set_throttle_timer(MockView_DoubleThrottling, 0)
164164
request = self.factory.get('/')
165165
for dummy in range(24):

0 commit comments

Comments
 (0)