1 files changed, 110 insertions, 0 deletions
diff --git a/spec/features/projects/artifacts/user_browses_artifacts_spec.rb b/spec/features/projects/artifacts/user_browses_artifacts_spec.rb
new file mode 100644
index 00000000000..9ebbbaea911
--- /dev/null
+++ b/spec/features/projects/artifacts/user_browses_artifacts_spec.rb
@@ -0,0 +1,110 @@
+require "spec_helper"
+
+describe "User browses artifacts" do
+  let(:project) { create(:project, :public) }
+  let(:pipeline) { create(:ci_empty_pipeline, project: project) }
+  let(:job) { create(:ci_build, :artifacts, pipeline: pipeline) }
+  let(:browse_url) { browse_project_job_artifacts_path(project, job, "other_artifacts_0.1.2") }
+
+  context "when visiting old URL" do
+    it "redirects to new URL" do
+      visit(browse_url.sub("/-/jobs", "/builds"))
+
+      expect(page.current_path).to eq(browse_url)
+    end
+  end
+
+  context "when browsing artifacts root directory" do
+    before do
+      visit(browse_project_job_artifacts_path(project, job))
+    end
+
+    it "shows artifacts" do
+      expect(page).not_to have_selector(".build-sidebar")
+
+      page.within(".tree-table") do
+        expect(page).to have_no_content("..")
+                   .and have_content("other_artifacts_0.1.2")
+                   .and have_content("ci_artifacts.txt")
+                   .and have_content("rails_sample.jpg")
+      end
+
+      page.within(".build-header") do
+        expect(page).to have_content("Job ##{job.id} in pipeline ##{pipeline.id} for #{pipeline.short_sha}")
+      end
+    end
+
+    it "shows an artifact" do
+      click_link("ci_artifacts.txt")
+
+      expect(page).to have_link("download it")
+    end
+  end
+
+  context "when browsing a directory with UTF-8 characters in its name" do
+    before do
+      visit(browse_project_job_artifacts_path(project, job))
+    end
+
+    it "shows correct content", :js do
+      page.within(".tree-table") do
+        click_link("tests_encoding")
+
+        expect(page).to have_no_content("non-utf8-dir")
+
+        click_link("utf8 test dir ✓")
+
+        expect(page).to have_content("..").and have_content("regular_file_2")
+      end
+    end
+  end
+
+  context "when browsing a directory with a text file" do
+    let(:txt_entry) { job.artifacts_metadata_entry("other_artifacts_0.1.2/doc_sample.txt") }
+
+    before do
+      allow(Gitlab.config.pages).to receive(:enabled).and_return(true)
+      allow(Gitlab.config.pages).to receive(:artifacts_server).and_return(true)
+    end
+
+    context "when the project is public" do
+      before do
+        visit(browse_url)
+      end
+
+      it "shows correct content" do
+        link = first(".tree-item-file-external-link")
+
+        expect(link[:target]).to eq("_blank")
+        expect(link[:rel]).to include("noopener").and include("noreferrer")
+        expect(page).to have_link("doc_sample.txt", href: file_project_job_artifacts_path(project, job, path: txt_entry.blob.path))
+                   .and have_selector(".js-artifact-tree-external-icon")
+
+        page.within(".tree-table") do
+          expect(page).to have_content("..").and have_content("another-subdirectory")
+        end
+
+        page.within(".repo-breadcrumb") do
+          expect(page).to have_content("other_artifacts_0.1.2")
+        end
+      end
+    end
+
+    context "when the project is private" do
+      let!(:private_project) { create(:project, :private) }
+      let(:pipeline) { create(:ci_empty_pipeline, project: private_project) }
+      let(:job) { create(:ci_build, :artifacts, pipeline: pipeline) }
+      let(:user) { create(:user) }
+
+      before do
+        private_project.add_developer(user)
+
+        sign_in(user)
+
+        visit(browse_project_job_artifacts_path(private_project, job, "other_artifacts_0.1.2"))
+      end
+
+      it { expect(page).to have_link("doc_sample.txt").and have_no_selector(".js-artifact-tree-external-icon") }
+    end
+  end
+end