summaryrefslogtreecommitdiff
path: root/spec/features/commits_spec.rb
diff options
context:
space:
mode:
Diffstat (limited to 'spec/features/commits_spec.rb')
-rw-r--r--spec/features/commits_spec.rb123
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