/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to breezy/plugins/gitlab/hoster.py

  • Committer: Breezy landing bot
  • Author(s): Jelmer Vernooij
  • Date: 2020-06-12 01:45:56 UTC
  • mfrom: (7513.1.2 pypy3)
  • Revision ID: breezy.the.bot@gmail.com-20200612014556-tsc8assk3d0luziu
Avoid deprecated behaviour in ElementTree.

Merged from https://code.launchpad.net/~jelmer/brz/pypy3/+merge/385611

Show diffs side-by-side

added added

removed removed

Lines of Context:
breezy/plugins/gitlab/hoster.py
16
16
 
17
17
"""Support for GitLab."""
18
18
 
19
 
from __future__ import absolute_import
20
 
 
21
19
import json
22
20
import os
23
21
import time
30
28
    urlutils,
31
29
    )
32
30
from ...git.urls import git_url_to_bzr_url
33
 
from ...sixish import PY3
34
31
from ...trace import mutter
35
32
from ...transport import get_transport
36
33
 
169
166
        raise NotGitLabUrl(url)
170
167
    path = path.strip('/')
171
168
    parts = path.split('/')
 
169
    if len(parts) < 2:
 
170
        raise NotMergeRequestUrl(host, url)
172
171
    if parts[-2] != 'merge_requests':
173
172
        raise NotMergeRequestUrl(host, url)
174
 
    return host, '/'.join(parts[:-2]), int(parts[-1])
 
173
    if parts[-3] == '-':
 
174
        project_name = '/'.join(parts[:-3])
 
175
    else:
 
176
        project_name = '/'.join(parts[:-2])
 
177
    return host, project_name, int(parts[-1])
175
178
 
176
179
 
177
180
class GitLabMergeProposal(MergeProposal):
247
250
            return False
248
251
        elif self._mr['merge_status'] == 'can_be_merged':
249
252
            return True
 
253
        elif self._mr['merge_status'] in (
 
254
                'unchecked', 'cannot_be_merged_recheck'):
 
255
            # See https://gitlab.com/gitlab-org/gitlab/-/commit/7517105303c for
 
256
            # an explanation of the distinction between unchecked and
 
257
            # cannot_be_merged_recheck
 
258
            return None
250
259
        else:
251
260
            raise ValueError(self._mr['merge_status'])
252
261
 
265
274
 
266
275
 
267
276
def gitlab_url_to_bzr_url(url, name):
268
 
    if not PY3:
269
 
        name = name.encode('utf-8')
270
277
    return git_url_to_bzr_url(url, branch=name)
271
278
 
272
279
 
285
292
    def base_url(self):
286
293
        return self.transport.base
287
294
 
 
295
    @property
 
296
    def base_hostname(self):
 
297
        return urlutils.parse_url(self.base_url)[3]
 
298
 
288
299
    def _api_request(self, method, path, fields=None, body=None):
289
300
        return self.transport.request(
290
301
            method, urlutils.join(self.base_url, 'api', 'v4', path),
379
390
            parameters['owner_id'] = urlutils.quote(owner, '')
380
391
        return self._list_paged(path, parameters, per_page=DEFAULT_PAGE_SIZE)
381
392
 
 
393
    def _get_merge_request(self, project, merge_id):
 
394
        path = 'projects/%s/merge_requests/%d' % (urlutils.quote(str(project), ''), merge_id)
 
395
        response = self._api_request('GET', path)
 
396
        if response.status == 403:
 
397
            raise errors.PermissionDenied(response.text)
 
398
        if response.status != 200:
 
399
            raise errors.InvalidHttpResponse(path, response.text)
 
400
        return json.loads(response.data)
 
401
 
382
402
    def _list_projects(self, owner):
383
403
        path = 'users/%s/projects' % urlutils.quote(str(owner), '')
384
404
        parameters = {}
490
510
            (host, project, branch_name) = parse_gitlab_branch_url(branch)
491
511
        except NotGitLabUrl:
492
512
            return False
493
 
        return (self.base_url == ('https://%s' % host))
 
513
        return self.base_hostname == host
494
514
 
495
515
    def check(self):
496
516
        response = self._api_request('GET', 'user')
545
565
        except NotGitLabUrl:
546
566
            raise UnsupportedHoster(url)
547
567
        except NotMergeRequestUrl as e:
548
 
            if self.base_url == ('https://%s' % e.host):
 
568
            if self.base_hostname == e.host:
549
569
                raise
550
570
            else:
551
571
                raise UnsupportedHoster(url)
552
 
        if self.base_url != ('https://%s' % host):
 
572
        if self.base_hostname != host:
553
573
            raise UnsupportedHoster(url)
554
574
        project = self._get_project(project)
555
 
        mr = project.mergerequests.get(merge_id)
556
 
        return GitLabMergeProposal(mr)
 
575
        mr = self._get_merge_request(project['path_with_namespace'], merge_id)
 
576
        return GitLabMergeProposal(self, mr)
557
577
 
558
578
    def delete_project(self, project):
559
579
        path = 'projects/%s' % urlutils.quote(str(project), '')
629
649
        try:
630
650
            merge_request = self.gl._create_mergerequest(**kwargs)
631
651
        except MergeRequestExists:
632
 
            raise ProposalExists(self.source_branch.user_url)
 
652
            raise MergeProposalExists(self.source_branch.user_url)
633
653
        return GitLabMergeProposal(self.gl, merge_request)
634
654
 
635
655