diff options
| author | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2019-05-06 10:08:42 +0200 |
|---|---|---|
| committer | Filipa Lacerda <filipa@gitlab.com> | 2019-05-24 10:04:04 +0100 |
| commit | b9c19f6b0f5c85956bf85935452c34fc6a21bcb0 (patch) | |
| tree | a847bd3653357f6ebe5aa80ed416c1f0e7fe3e37 /spec/serializers | |
| parent | 819746dc6544820f41d66b87c2c68efc9b17e0f4 (diff) | |
| download | gitlab-ce-b9c19f6b0f5c85956bf85935452c34fc6a21bcb0.tar.gz | |
Do not serialize a pipeline again when showing a build
This change makes it possible to avoid additional serialization of
entire pipeline when rendering a build details page. Instead we expose
more information from the build entity itself what is much cheaper than
serializing entire pipeline.
Additionally we do not need to serialize `latest?` flag, which involves
multiple Gitaly calls.
Diffstat (limited to 'spec/serializers')
| -rw-r--r-- | spec/serializers/pipeline_entity_spec.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/spec/serializers/pipeline_entity_spec.rb b/spec/serializers/pipeline_entity_spec.rb index 47f767ae4ab..93d18f70cdc 100644 --- a/spec/serializers/pipeline_entity_spec.rb +++ b/spec/serializers/pipeline_entity_spec.rb @@ -48,8 +48,8 @@ describe PipelineEntity do it 'contains flags' do expect(subject).to include :flags expect(subject[:flags]) - .to include :latest, :stuck, :auto_devops, - :yaml_errors, :retryable, :cancelable, :merge_request + .to include :stuck, :auto_devops, :yaml_errors, + :retryable, :cancelable, :merge_request end end |