2017-08-17 22:00:37 +05:30
|
|
|
require 'spec_helper'
|
|
|
|
|
|
|
|
describe Projects::ArtifactsController do
|
2018-03-17 18:26:18 +05:30
|
|
|
let(:user) { project.owner }
|
|
|
|
set(:project) { create(:project, :repository, :public) }
|
2017-08-17 22:00:37 +05:30
|
|
|
|
|
|
|
let(:pipeline) do
|
|
|
|
create(:ci_pipeline,
|
|
|
|
project: project,
|
|
|
|
sha: project.commit.sha,
|
|
|
|
ref: project.default_branch,
|
|
|
|
status: 'success')
|
|
|
|
end
|
|
|
|
|
2017-09-10 17:25:29 +05:30
|
|
|
let(:job) { create(:ci_build, :success, :artifacts, pipeline: pipeline) }
|
2017-08-17 22:00:37 +05:30
|
|
|
|
|
|
|
before do
|
|
|
|
sign_in(user)
|
|
|
|
end
|
|
|
|
|
|
|
|
describe 'GET download' do
|
2018-12-05 23:21:45 +05:30
|
|
|
def download_artifact(extra_params = {})
|
|
|
|
params = { namespace_id: project.namespace, project_id: project, job_id: job }.merge(extra_params)
|
2017-08-17 22:00:37 +05:30
|
|
|
|
2018-12-05 23:21:45 +05:30
|
|
|
get :download, params
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when no file type is supplied' do
|
|
|
|
it 'sends the artifacts file' do
|
|
|
|
expect(controller).to receive(:send_file).with(job.artifacts_file.path, hash_including(disposition: 'attachment')).and_call_original
|
|
|
|
|
|
|
|
download_artifact
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when a file type is supplied' do
|
|
|
|
context 'when an invalid file type is supplied' do
|
|
|
|
let(:file_type) { 'invalid' }
|
|
|
|
|
|
|
|
it 'returns 404' do
|
|
|
|
download_artifact(file_type: file_type)
|
|
|
|
|
|
|
|
expect(response).to have_gitlab_http_status(404)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when codequality file type is supplied' do
|
|
|
|
let(:file_type) { 'codequality' }
|
|
|
|
|
2018-12-13 13:39:08 +05:30
|
|
|
context 'when file is stored locally' do
|
|
|
|
before do
|
|
|
|
create(:ci_job_artifact, :codequality, job: job)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'sends the codequality report' do
|
|
|
|
expect(controller).to receive(:send_file).with(job.job_artifacts_codequality.file.path, hash_including(disposition: 'attachment')).and_call_original
|
|
|
|
|
|
|
|
download_artifact(file_type: file_type)
|
|
|
|
end
|
2018-12-05 23:21:45 +05:30
|
|
|
end
|
|
|
|
|
2018-12-13 13:39:08 +05:30
|
|
|
context 'when file is stored remotely' do
|
|
|
|
before do
|
|
|
|
stub_artifacts_object_storage
|
|
|
|
create(:ci_job_artifact, :remote_store, :codequality, job: job)
|
|
|
|
end
|
2018-12-05 23:21:45 +05:30
|
|
|
|
2018-12-13 13:39:08 +05:30
|
|
|
it 'sends the codequality report' do
|
|
|
|
expect(controller).to receive(:redirect_to).and_call_original
|
|
|
|
|
|
|
|
download_artifact(file_type: file_type)
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when proxied' do
|
|
|
|
it 'sends the codequality report' do
|
|
|
|
expect(Gitlab::Workhorse).to receive(:send_url).and_call_original
|
|
|
|
|
|
|
|
download_artifact(file_type: file_type, proxy: true)
|
|
|
|
end
|
|
|
|
end
|
2018-12-05 23:21:45 +05:30
|
|
|
end
|
|
|
|
end
|
2017-08-17 22:00:37 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe 'GET browse' do
|
|
|
|
context 'when the directory exists' do
|
|
|
|
it 'renders the browse view' do
|
2017-09-10 17:25:29 +05:30
|
|
|
get :browse, namespace_id: project.namespace, project_id: project, job_id: job, path: 'other_artifacts_0.1.2'
|
2017-08-17 22:00:37 +05:30
|
|
|
|
|
|
|
expect(response).to render_template('projects/artifacts/browse')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the directory does not exist' do
|
|
|
|
it 'responds Not Found' do
|
2017-09-10 17:25:29 +05:30
|
|
|
get :browse, namespace_id: project.namespace, project_id: project, job_id: job, path: 'unknown'
|
2017-08-17 22:00:37 +05:30
|
|
|
|
|
|
|
expect(response).to be_not_found
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe 'GET file' do
|
2018-03-17 18:26:18 +05:30
|
|
|
before do
|
|
|
|
allow(Gitlab.config.pages).to receive(:enabled).and_return(true)
|
|
|
|
end
|
2017-08-17 22:00:37 +05:30
|
|
|
|
2018-03-17 18:26:18 +05:30
|
|
|
context 'when the file is served by GitLab Pages' do
|
|
|
|
before do
|
|
|
|
allow(Gitlab.config.pages).to receive(:artifacts_server).and_return(true)
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the file exists' do
|
|
|
|
it 'renders the file view' do
|
|
|
|
get :file, namespace_id: project.namespace, project_id: project, job_id: job, path: 'ci_artifacts.txt'
|
|
|
|
|
|
|
|
expect(response).to have_gitlab_http_status(302)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the file does not exist' do
|
|
|
|
it 'responds Not Found' do
|
|
|
|
get :file, namespace_id: project.namespace, project_id: project, job_id: job, path: 'unknown'
|
|
|
|
|
|
|
|
expect(response).to be_not_found
|
|
|
|
end
|
2017-08-17 22:00:37 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2018-03-17 18:26:18 +05:30
|
|
|
context 'when the file is served through Rails' do
|
|
|
|
context 'when the file exists' do
|
|
|
|
it 'renders the file view' do
|
|
|
|
get :file, namespace_id: project.namespace, project_id: project, job_id: job, path: 'ci_artifacts.txt'
|
2017-08-17 22:00:37 +05:30
|
|
|
|
2018-03-17 18:26:18 +05:30
|
|
|
expect(response).to have_gitlab_http_status(:ok)
|
|
|
|
expect(response).to render_template('projects/artifacts/file')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the file does not exist' do
|
|
|
|
it 'responds Not Found' do
|
|
|
|
get :file, namespace_id: project.namespace, project_id: project, job_id: job, path: 'unknown'
|
|
|
|
|
|
|
|
expect(response).to be_not_found
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the project is private' do
|
|
|
|
let(:private_project) { create(:project, :repository, :private) }
|
|
|
|
let(:pipeline) { create(:ci_pipeline, project: private_project) }
|
|
|
|
let(:job) { create(:ci_build, :success, :artifacts, pipeline: pipeline) }
|
|
|
|
|
|
|
|
before do
|
|
|
|
private_project.add_developer(user)
|
|
|
|
|
|
|
|
allow(Gitlab.config.pages).to receive(:artifacts_server).and_return(true)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not redirect the request' do
|
|
|
|
get :file, namespace_id: private_project.namespace, project_id: private_project, job_id: job, path: 'ci_artifacts.txt'
|
|
|
|
|
|
|
|
expect(response).to have_gitlab_http_status(:ok)
|
|
|
|
expect(response).to render_template('projects/artifacts/file')
|
2017-08-17 22:00:37 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe 'GET raw' do
|
2018-03-17 18:26:18 +05:30
|
|
|
subject { get(:raw, namespace_id: project.namespace, project_id: project, job_id: job, path: path) }
|
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
context 'when the file exists' do
|
2018-03-17 18:26:18 +05:30
|
|
|
let(:path) { 'ci_artifacts.txt' }
|
2017-08-17 22:00:37 +05:30
|
|
|
|
2018-03-17 18:26:18 +05:30
|
|
|
shared_examples 'a valid file' do
|
|
|
|
it 'serves the file using workhorse' do
|
|
|
|
subject
|
2017-08-17 22:00:37 +05:30
|
|
|
|
2018-03-17 18:26:18 +05:30
|
|
|
expect(response).to have_gitlab_http_status(200)
|
|
|
|
expect(send_data).to start_with('artifacts-entry:')
|
2017-08-17 22:00:37 +05:30
|
|
|
|
2018-03-17 18:26:18 +05:30
|
|
|
expect(params.keys).to eq(%w(Archive Entry))
|
|
|
|
expect(params['Archive']).to start_with(archive_path)
|
|
|
|
# On object storage, the URL can end with a query string
|
|
|
|
expect(params['Archive']).to match(/build_artifacts.zip(\?[^?]+)?$/)
|
|
|
|
expect(params['Entry']).to eq(Base64.encode64('ci_artifacts.txt'))
|
|
|
|
end
|
2017-08-17 22:00:37 +05:30
|
|
|
|
2018-03-17 18:26:18 +05:30
|
|
|
def send_data
|
|
|
|
response.headers[Gitlab::Workhorse::SEND_DATA_HEADER]
|
|
|
|
end
|
2017-08-17 22:00:37 +05:30
|
|
|
|
2018-03-17 18:26:18 +05:30
|
|
|
def params
|
|
|
|
@params ||= begin
|
|
|
|
base64_params = send_data.sub(/\Aartifacts\-entry:/, '')
|
|
|
|
JSON.parse(Base64.urlsafe_decode64(base64_params))
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2017-08-17 22:00:37 +05:30
|
|
|
|
2018-03-17 18:26:18 +05:30
|
|
|
context 'when using local file storage' do
|
|
|
|
it_behaves_like 'a valid file' do
|
|
|
|
let(:job) { create(:ci_build, :success, :artifacts, pipeline: pipeline) }
|
2018-05-09 12:01:36 +05:30
|
|
|
let(:store) { ObjectStorage::Store::LOCAL }
|
2018-03-17 18:26:18 +05:30
|
|
|
let(:archive_path) { JobArtifactUploader.root }
|
|
|
|
end
|
2017-08-17 22:00:37 +05:30
|
|
|
end
|
2018-05-09 12:01:36 +05:30
|
|
|
|
|
|
|
context 'when using remote file storage' do
|
|
|
|
before do
|
|
|
|
stub_artifacts_object_storage
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'a valid file' do
|
|
|
|
let!(:artifact) { create(:ci_job_artifact, :archive, :remote_store, job: job) }
|
|
|
|
let!(:job) { create(:ci_build, :success, pipeline: pipeline) }
|
|
|
|
let(:store) { ObjectStorage::Store::REMOTE }
|
|
|
|
let(:archive_path) { 'https://' }
|
|
|
|
end
|
|
|
|
end
|
2017-08-17 22:00:37 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe 'GET latest_succeeded' do
|
2017-09-10 17:25:29 +05:30
|
|
|
def params_from_ref(ref = pipeline.ref, job_name = job.name, path = 'browse')
|
2017-08-17 22:00:37 +05:30
|
|
|
{
|
|
|
|
namespace_id: project.namespace,
|
|
|
|
project_id: project,
|
|
|
|
ref_name_and_path: File.join(ref, path),
|
2017-09-10 17:25:29 +05:30
|
|
|
job: job_name
|
2017-08-17 22:00:37 +05:30
|
|
|
}
|
|
|
|
end
|
|
|
|
|
2017-09-10 17:25:29 +05:30
|
|
|
context 'cannot find the job' do
|
2017-08-17 22:00:37 +05:30
|
|
|
shared_examples 'not found' do
|
2018-03-17 18:26:18 +05:30
|
|
|
it { expect(response).to have_gitlab_http_status(:not_found) }
|
2017-08-17 22:00:37 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
context 'has no such ref' do
|
|
|
|
before do
|
2017-09-10 17:25:29 +05:30
|
|
|
get :latest_succeeded, params_from_ref('TAIL', job.name)
|
2017-08-17 22:00:37 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'not found'
|
|
|
|
end
|
|
|
|
|
2017-09-10 17:25:29 +05:30
|
|
|
context 'has no such job' do
|
2017-08-17 22:00:37 +05:30
|
|
|
before do
|
|
|
|
get :latest_succeeded, params_from_ref(pipeline.ref, 'NOBUILD')
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'not found'
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'has no path' do
|
|
|
|
before do
|
2017-09-10 17:25:29 +05:30
|
|
|
get :latest_succeeded, params_from_ref(pipeline.sha, job.name, '')
|
2017-08-17 22:00:37 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'not found'
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2017-09-10 17:25:29 +05:30
|
|
|
context 'found the job and redirect' do
|
|
|
|
shared_examples 'redirect to the job' do
|
2017-08-17 22:00:37 +05:30
|
|
|
it 'redirects' do
|
2017-09-10 17:25:29 +05:30
|
|
|
path = browse_project_job_artifacts_path(project, job)
|
2017-08-17 22:00:37 +05:30
|
|
|
|
|
|
|
expect(response).to redirect_to(path)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'with regular branch' do
|
|
|
|
before do
|
|
|
|
pipeline.update(ref: 'master',
|
|
|
|
sha: project.commit('master').sha)
|
|
|
|
|
|
|
|
get :latest_succeeded, params_from_ref('master')
|
|
|
|
end
|
|
|
|
|
2017-09-10 17:25:29 +05:30
|
|
|
it_behaves_like 'redirect to the job'
|
2017-08-17 22:00:37 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
context 'with branch name containing slash' do
|
|
|
|
before do
|
|
|
|
pipeline.update(ref: 'improve/awesome',
|
|
|
|
sha: project.commit('improve/awesome').sha)
|
|
|
|
|
|
|
|
get :latest_succeeded, params_from_ref('improve/awesome')
|
|
|
|
end
|
|
|
|
|
2017-09-10 17:25:29 +05:30
|
|
|
it_behaves_like 'redirect to the job'
|
2017-08-17 22:00:37 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
context 'with branch name and path containing slashes' do
|
|
|
|
before do
|
|
|
|
pipeline.update(ref: 'improve/awesome',
|
|
|
|
sha: project.commit('improve/awesome').sha)
|
|
|
|
|
2017-09-10 17:25:29 +05:30
|
|
|
get :latest_succeeded, params_from_ref('improve/awesome', job.name, 'file/README.md')
|
2017-08-17 22:00:37 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
it 'redirects' do
|
2017-09-10 17:25:29 +05:30
|
|
|
path = file_project_job_artifacts_path(project, job, 'README.md')
|
2017-08-17 22:00:37 +05:30
|
|
|
|
|
|
|
expect(response).to redirect_to(path)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|