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:
authorGrzegorz Bizon <grzesiek.bizon@gmail.com>2017-07-20 10:18:45 +0300
committerGrzegorz Bizon <grzesiek.bizon@gmail.com>2017-07-20 10:20:25 +0300
commitc9749e22383661c0772addfcf4274ec3a81bd229 (patch)
tree7786dc798526f1d807706ea9ce13ef74ddf44fd5 /spec/serializers
parent0275914919551de1ffd5819bd9da7bf05d6a7668 (diff)
Improve build details serializable entity specs
Diffstat (limited to 'spec/serializers')
-rw-r--r--spec/serializers/build_details_entity_spec.rb10
1 files changed, 5 insertions, 5 deletions
diff --git a/spec/serializers/build_details_entity_spec.rb b/spec/serializers/build_details_entity_spec.rb
index e688035cecc..2c981154f0d 100644
--- a/spec/serializers/build_details_entity_spec.rb
+++ b/spec/serializers/build_details_entity_spec.rb
@@ -46,8 +46,8 @@ describe BuildDetailsEntity do
expect(subject).to include(:new_issue_path)
end
- it 'exposes details of the merge request' do
- expect(subject[:merge_request]).to include(:iid, :path)
+ it 'exposes correct details of the merge request' do
+ expect(subject[:merge_request][:iid]).to eq merge_request.iid
end
it 'has a correct merge request path' do
@@ -78,7 +78,7 @@ describe BuildDetailsEntity do
end
it 'exposes details of the merge request' do
- expect(subject[:merge_request]).to include(:iid, :path)
+ expect(subject[:merge_request][:iid]).to eq merge_request.iid
end
it 'has a correct merge request path' do
@@ -88,7 +88,7 @@ describe BuildDetailsEntity do
end
context 'when the build has been erased' do
- let(:build) { create(:ci_build, :erasable, project: project) }
+ let(:build) { create(:ci_build, :erased, project: project) }
it 'exposes the user whom erased the build' do
expect(subject).to include(:erase_path)
@@ -96,7 +96,7 @@ describe BuildDetailsEntity do
end
context 'when the build has been erased' do
- let(:build) { create(:ci_build, erased_at: Time.now, project: project, erased_by: user) }
+ let(:build) { create(:ci_build, :erased, project: project, erased_by: user) }
it 'exposes the user whom erased the build' do
expect(subject).to include(:erased_by)