Commit b4e8e7ed authored by Shinya Maeda's avatar Shinya Maeda

Add feartue spec. Fix failed test

parent aa91fdff
...@@ -349,6 +349,18 @@ describe 'Pipelines', :js do ...@@ -349,6 +349,18 @@ describe 'Pipelines', :js do
it { expect(page).not_to have_selector('.build-artifacts') } it { expect(page).not_to have_selector('.build-artifacts') }
end end
context 'with trace artifact' do
before do
create(:ci_build, :success, :trace_artifact, pipeline: pipeline)
visit_project_pipelines
end
it 'does not show trace artifact as artifacts'
expect(page).not_to have_selector('.build-artifacts')
end
end
end end
context 'mini pipeline graph' do context 'mini pipeline graph' do
......
...@@ -66,7 +66,7 @@ describe MigrateOldArtifacts do ...@@ -66,7 +66,7 @@ describe MigrateOldArtifacts do
end end
it 'all files do have artifacts' do it 'all files do have artifacts' do
Ci::Build.with_artifacts do |build| Ci::Build.with_artifacts(:archive) do |build|
expect(build).to have_artifacts expect(build).to have_artifacts
end end
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment