# frozen_string_literal: true require "spec_helper" describe Projects::RepositoriesController do let(:project) { create(:project, :repository) } describe "GET archive" do context 'as a guest' do it 'responds with redirect in correct format' do get :archive, params: { namespace_id: project.namespace, project_id: project, id: "master" }, format: "zip" expect(response.header["Content-Type"]).to start_with('text/html') expect(response).to be_redirect end end context 'as a user' do let(:user) { create(:user) } before do project.add_developer(user) sign_in(user) end it "uses Gitlab::Workhorse" do get :archive, params: { namespace_id: project.namespace, project_id: project, id: "master" }, format: "zip" expect(response.header[Gitlab::Workhorse::SEND_DATA_HEADER]).to start_with("git-archive:") end it 'responds with redirect to the short name archive if fully qualified' do get :archive, params: { namespace_id: project.namespace, project_id: project, id: "master/#{project.path}-master" }, format: "zip" expect(assigns(:ref)).to eq("master") expect(response.header[Gitlab::Workhorse::SEND_DATA_HEADER]).to start_with("git-archive:") end it 'handles legacy queries with no ref' do get :archive, params: { namespace_id: project.namespace, project_id: project }, format: "zip" expect(response.header[Gitlab::Workhorse::SEND_DATA_HEADER]).to start_with("git-archive:") end it 'handles legacy queries with the ref specified as ref in params' do get :archive, params: { namespace_id: project.namespace, project_id: project, ref: 'feature' }, format: 'zip' expect(response).to have_gitlab_http_status(200) expect(assigns(:ref)).to eq('feature') expect(response.header[Gitlab::Workhorse::SEND_DATA_HEADER]).to start_with("git-archive:") end it 'handles legacy queries with the ref specified as id in params' do get :archive, params: { namespace_id: project.namespace, project_id: project, id: 'feature' }, format: 'zip' expect(response).to have_gitlab_http_status(200) expect(assigns(:ref)).to eq('feature') expect(response.header[Gitlab::Workhorse::SEND_DATA_HEADER]).to start_with("git-archive:") end it 'prioritizes the id param over the ref param when both are specified' do get :archive, params: { namespace_id: project.namespace, project_id: project, id: 'feature', ref: 'feature_conflict' }, format: 'zip' expect(response).to have_gitlab_http_status(200) expect(assigns(:ref)).to eq('feature') expect(response.header[Gitlab::Workhorse::SEND_DATA_HEADER]).to start_with("git-archive:") end context "when the service raises an error" do before do allow(Gitlab::Workhorse).to receive(:send_git_archive).and_raise("Archive failed") end it "renders Not Found" do get :archive, params: { namespace_id: project.namespace, project_id: project, id: "master" }, format: "zip" expect(response).to have_gitlab_http_status(404) end end describe 'caching' do it 'sets appropriate caching headers' do get_archive expect(response).to have_gitlab_http_status(200) expect(response.header['ETag']).to be_present expect(response.header['Cache-Control']).to include('max-age=60, private') end context 'when project is public' do let(:project) { create(:project, :repository, :public) } it 'sets appropriate caching headers' do get_archive expect(response).to have_gitlab_http_status(200) expect(response.header['ETag']).to be_present expect(response.header['Cache-Control']).to include('max-age=60, public') end end context 'when ref is a commit SHA' do it 'max-age is set to 3600 in Cache-Control header' do get_archive('ddd0f15ae83993f5cb66a927a28673882e99100b') expect(response).to have_gitlab_http_status(200) expect(response.header['Cache-Control']).to include('max-age=3600') end end context 'when If-None-Modified header is set' do it 'returns a 304 status' do # Get the archive cached first get_archive request.headers['If-None-Match'] = response.headers['ETag'] get_archive expect(response).to have_gitlab_http_status(304) end end def get_archive(id = 'feature') get :archive, params: { namespace_id: project.namespace, project_id: project, id: id }, format: 'zip' end end end end end