Commit 5aa9f137 authored by Michael Kozono's avatar Michael Kozono

Merge branch 'rs-remove-remote_mirrors_api' into 'master'

Remove `remote_mirrors_api` feature flag

See merge request gitlab-org/gitlab!26695
parents 72faadc1 02787532
......@@ -5,9 +5,6 @@ module API
include PaginationParams
before do
# TODO: Remove flag: https://gitlab.com/gitlab-org/gitlab/issues/38121
not_found! unless Feature.enabled?(:remote_mirrors_api, user_project)
unauthorized! unless can?(current_user, :admin_remote_mirror, user_project)
end
......
......@@ -24,19 +24,6 @@ describe API::RemoteMirrors do
expect(response).to have_gitlab_http_status(:success)
expect(response).to match_response_schema('remote_mirrors')
end
# TODO: Remove flag: https://gitlab.com/gitlab-org/gitlab/issues/38121
context 'with the `remote_mirrors_api` feature disabled' do
before do
stub_feature_flags(remote_mirrors_api: false)
end
it 'responds with `not_found`' do
get api(route, user)
expect(response).to have_gitlab_http_status(:not_found)
end
end
end
describe 'POST /projects/:id/remote_mirrors' do
......@@ -132,18 +119,5 @@ describe API::RemoteMirrors do
expect(json_response['keep_divergent_refs']).to eq(true)
end
end
# TODO: Remove flag: https://gitlab.com/gitlab-org/gitlab/issues/38121
context 'with the `remote_mirrors_api` feature disabled' do
before do
stub_feature_flags(remote_mirrors_api: false)
end
it 'responds with `not_found`' do
put api(route[mirror.id], user)
expect(response).to have_gitlab_http_status(:not_found)
end
end
end
end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment