Merge branch '4635-unify-jobs-and-builds-sepc' into 'master'

Unify and split CE/EE for spec/requests/api/jobs_spec.rb

See merge request gitlab-org/gitlab-ce!16804

spec/requests/api/jobs_spec.rb

@@ -300,14 +300,7 @@ describe API::Jobs do
   end
 
   describe 'GET /projects/:id/jobs/:job_id/artifacts' do
-    before do
-      get api("/projects/#{project.id}/jobs/#{job.id}/artifacts", api_user)
-    end
-
-    context 'job with artifacts' do
-      let(:job) { create(:ci_build, :artifacts, pipeline: pipeline) }
-
-      context 'authorized user' do
+    shared_examples 'downloads artifact' do
       let(:download_headers) do
         { 'Content-Transfer-Encoding' => 'binary',
           'Content-Disposition' => 'attachment; filename=ci_build_artifacts.zip' }
@@ -320,24 +313,40 @@ describe API::Jobs do
       end
     end
 
-      context 'when anonymous user is accessing private artifacts' do
+    context 'normal authentication' do
+      context 'job with artifacts' do
+        context 'when artifacts are stored locally' do
+          let(:job) { create(:ci_build, :artifacts, pipeline: pipeline) }
+
+          before do
+            get api("/projects/#{project.id}/jobs/#{job.id}/artifacts", api_user)
+          end
+
+          context 'authorized user' do
+            it_behaves_like 'downloads artifact'
+          end
+
+          context 'unauthorized user' do
             let(:api_user) { nil }
 
-        it 'hides artifacts and rejects request' do
-          expect(project).to be_private
+            it 'does not return specific job artifacts' do
               expect(response).to have_gitlab_http_status(404)
             end
           end
         end
 
         it 'does not return job artifacts if not uploaded' do
+          get api("/projects/#{project.id}/jobs/#{job.id}/artifacts", api_user)
+
           expect(response).to have_gitlab_http_status(404)
         end
       end
+    end
+  end
 
   describe 'GET /projects/:id/artifacts/:ref_name/download?job=name' do
     let(:api_user) { reporter }
-    let(:job) { create(:ci_build, :artifacts, pipeline: pipeline) }
+    let(:job) { create(:ci_build, :artifacts, pipeline: pipeline, user: api_user) }
 
     before do
       job.success
@@ -396,6 +405,7 @@ describe API::Jobs do
 
     context 'find proper job' do
       shared_examples 'a valid file' do
+        context 'when artifacts are stored locally' do
           let(:download_headers) do
             { 'Content-Transfer-Encoding' => 'binary',
               'Content-Disposition' =>
@@ -405,6 +415,7 @@ describe API::Jobs do
           it { expect(response).to have_gitlab_http_status(200) }
           it { expect(response.headers).to include(download_headers) }
         end
+      end
 
       context 'with regular branch' do
         before do

spec/requests/api/v3/builds_spec.rb

@@ -4,16 +4,18 @@ describe API::V3::Builds do
   set(:user) { create(:user) }
   let(:api_user) { user }
   set(:project) { create(:project, :repository, creator: user, public_builds: false) }
-  set(:developer) { create(:project_member, :developer, user: user, project: project) }
-  set(:reporter) { create(:project_member, :reporter, project: project) }
-  set(:guest) { create(:project_member, :guest, project: project) }
-  set(:pipeline) { create(:ci_empty_pipeline, project: project, sha: project.commit.id, ref: project.default_branch) }
-  let!(:build) { create(:ci_build, pipeline: pipeline) }
+  let!(:developer) { create(:project_member, :developer, user: user, project: project) }
+  let(:reporter) { create(:project_member, :reporter, project: project) }
+  let(:guest) { create(:project_member, :guest, project: project) }
+  let(:pipeline) { create(:ci_empty_pipeline, project: project, sha: project.commit.id, ref: project.default_branch) }
+  let(:build) { create(:ci_build, pipeline: pipeline) }
 
   describe 'GET /projects/:id/builds ' do
     let(:query) { '' }
 
     before do |example|
+      build
+
       create(:ci_build, :skipped, pipeline: pipeline)
 
       unless example.metadata[:skip_before_request]
@@ -110,6 +112,10 @@ describe API::V3::Builds do
   end
 
   describe 'GET /projects/:id/repository/commits/:sha/builds' do
+    before do
+      build
+    end
+
     context 'when commit does not exist in repository' do
       before do
         get v3_api("/projects/#{project.id}/repository/commits/1a271fd1/builds", api_user)
@@ -214,6 +220,7 @@ describe API::V3::Builds do
     end
 
     context 'job with artifacts' do
+      context 'when artifacts are stored locally' do
         let(:build) { create(:ci_build, :artifacts, pipeline: pipeline) }
 
         context 'authorized user' do
@@ -228,6 +235,7 @@ describe API::V3::Builds do
             expect(response.body).to match_file(build.artifacts_file.file.file)
           end
         end
+      end
 
       context 'unauthorized user' do
         let(:api_user) { nil }
@@ -303,6 +311,7 @@ describe API::V3::Builds do
 
     context 'find proper job' do
       shared_examples 'a valid file' do
+        context 'when artifacts are stored locally' do
           let(:download_headers) do
             { 'Content-Transfer-Encoding' => 'binary',
               'Content-Disposition' =>
@@ -312,6 +321,7 @@ describe API::V3::Builds do
           it { expect(response).to have_gitlab_http_status(200) }
           it { expect(response.headers).to include(download_headers) }
         end
+      end
 
       context 'with regular branch' do
         before do