diff options
author | Francisco Javier López <fjlopez@gitlab.com> | 2019-04-01 15:13:22 +0000 |
---|---|---|
committer | Nick Thomas <nick@gitlab.com> | 2019-04-01 15:13:22 +0000 |
commit | 45293f66facc2792f15bd06bb5b05d55fc155cba (patch) | |
tree | 62c61b7d0db4e23c6d21c2c18a9585f73a4df993 | |
parent | 0a480e3c32c4fd783b6a6271dad7c423e0d8bdd1 (diff) | |
download | gitlab-ce-45293f66facc2792f15bd06bb5b05d55fc155cba.tar.gz |
Fix MergeRequest#has_commits? nil comparison
-rw-r--r-- | app/models/merge_request.rb | 2 | ||||
-rw-r--r-- | spec/models/merge_request_spec.rb | 13 |
2 files changed, 11 insertions, 4 deletions
diff --git a/app/models/merge_request.rb b/app/models/merge_request.rb index b52c480a487..d3951cd72db 100644 --- a/app/models/merge_request.rb +++ b/app/models/merge_request.rb @@ -1343,7 +1343,7 @@ class MergeRequest < ApplicationRecord end def has_commits? - merge_request_diff && commits_count > 0 + merge_request_diff && commits_count.to_i > 0 end def has_no_commits? diff --git a/spec/models/merge_request_spec.rb b/spec/models/merge_request_spec.rb index 092504cde9e..4b3a77e6bc2 100644 --- a/spec/models/merge_request_spec.rb +++ b/spec/models/merge_request_spec.rb @@ -2714,14 +2714,21 @@ describe MergeRequest do end describe '#has_commits?' do - before do + it 'returns true when merge request diff has commits' do allow(subject.merge_request_diff).to receive(:commits_count) .and_return(2) - end - it 'returns true when merge request diff has commits' do expect(subject.has_commits?).to be_truthy end + + context 'when commits_count is nil' do + it 'returns false' do + allow(subject.merge_request_diff).to receive(:commits_count) + .and_return(nil) + + expect(subject.has_commits?).to be_falsey + end + end end describe '#has_no_commits?' do |