368 lines
13 KiB
Ruby
368 lines
13 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
require 'spec_helper'
|
|
|
|
RSpec.describe API::ImportGithub, feature_category: :importers do
|
|
let(:token) { "ghp_asdasd12345" }
|
|
let(:provider) { :github }
|
|
let(:access_params) { { github_access_token: token } }
|
|
let(:provider_username) { user.username }
|
|
let(:provider_user) { double('provider', login: provider_username).as_null_object }
|
|
let(:provider_repo) do
|
|
{
|
|
name: 'vim',
|
|
full_name: "#{provider_username}/vim",
|
|
owner: double('provider', login: provider_username),
|
|
description: 'provider',
|
|
private: false,
|
|
clone_url: 'https://fake.url/vim.git',
|
|
has_wiki: true
|
|
}
|
|
end
|
|
|
|
let(:client) { double('client', user: provider_user, repository: provider_repo) }
|
|
|
|
before do
|
|
Grape::Endpoint.before_each do |endpoint|
|
|
allow(endpoint).to receive(:client).and_return(client)
|
|
end
|
|
end
|
|
|
|
after do
|
|
Grape::Endpoint.before_each nil
|
|
end
|
|
|
|
describe "POST /import/github" do
|
|
let_it_be(:user) { create(:user) }
|
|
let_it_be(:project) { create(:project) }
|
|
let(:scopes) { ["repo", "read:org"] }
|
|
|
|
before do
|
|
allow(client).to receive_message_chain(:octokit, :rate_limit)
|
|
allow(client).to receive_message_chain(:octokit, :repository).and_return({ status: 200 })
|
|
|
|
allow(::Gitlab::ApplicationRateLimiter).to receive(:throttled?).and_return(false)
|
|
end
|
|
|
|
it 'rejects requests when Github Importer is disabled' do
|
|
stub_application_setting(import_sources: nil)
|
|
|
|
post api("/import/github", user), params: {
|
|
target_namespace: user.namespace_path,
|
|
personal_access_token: token,
|
|
repo_id: non_existing_record_id
|
|
}
|
|
|
|
expect(response).to have_gitlab_http_status(:forbidden)
|
|
end
|
|
|
|
it 'returns 201 response when the project is imported successfully' do
|
|
allow(Gitlab::LegacyGithubImport::ProjectCreator)
|
|
.to receive(:new).with(provider_repo, provider_repo[:name], user.namespace, user, type: provider, **access_params)
|
|
.and_return(double(execute: project))
|
|
|
|
post api("/import/github", user), params: {
|
|
target_namespace: user.namespace_path,
|
|
personal_access_token: token,
|
|
repo_id: non_existing_record_id
|
|
}
|
|
expect(response).to have_gitlab_http_status(:created)
|
|
expect(json_response).to be_a Hash
|
|
expect(json_response['name']).to eq(project.name)
|
|
end
|
|
|
|
it 'returns 201 response when the project is imported successfully from GHE' do
|
|
allow(Gitlab::LegacyGithubImport::ProjectCreator)
|
|
.to receive(:new).with(provider_repo, provider_repo[:name], user.namespace, user, type: provider, **access_params)
|
|
.and_return(double(execute: project))
|
|
|
|
post api("/import/github", user), params: {
|
|
target_namespace: user.namespace_path,
|
|
personal_access_token: token,
|
|
repo_id: non_existing_record_id,
|
|
github_hostname: "https://github.somecompany.com/",
|
|
optional_stages: { attachments_import: true }
|
|
}
|
|
expect(response).to have_gitlab_http_status(:created)
|
|
expect(json_response).to be_a Hash
|
|
expect(json_response['name']).to eq(project.name)
|
|
end
|
|
|
|
it 'returns 422 response when user can not create projects in the chosen namespace' do
|
|
other_namespace = create(:group, name: 'other_namespace')
|
|
|
|
post api("/import/github", user), params: {
|
|
target_namespace: other_namespace.name,
|
|
personal_access_token: token,
|
|
repo_id: non_existing_record_id
|
|
}
|
|
|
|
expect(response).to have_gitlab_http_status(:unprocessable_entity)
|
|
end
|
|
|
|
context 'when target_namespace is blank' do
|
|
it 'returns 400 response' do
|
|
allow(Gitlab::LegacyGithubImport::ProjectCreator)
|
|
.to receive(:new).with(provider_repo, provider_repo[:name], user.namespace, user, type: provider, **access_params)
|
|
.and_return(double(execute: project))
|
|
|
|
post api("/import/github", user), params: {
|
|
target_namespace: '',
|
|
personal_access_token: token,
|
|
repo_id: non_existing_record_id
|
|
}
|
|
|
|
expect(response).to have_gitlab_http_status(:bad_request)
|
|
expect(json_response['error']).to eq 'target_namespace is empty'
|
|
end
|
|
end
|
|
|
|
context 'when unauthenticated user' do
|
|
it 'returns 403 response' do
|
|
post api("/import/github"), params: {
|
|
target_namespace: user.namespace_path,
|
|
personal_access_token: token,
|
|
repo_id: non_existing_record_id
|
|
}
|
|
|
|
expect(response).to have_gitlab_http_status(:unauthorized)
|
|
end
|
|
end
|
|
|
|
context 'with invalid timeout stategy' do
|
|
it 'returns 400 response' do
|
|
post api("/import/github", user), params: {
|
|
target_namespace: user.namespace_path,
|
|
personal_access_token: token,
|
|
repo_id: non_existing_record_id,
|
|
timeout_strategy: "invalid_strategy"
|
|
}
|
|
|
|
expect(response).to have_gitlab_http_status(:bad_request)
|
|
end
|
|
end
|
|
|
|
context 'with a valid token' do
|
|
before do
|
|
allow(client).to receive_message_chain(:octokit, :repository).and_return({ status: 200 })
|
|
end
|
|
|
|
it 'proceeds with the import' do
|
|
allow(Gitlab::LegacyGithubImport::ProjectCreator)
|
|
.to receive(:new).with(provider_repo, provider_repo[:name], user.namespace, user, type: provider, **access_params)
|
|
.and_return(double(execute: project))
|
|
|
|
post api("/import/github", user), params: {
|
|
target_namespace: user.namespace_path,
|
|
personal_access_token: token,
|
|
repo_id: non_existing_record_id
|
|
}
|
|
|
|
expect(response).to have_gitlab_http_status(:created)
|
|
expect(json_response).to be_a Hash
|
|
expect(json_response['name']).to eq(project.name)
|
|
end
|
|
end
|
|
|
|
context 'with an invalid token' do
|
|
let(:exception) { Octokit::Forbidden.new(status: 403, body: 'Forbidden') }
|
|
let(:docs_link) do
|
|
ActionController::Base.helpers.link_to(
|
|
_('documentation'),
|
|
Rails.application.routes.url_helpers.help_page_url(
|
|
'user/project/import/github.md', anchor: 'use-a-github-personal-access-token'
|
|
),
|
|
target: '_blank',
|
|
rel: 'noopener noreferrer'
|
|
)
|
|
end
|
|
|
|
context 'when collaborators import is nil' do
|
|
before do
|
|
allow(client).to receive_message_chain(:octokit, :repository).and_raise(exception)
|
|
end
|
|
|
|
it 'raises an error' do
|
|
expect(Gitlab::LegacyGithubImport::ProjectCreator).not_to receive(:new)
|
|
|
|
post api("/import/github", user), params: {
|
|
target_namespace: user.namespace_path,
|
|
personal_access_token: token,
|
|
repo_id: non_existing_record_id
|
|
}
|
|
|
|
expect(response).to have_gitlab_http_status(:unprocessable_entity)
|
|
expect(json_response['errors']).to eq("Your GitHub personal access token does not have read access to the repository. " \
|
|
"Please use a classic GitHub personal access token with the `repo` scope. " \
|
|
"Fine-grained tokens are not supported.")
|
|
end
|
|
end
|
|
|
|
context 'when collaborators import is false' do
|
|
before do
|
|
allow(client).to receive_message_chain(:octokit, :repository).and_raise(exception)
|
|
end
|
|
|
|
it 'raises an error' do
|
|
expect(Gitlab::LegacyGithubImport::ProjectCreator).not_to receive(:new)
|
|
|
|
post api("/import/github", user), params: {
|
|
target_namespace: user.namespace_path,
|
|
personal_access_token: token,
|
|
repo_id: non_existing_record_id,
|
|
optional_stages: { collaborators_import: false }
|
|
}
|
|
|
|
expect(response).to have_gitlab_http_status(:unprocessable_entity)
|
|
expect(json_response['errors']).to eq("Your GitHub personal access token does not have read access to the repository. " \
|
|
"Please use a classic GitHub personal access token with the `repo` scope. " \
|
|
"Fine-grained tokens are not supported.")
|
|
end
|
|
end
|
|
|
|
context 'when collaborators import is true' do
|
|
before do
|
|
allow(client).to receive_message_chain(:octokit, :repository).and_return({ status: 200 })
|
|
allow(client).to receive_message_chain(:octokit, :collaborators).and_raise(exception)
|
|
end
|
|
|
|
it 'raises an error' do
|
|
expect(Gitlab::LegacyGithubImport::ProjectCreator).not_to receive(:new)
|
|
|
|
post api("/import/github", user), params: {
|
|
target_namespace: user.namespace_path,
|
|
personal_access_token: token,
|
|
repo_id: non_existing_record_id,
|
|
optional_stages: { collaborators_import: true }
|
|
}
|
|
|
|
expect(response).to have_gitlab_http_status(:unprocessable_entity)
|
|
expect(json_response['errors']).to eq("Your GitHub personal access token does not have read access to collaborators. " \
|
|
"Please use a classic GitHub personal access token with the `read:org` scope. " \
|
|
"Fine-grained tokens are not supported.")
|
|
end
|
|
end
|
|
end
|
|
|
|
context 'when request exceeds rate limits' do
|
|
it 'throttles the endpoint' do
|
|
allow(::Gitlab::ApplicationRateLimiter).to receive(:throttled?).and_return(true)
|
|
|
|
post api("/import/github", user), params: {
|
|
target_namespace: user.namespace_path,
|
|
personal_access_token: token,
|
|
repo_id: non_existing_record_id
|
|
}
|
|
|
|
expect(response).to have_gitlab_http_status(:too_many_requests)
|
|
expect(json_response['errors']).to eq('This endpoint has been requested too many times. Try again later.')
|
|
end
|
|
end
|
|
end
|
|
|
|
describe "POST /import/github/cancel" do
|
|
let_it_be(:user) { create(:user) }
|
|
let_it_be(:project) { create(:project, :import_started, import_type: 'github', import_url: 'https://fake.url') }
|
|
|
|
context 'when project import was canceled' do
|
|
before do
|
|
allow(Import::Github::CancelProjectImportService)
|
|
.to receive(:new).with(project, user)
|
|
.and_return(double(execute: { status: :success, project: project }))
|
|
end
|
|
|
|
it 'returns success' do
|
|
post api("/import/github/cancel", user), params: {
|
|
project_id: project.id
|
|
}
|
|
|
|
expect(response).to have_gitlab_http_status(:ok)
|
|
end
|
|
end
|
|
|
|
context 'when project import was not canceled' do
|
|
before do
|
|
allow(Import::Github::CancelProjectImportService)
|
|
.to receive(:new).with(project, user)
|
|
.and_return(double(execute: { status: :error, message: 'The import cannot be canceled because it is finished', http_status: :bad_request }))
|
|
end
|
|
|
|
it 'returns error' do
|
|
post api("/import/github/cancel", user), params: {
|
|
project_id: project.id
|
|
}
|
|
|
|
expect(response).to have_gitlab_http_status(:bad_request)
|
|
expect(json_response['message']).to eq('The import cannot be canceled because it is finished')
|
|
end
|
|
end
|
|
|
|
context 'when unauthenticated user' do
|
|
it 'returns 403 response' do
|
|
post api("/import/github/cancel"), params: {
|
|
project_id: project.id
|
|
}
|
|
|
|
expect(response).to have_gitlab_http_status(:unauthorized)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe 'POST /import/github/gists' do
|
|
let_it_be(:user) { create(:user) }
|
|
let(:params) { { personal_access_token: token } }
|
|
|
|
context 'when gists import was started' do
|
|
before do
|
|
allow(Import::Github::GistsImportService)
|
|
.to receive(:new).with(user, client, access_params)
|
|
.and_return(double(execute: { status: :success }))
|
|
end
|
|
|
|
it 'returns 202' do
|
|
post api('/import/github/gists', user), params: params
|
|
|
|
expect(response).to have_gitlab_http_status(:accepted)
|
|
end
|
|
end
|
|
|
|
context 'when gists import is in progress' do
|
|
before do
|
|
allow(Import::Github::GistsImportService)
|
|
.to receive(:new).with(user, client, access_params)
|
|
.and_return(double(execute: { status: :error, message: 'Import already in progress', http_status: :unprocessable_entity }))
|
|
end
|
|
|
|
it 'returns 422 error' do
|
|
post api('/import/github/gists', user), params: params
|
|
|
|
expect(response).to have_gitlab_http_status(:unprocessable_entity)
|
|
expect(json_response['errors']).to eq('Import already in progress')
|
|
end
|
|
end
|
|
|
|
context 'when unauthenticated user' do
|
|
it 'returns 403 error' do
|
|
post api('/import/github/gists'), params: params
|
|
|
|
expect(response).to have_gitlab_http_status(:unauthorized)
|
|
end
|
|
end
|
|
|
|
context 'when rate limit reached' do
|
|
before do
|
|
allow_next_instance_of(Import::Github::GistsImportService) do |service|
|
|
allow(service).to receive(:execute)
|
|
.and_return({ status: :error, message: 'GitHub API rate limit exceeded', http_status: 429 })
|
|
end
|
|
end
|
|
|
|
it 'returns 429 error' do
|
|
post api('/import/github/gists', user), params: params
|
|
|
|
expect(response).to have_gitlab_http_status(:too_many_requests)
|
|
end
|
|
end
|
|
end
|
|
end
|