Merge branch 'yisiliang/gitlab-ce-avoid-nil-safe_message' into 'master'

Avoid nil safe message

See merge request gitlab-org/gitlab-ce!21326

app/models/commit.rb

@@ -193,6 +193,7 @@ class Commit
   # otherwise returns commit message without first line
   def description
     return safe_message if full_title.length >= 100
+    return no_commit_message if safe_message.blank?
     safe_message.split("\n", 2)[1].try(:chomp)
   end

changelogs/unreleased/21326-avoid-nil-safe-message.yml

+---
+title: "Avoid nil safe message"
+merge_request: 21326
+author: Yi Siliang
+type: fixed

spec/models/commit_spec.rb

@@ -225,6 +225,12 @@ eos
   end
   describe 'description' do
+    it 'returns no_commit_message when safe_message is blank' do
+      allow(commit).to receive(:safe_message).and_return(nil)
+
+      expect(commit.description).to eq('--no commit message')
+    end
+
     it 'returns description of commit message if title less than 100 characters' do
       message = <<eos
 Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec sodales id felis id blandit.