diff options
Diffstat (limited to 'spec/features/commits_spec.rb')
-rw-r--r-- | spec/features/commits_spec.rb | 123 |
1 files changed, 62 insertions, 61 deletions
diff --git a/spec/features/commits_spec.rb b/spec/features/commits_spec.rb index 5c6c1c4fd15..5bc56c8325c 100644 --- a/spec/features/commits_spec.rb +++ b/spec/features/commits_spec.rb @@ -1,10 +1,10 @@ -require 'spec_helper' +require "spec_helper" -describe 'Commits' do +describe "Commits" do let(:project) { create(:project, :repository) } let(:user) { create(:user) } - describe 'CI' do + describe "CI" do before do sign_in(user) stub_ci_pipeline_to_return_yaml_file @@ -14,104 +14,104 @@ describe 'Commits' do let!(:pipeline) do create(:ci_pipeline, - project: project, - user: creator, - ref: project.default_branch, - sha: project.commit.sha, - status: :success, - created_at: 5.months.ago) + project: project, + user: creator, + ref: project.default_branch, + sha: project.commit.sha, + status: :success, + created_at: 5.months.ago) end - context 'commit status is Generic Commit Status' do + context "commit status is Generic Commit Status" do let!(:status) { create(:generic_commit_status, pipeline: pipeline) } before do project.add_reporter(user) end - describe 'Commit builds' do + describe "Commit builds" do before do visit pipeline_path(pipeline) end it { expect(page).to have_content pipeline.sha[0..7] } - it 'contains generic commit status build' do - page.within('.table-holder') do + it "contains generic commit status build" do + page.within(".table-holder") do expect(page).to have_content "##{status.id}" # build id - expect(page).to have_content 'generic' # build name + expect(page).to have_content "generic" # build name end end end end - context 'commit status is Ci Build' do + context "commit status is Ci Build" do let!(:build) { create(:ci_build, pipeline: pipeline) } - let(:artifacts_file) { fixture_file_upload('spec/fixtures/banana_sample.gif', 'image/gif') } + let(:artifacts_file) { fixture_file_upload("spec/fixtures/banana_sample.gif", "image/gif") } - context 'when logged as developer' do + context "when logged as developer" do before do project.add_developer(user) end - describe 'Project commits' do + describe "Project commits" do let!(:pipeline_from_other_branch) do create(:ci_pipeline, - project: project, - ref: 'fix', - sha: project.commit.sha, - status: :failed) + project: project, + ref: "fix", + sha: project.commit.sha, + status: :failed) end before do visit project_commits_path(project, :master) end - it 'shows correct build status from default branch' do + it "shows correct build status from default branch" do page.within("//li[@id='commit-#{pipeline.short_sha}']") do - expect(page).to have_css('.ci-status-link') - expect(page).to have_css('.ci-status-icon-success') + expect(page).to have_css(".ci-status-link") + expect(page).to have_css(".ci-status-icon-success") end end end - describe 'Commit builds', :js do + describe "Commit builds", :js do before do visit pipeline_path(pipeline) end - it 'shows pipeline`s data' do + it "shows pipeline`s data" do expect(page).to have_content pipeline.sha[0..7] expect(page).to have_content pipeline.git_commit_message expect(page).to have_content pipeline.user.name end end - context 'Download artifacts' do + context "Download artifacts" do before do build.update(legacy_artifacts_file: artifacts_file) end it do visit pipeline_path(pipeline) - click_on 'Download artifacts' - expect(page.response_headers['Content-Type']).to eq(artifacts_file.content_type) + click_on "Download artifacts" + expect(page.response_headers["Content-Type"]).to eq(artifacts_file.content_type) end end - describe 'Cancel all builds' do - it 'cancels commit', :js do + describe "Cancel all builds" do + it "cancels commit", :js do visit pipeline_path(pipeline) - click_on 'Cancel running' - expect(page).to have_content 'canceled' + click_on "Cancel running" + expect(page).to have_content "canceled" end end - describe 'Cancel build' do - it 'cancels build', :js do + describe "Cancel build" do + it "cancels build", :js do visit pipeline_path(pipeline) - find('.js-btn-cancel-pipeline').click - expect(page).to have_content 'canceled' + find(".js-btn-cancel-pipeline").click + expect(page).to have_content "canceled" end end end @@ -123,24 +123,25 @@ describe 'Commits' do visit pipeline_path(pipeline) end - it 'Renders header', :js do + it "Renders header", :js do expect(page).to have_content pipeline.sha[0..7] expect(page).to have_content pipeline.git_commit_message expect(page).to have_content pipeline.user.name - expect(page).not_to have_link('Cancel running') - expect(page).not_to have_link('Retry') + expect(page).not_to have_link("Cancel running") + expect(page).not_to have_link("Retry") end it do - expect(page).to have_link('Download artifacts') + expect(page).to have_link("Download artifacts") end end - context 'when accessing internal project with disallowed access', :js do + context "when accessing internal project with disallowed access", :js do before do project.update( visibility_level: Gitlab::VisibilityLevel::INTERNAL, - public_builds: false) + public_builds: false + ) build.update(legacy_artifacts_file: artifacts_file) visit pipeline_path(pipeline) end @@ -150,48 +151,48 @@ describe 'Commits' do expect(page).to have_content pipeline.git_commit_message expect(page).to have_content pipeline.user.name - expect(page).not_to have_link('Cancel running') - expect(page).not_to have_link('Retry') + expect(page).not_to have_link("Cancel running") + expect(page).not_to have_link("Retry") end end end - describe '.gitlab-ci.yml not found warning' do + describe ".gitlab-ci.yml not found warning" do before do project.add_reporter(user) end - context 'ci builds enabled' do - it 'does not show warning' do + context "ci builds enabled" do + it "does not show warning" do visit pipeline_path(pipeline) - expect(page).not_to have_content '.gitlab-ci.yml not found in this commit' + expect(page).not_to have_content ".gitlab-ci.yml not found in this commit" end - it 'shows warning' do + it "shows warning" do stub_ci_pipeline_yaml_file(nil) visit pipeline_path(pipeline) - expect(page).to have_content '.gitlab-ci.yml not found in this commit' + expect(page).to have_content ".gitlab-ci.yml not found in this commit" end end - context 'ci builds disabled' do - it 'does not show warning' do + context "ci builds disabled" do + it "does not show warning" do stub_ci_builds_disabled stub_ci_pipeline_yaml_file(nil) visit pipeline_path(pipeline) - expect(page).not_to have_content '.gitlab-ci.yml not found in this commit' + expect(page).not_to have_content ".gitlab-ci.yml not found in this commit" end end end end - context 'viewing commits for a branch' do - let(:branch_name) { 'master' } + context "viewing commits for a branch" do + let(:branch_name) { "master" } before do project.add_maintainer(user) @@ -199,7 +200,7 @@ describe 'Commits' do visit project_commits_path(project, branch_name) end - it 'includes the committed_date for each commit' do + it "includes the committed_date for each commit" do commits = project.repository.commits(branch_name, limit: 40) commits.each do |commit| @@ -207,11 +208,11 @@ describe 'Commits' do end end - it 'shows the ref switcher with the multi-file editor enabled', :js do - set_cookie('new_repo', 'true') + it "shows the ref switcher with the multi-file editor enabled", :js do + set_cookie("new_repo", "true") visit project_commits_path(project, branch_name) - expect(find('.js-project-refs-dropdown')).to have_content branch_name + expect(find(".js-project-refs-dropdown")).to have_content branch_name end end end |