fix: return response with commit data

nejch · nejch · commit b77b945c7e00 · 2020-02-19T00:30:48.000+01:00
diff --git a/gitlab/v4/objects.py b/gitlab/v4/objects.py
@@ -2148,10 +2148,13 @@ def revert(self, branch, **kwargs):
         Raises:
             GitlabAuthenticationError: If authentication is not correct
             GitlabRevertError: If the revert could not be performed
+
+        Returns:
+            dict: The new commit data (*not* a RESTObject)
         """
         path = "%s/%s/revert" % (self.manager.path, self.get_id())
         post_data = {"branch": branch}
-        self.manager.gitlab.http_post(path, post_data=post_data, **kwargs)
+        return self.manager.gitlab.http_post(path, post_data=post_data, **kwargs)
 
 
 class ProjectCommitManager(RetrieveMixin, CreateMixin, RESTManager):
diff --git a/tools/python_test_v4.py b/tools/python_test_v4.py
@@ -463,13 +463,12 @@
 # assert len(discussion.attributes["notes"]) == 1
 
 # Revert commit
-commit.revert(branch="master")
-revert_commit = admin_project.commits.list()[0]
+revert_commit = commit.revert(branch="master")
 
 expected_message = 'Revert "{}"\n\nThis reverts commit {}'.format(
     commit.message, commit.id
 )
-assert revert_commit.message == expected_message
+assert revert_commit["message"] == expected_message
 
 try:
     commit.revert(branch="master")
