Welcome to mirror list, hosted at ThFree Co, Russian Federation.

gitlab.com/gitlab-org/gitlab-foss.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorShinya Maeda <shinya@gitlab.com>2018-08-03 14:08:13 +0300
committerShinya Maeda <shinya@gitlab.com>2018-08-03 14:08:13 +0300
commit41f28a9ffabf4eb45c53836ea4de3b7a49229eaa (patch)
tree174d41cf6bb28ccc3c88727618886a431364a63c /spec/factories
parent06b8f47cf3b8ce65012fe905f6d3953ff175fa85 (diff)
Add factory for parsers. Add required specification in json schema matcher. Improved test code.
Diffstat (limited to 'spec/factories')
-rw-r--r--spec/factories/ci/builds.rb5
-rw-r--r--spec/factories/ci/pipelines.rb4
-rw-r--r--spec/factories/merge_requests.rb9
3 files changed, 8 insertions, 10 deletions
diff --git a/spec/factories/ci/builds.rb b/spec/factories/ci/builds.rb
index 8bd1f1ae4e0..9813190925b 100644
--- a/spec/factories/ci/builds.rb
+++ b/spec/factories/ci/builds.rb
@@ -188,9 +188,8 @@ FactoryBot.define do
end
trait :test_reports do
- after(:create) do |build|
- create(:ci_job_artifact, :junit, job: build)
- build.reload
+ after(:build) do |build|
+ build.job_artifacts << create(:ci_job_artifact, :junit, job: build)
end
end
diff --git a/spec/factories/ci/pipelines.rb b/spec/factories/ci/pipelines.rb
index 457fcbad620..a6ff226fa75 100644
--- a/spec/factories/ci/pipelines.rb
+++ b/spec/factories/ci/pipelines.rb
@@ -70,11 +70,11 @@ FactoryBot.define do
protected true
end
- trait :test_reports do
+ trait :with_test_reports do
status :success
after(:build) do |pipeline, evaluator|
- create(:ci_build, :test_reports, pipeline: pipeline, project: pipeline.project)
+ pipeline.builds << build(:ci_build, :test_reports, pipeline: pipeline, project: pipeline.project)
end
end
end
diff --git a/spec/factories/merge_requests.rb b/spec/factories/merge_requests.rb
index 4c36ec8ec58..3268b607a76 100644
--- a/spec/factories/merge_requests.rb
+++ b/spec/factories/merge_requests.rb
@@ -90,15 +90,14 @@ FactoryBot.define do
end
trait :with_test_reports do
- after(:create) do |merge_request|
- create(:ci_pipeline,
+ after(:build) do |merge_request|
+ merge_request.head_pipeline = build(
+ :ci_pipeline,
:success,
:test_reports,
project: merge_request.source_project,
ref: merge_request.source_branch,
- sha: merge_request.diff_head_sha).tap do |pipeline|
- merge_request.update!(head_pipeline_id: pipeline.id)
- end
+ sha: merge_request.diff_head_sha)
end
end