Skip to content
Snippets Groups Projects
Unverified Commit 40e68c11 authored by Andrejs Cunskis's avatar Andrejs Cunskis Committed by GitLab
Browse files

Merge branch 'fix-rate-limit-e2e' into 'master'

Increase rate limit for rate limit per user e2e tests

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/169269



Merged-by: default avatarAndrejs Cunskis <acunskis@gitlab.com>
Approved-by: default avatarNick Westbury <nwestbury@gitlab.com>
Approved-by: default avatarAndrejs Cunskis <acunskis@gitlab.com>
Co-authored-by: default avatarSofia Vistas <svistas@gitlab.com>
parents eb2c0d80 862f74a1
Branches andrey-remove-group-caching
No related tags found
No related merge requests found
Loading
Loading
@@ -16,11 +16,11 @@ module QA
it 'throttles authenticated api requests by user',
testcase: 'https://gitlab.com/gitlab-org/gitlab/-/quality/test_cases/347881' do
with_application_settings(
throttle_authenticated_api_requests_per_period: 5,
throttle_authenticated_api_requests_per_period: 100,
throttle_authenticated_api_period_in_seconds: 60,
throttle_authenticated_api_enabled: true
) do
5.times do
100.times do
res = RestClient.get request.url
expect(res.code).to be(200)
end
Loading
Loading
@@ -30,15 +30,15 @@ module QA
end
end
end
end
 
private
private
 
def with_application_settings(**hargs)
QA::Runtime::ApplicationSettings.set_application_settings(**hargs)
yield
ensure
QA::Runtime::ApplicationSettings.restore_application_settings(*hargs.keys)
def with_application_settings(**hargs)
QA::Runtime::ApplicationSettings.set_application_settings(**hargs)
yield
ensure
QA::Runtime::ApplicationSettings.restore_application_settings(*hargs.keys)
end
end
end
end
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment