diff options
author | Shinya Maeda <shinya@gitlab.com> | 2018-03-14 00:52:34 +0900 |
---|---|---|
committer | Shinya Maeda <shinya@gitlab.com> | 2018-03-15 18:04:17 +0900 |
commit | b4e8e7ed3aab3a55a6a77b8c6473ad5fafb75ae6 (patch) | |
tree | a682038932d1f7d6c18bc4c5e7b60052c98a03aa | |
parent | aa91fdff715cbf8d953754fd07fbded0c71e8dab (diff) | |
download | gitlab-ce-b4e8e7ed3aab3a55a6a77b8c6473ad5fafb75ae6.tar.gz |
Add feartue spec. Fix failed test
-rw-r--r-- | spec/features/projects/pipelines/pipelines_spec.rb | 12 | ||||
-rw-r--r-- | spec/migrations/migrate_old_artifacts_spec.rb | 2 |
2 files changed, 13 insertions, 1 deletions
diff --git a/spec/features/projects/pipelines/pipelines_spec.rb b/spec/features/projects/pipelines/pipelines_spec.rb index 33ad59abfdf..99135bfe89d 100644 --- a/spec/features/projects/pipelines/pipelines_spec.rb +++ b/spec/features/projects/pipelines/pipelines_spec.rb @@ -349,6 +349,18 @@ describe 'Pipelines', :js do it { expect(page).not_to have_selector('.build-artifacts') } 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 context 'mini pipeline graph' do diff --git a/spec/migrations/migrate_old_artifacts_spec.rb b/spec/migrations/migrate_old_artifacts_spec.rb index 92eb1d9ce86..3a6fab6572c 100644 --- a/spec/migrations/migrate_old_artifacts_spec.rb +++ b/spec/migrations/migrate_old_artifacts_spec.rb @@ -66,7 +66,7 @@ describe MigrateOldArtifacts do end 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 end end |