Fix push events with multiline commit message

Author: NotSqrt

mattermost_gitlab/event_formatter.py

@@ -72,11 +72,11 @@ def format(self):
         if self.data['total_commits_count'] > 1:
             description += "s"
 
-        suffix = ''
+        suffix = '.'
         if len(self.data['commits']) > 0:
-            suffix = '\n'
+            suffix = ':\n'
 
-        text = '%s pushed %s into the `%s` branch for project [%s](%s).%s' % (
+        text = '%s pushed %s into the `%s` branch for project [%s](%s)%s' % (
             self.data['user_name'],
             description,
             self.data['ref'],
@@ -85,10 +85,12 @@ def format(self):
             suffix
         )
         for val in self.data['commits']:
-            text += "[%s](%s)" % (val['message'], val['url'])
+            header = val['message'].splitlines()[0]
+            text += "* [%s](%s)\n" % (header, val['url'])
 
         return text
 
+
 class IssueEvent(BaseEvent):
 
     @property

tests/data/gitlab/push/commit_dev_branch.md

@@ -1,2 +1,2 @@
-Example User pushed 1 commit into the `refs/heads/dev` branch for project [example repository](http://gitlab.example.com/root/example-repository).
-[bump2 with unicode ⇗ ⟾](http://gitlab.example.com/root/example-repository/commit/c5ccce8cd46a277dd0723ad5954f44a562c5c67e)
+Example User pushed 1 commit into the `refs/heads/dev` branch for project [example repository](http://gitlab.example.com/root/example-repository):
+* [bump2 with unicode ⇗ ⟾](http://gitlab.example.com/root/example-repository/commit/c5ccce8cd46a277dd0723ad5954f44a562c5c67e)

tests/data/gitlab/push/commit_master_branch.md

@@ -1,2 +1,2 @@
-Example User pushed 1 commit into the `refs/heads/master` branch for project [example repository](http://gitlab.example.com/root/example-repository).
-[bump](http://gitlab.example.com/root/example-repository/commit/2e03a84ed69727606a8d662b60125c6c48f6484f)
+Example User pushed 1 commit into the `refs/heads/master` branch for project [example repository](http://gitlab.example.com/root/example-repository):
+* [bump](http://gitlab.example.com/root/example-repository/commit/2e03a84ed69727606a8d662b60125c6c48f6484f)

tests/data/gitlab/push/commit_merge_request.md

@@ -1,8 +1,3 @@
-Example User pushed 2 commits into the `refs/heads/master` branch for project [example repository](http://gitlab.example.com/root/example-repository).
-[bump2 with unicode ⇗ ⟾](http://gitlab.example.com/root/example-repository/commit/c5ccce8cd46a277dd0723ad5954f44a562c5c67e)[Merge branch 'dev' into 'master'
-
-merge cool feature
-
-New description
-
-See merge request !1](http://gitlab.example.com/root/example-repository/commit/d2b94519881dc201da8151d6344eab0055f15a44)
+Example User pushed 2 commits into the `refs/heads/master` branch for project [example repository](http://gitlab.example.com/root/example-repository):
+* [bump2 with unicode ⇗ ⟾](http://gitlab.example.com/root/example-repository/commit/c5ccce8cd46a277dd0723ad5954f44a562c5c67e)
+* [Merge branch 'dev' into 'master'](http://gitlab.example.com/root/example-repository/commit/d2b94519881dc201da8151d6344eab0055f15a44)

tests/data/gitlab/push/first_commit.md

@@ -1,2 +1,2 @@
-Example User pushed the first commit into the `refs/heads/master` branch for project [example repository](http://gitlab.example.com/root/example-repository).
-[Create project](http://gitlab.example.com/root/example-repository/commit/33477153c450b1ce27025406b9e1e3c6de16c3b9)
+Example User pushed the first commit into the `refs/heads/master` branch for project [example repository](http://gitlab.example.com/root/example-repository):
+* [Create project](http://gitlab.example.com/root/example-repository/commit/33477153c450b1ce27025406b9e1e3c6de16c3b9)