mirror of
https://github.com/zulip/zulip.git
synced 2026-06-24 21:08:25 +08:00
RateLimitTests: Remove now-unnecessary API key logic.
This commit is contained in:
parent
03bf8893e2
commit
63f92adfbb
@ -66,7 +66,7 @@ class RateLimitTests(ZulipTestCase):
|
||||
settings.RATE_LIMITING = False
|
||||
remove_ratelimit_rule(1, 5)
|
||||
|
||||
def send_api_message(self, email, api_key, content):
|
||||
def send_api_message(self, email, content):
|
||||
# type: (text_type, text_type, text_type) -> HttpResponse
|
||||
return self.client_post("/api/v1/messages", {"type": "stream",
|
||||
"to": "Verona",
|
||||
@ -80,9 +80,8 @@ class RateLimitTests(ZulipTestCase):
|
||||
email = "hamlet@zulip.com"
|
||||
user = get_user_profile_by_email(email)
|
||||
clear_user_history(user)
|
||||
api_key = self.get_api_key(email)
|
||||
|
||||
result = self.send_api_message(email, api_key, "some stuff")
|
||||
result = self.send_api_message(email, "some stuff")
|
||||
self.assertTrue('X-RateLimit-Remaining' in result)
|
||||
self.assertTrue('X-RateLimit-Limit' in result)
|
||||
self.assertTrue('X-RateLimit-Reset' in result)
|
||||
@ -92,11 +91,10 @@ class RateLimitTests(ZulipTestCase):
|
||||
email = "hamlet@zulip.com"
|
||||
user = get_user_profile_by_email(email)
|
||||
clear_user_history(user)
|
||||
api_key = self.get_api_key(email)
|
||||
result = self.send_api_message(email, api_key, "some stuff")
|
||||
result = self.send_api_message(email, "some stuff")
|
||||
limit = int(result['X-RateLimit-Remaining'])
|
||||
|
||||
result = self.send_api_message(email, api_key, "some stuff 2")
|
||||
result = self.send_api_message(email, "some stuff 2")
|
||||
newlimit = int(result['X-RateLimit-Remaining'])
|
||||
self.assertEqual(limit, newlimit + 1)
|
||||
|
||||
@ -107,9 +105,8 @@ class RateLimitTests(ZulipTestCase):
|
||||
user = get_user_profile_by_email(email)
|
||||
clear_user_history(user)
|
||||
|
||||
api_key = self.get_api_key(email)
|
||||
for i in range(6):
|
||||
result = self.send_api_message(email, api_key, "some stuff %s" % (i,))
|
||||
result = self.send_api_message(email, "some stuff %s" % (i,))
|
||||
|
||||
self.assertEqual(result.status_code, 429)
|
||||
json = ujson.loads(result.content)
|
||||
@ -123,6 +120,6 @@ class RateLimitTests(ZulipTestCase):
|
||||
# after some time has passed.
|
||||
time.sleep(1)
|
||||
|
||||
result = self.send_api_message(email, api_key, "Good message")
|
||||
result = self.send_api_message(email, "Good message")
|
||||
|
||||
self.assert_json_success(result)
|
||||
|
||||
Loading…
Reference in New Issue
Block a user