From 08f19b3d79dd50bab5afe58fe1b3b3825ddf9c25 Mon Sep 17 00:00:00 2001 From: Gauvain Pocentek Date: Sat, 20 Jan 2018 13:43:23 +0100 Subject: [PATCH] Add support for MR participants API Fixes #387 --- gitlab/v4/objects.py | 24 ++++++++++++++++++++++++ tools/python_test_v4.py | 6 ++++++ 2 files changed, 30 insertions(+) diff --git a/gitlab/v4/objects.py b/gitlab/v4/objects.py index e4a544730..211527da1 100644 --- a/gitlab/v4/objects.py +++ b/gitlab/v4/objects.py @@ -1557,6 +1557,30 @@ def merge(self, merge_commit_message=None, **kwargs) self._update_attrs(server_data) + @cli.register_custom_action('ProjectMergeRequest') + @exc.on_http_error(exc.GitlabListError) + def participants(self, **kwargs): + """List the merge request participants. + + Args: + all (bool): If True, return all the items, without pagination + per_page (int): Number of items to retrieve per request + page (int): ID of the page to return (starts with page 1) + as_list (bool): If set to False and no pagination option is + defined, return a generator instead of a list + **kwargs: Extra options to send to the server (e.g. sudo) + + Raises: + GitlabAuthenticationError: If authentication is not correct + GitlabListError: If the list could not be retrieved + + Returns: + RESTObjectList: The list of participants + """ + + path = '%s/%s/participants' % (self.manager.path, self.get_id()) + return self.manager.gitlab.http_get(path, **kwargs) + class ProjectMergeRequestManager(CRUDMixin, RESTManager): _path = '/projects/%(project_id)s/merge_requests' diff --git a/tools/python_test_v4.py b/tools/python_test_v4.py index e06502018..69596b8f1 100644 --- a/tools/python_test_v4.py +++ b/tools/python_test_v4.py @@ -527,6 +527,12 @@ mr = admin_project.mergerequests.create({'source_branch': 'branch1', 'target_branch': 'master', 'title': 'MR readme2'}) + +# basic testing: only make sure that the methods exist +mr.commits() +mr.changes() +#mr.participants() # not yet available + mr.merge() admin_project.branches.delete('branch1')