Commit e0496ae5 authored by Furkan Ayhan's avatar Furkan Ayhan

Merge branch '347495-follow-up-test-coverage' into 'master'

Add tests on Ci::Pipeline entity

See merge request gitlab-org/gitlab!77552
parents 5f8ed6b1 9a898d05
...@@ -3,14 +3,31 @@ ...@@ -3,14 +3,31 @@
require 'spec_helper' require 'spec_helper'
RSpec.describe API::Entities::Ci::Pipeline do RSpec.describe API::Entities::Ci::Pipeline do
let_it_be(:pipeline) { create(:ci_empty_pipeline) } let_it_be(:user) { create(:user) }
let_it_be(:pipeline) { create(:ci_empty_pipeline, user: user) }
let_it_be(:job) { create(:ci_build, name: "rspec", coverage: 30.212, pipeline: pipeline) } let_it_be(:job) { create(:ci_build, name: "rspec", coverage: 30.212, pipeline: pipeline) }
let(:entity) { described_class.new(pipeline) } let(:entity) { described_class.new(pipeline) }
subject { entity.as_json } subject { entity.as_json }
it 'returns the coverage as a string' do exposed_fields = %i[before_sha tag yaml_errors created_at updated_at started_at finished_at committed_at duration queued_duration]
exposed_fields.each do |field|
it "exposes pipeline #{field}" do
expect(subject[field]).to eq(pipeline.public_send(field))
end
end
it 'exposes pipeline user basic information' do
expect(subject[:user].keys).to include(:avatar_url, :web_url)
end
it 'exposes pipeline detailed status' do
expect(subject[:detailed_status].keys).to include(:icon, :favicon)
end
it 'exposes pipeline coverage as a string' do
expect(subject[:coverage]).to eq '30.21' expect(subject[:coverage]).to eq '30.21'
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