GitHubApi.auth_token() - Code Metrics - Inspection of "Refactor and polish" - michaeljoseph/changes - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#127)

by Michael

created 2017-11-06 22:14 UTC

GitHubApi.auth_token() A

↳ Parent: GitHubApi

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes

Metric	Value
cc	1
c	0
b	0
f	0
dl	0
loc	3
ccs	2
cts	2
cp	1
crap	1
rs	10

import attr
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
import requests
import uritemplate
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
from pathlib import Path


EXT_TO_MIME_TYPE = {
    '.gz': 'application/x-gzip',
    '.whl': 'application/zip',
    '.zip': 'application/zip',
}


@attr.s
class GitHubApi(object):
    ISSUE_ENDPOINT = (
        'https://api.github.com/repos{/owner}{/repo}/issues{/number}'
    )
    LABELS_ENDPOINT = (
        'https://api.github.com/repos{/owner}{/repo}/labels'
    )
    RELEASES_ENDPOINT = (
        'https://api.github.com/repos{/owner}{/repo}/releases'
    )

    repository = attr.ib()

    @property
    def owner(self):
        return self.repository.owner

    @property
    def repo(self):
        return self.repository.repo

    @property
    def auth_token(self):
        return self.repository.auth_token

    def pull_request(self, pr_num):
        pull_request_api_url = uritemplate.expand(
            self.ISSUE_ENDPOINT,
            dict(
                owner=self.owner,
                repo=self.repo,
                number=pr_num
            ),
        )

        return requests.get(
            pull_request_api_url,
            headers={
                'Authorization': 'token {}'.format(self.auth_token)
            },
        ).json()

    def labels(self):
        labels_api_url = uritemplate.expand(
            self.LABELS_ENDPOINT,
            dict(
                owner=self.owner,
                repo=self.repo,
            ),
        )

        return requests.get(
            labels_api_url,
            headers={
                'Authorization': 'token {}'.format(self.auth_token)
            },
        ).json()

    def create_release(self, release, uploads=None):
        params = {
            'tag_name': release.version,
            'name': release.name,
            'body': release.description,
            # 'prerelease': True,
        }

        releases_api_url = uritemplate.expand(
            self.RELEASES_ENDPOINT,
            dict(
                owner=self.owner,
                repo=self.repo,
            )
        )

        return requests.post(
            releases_api_url,
            headers={
                'Authorization': 'token {}'.format(self.auth_token)
            },
            # auth=(self.auth_token, 'x-oauth-basic'),
            json=params,
        ).json()

        upload_responses = []
        upload_url = response['upload_url']

        for upload in uploads:
            upload = Path(upload)
            upload_responses.append(requests.post(
                uritemplate.expand(
                    upload_url,
                    dict(name=upload.name)
                ),
                # auth=(gh_token, 'x-oauth-basic'),
                headers={
                    'authorization': 'token {}'.format(self.auth_token),
                    'content-type': EXT_TO_MIME_TYPE[distribution.ext],

                },
                data=upload.read_bytes(),

                verify=False,
            ))

        return response, upload_responses



1	3	import attr
		0 ignored issues – show Configuration introduced 2017-10-31 07:14 UTC by Report Bug Copy Issue Report The import `attr` could not be resolved. This can be caused by one of the following: 1. Missing Dependencies This error could indicate a configuration issue of Pylint. Make sure that your libraries are available by adding the necessary commands. # .scrutinizer.yml before_commands: - sudo pip install abc # Python2 - sudo pip3 install abc # Python3 Tip: We are currently not using virtualenv to run pylint, when installing your modules make sure to use the command for the correct version. 2. Missing __init__.py files This error could also result from missing `__init__.py` files in your module folders. Make sure that you place one file in each sub-folder. Loading history...
2	3	import requests
3	3	import uritemplate
		0 ignored issues – show Configuration introduced 2017-10-31 07:14 UTC by Report Bug Copy Issue Report The import `uritemplate` could not be resolved. This can be caused by one of the following: 1. Missing Dependencies This error could indicate a configuration issue of Pylint. Make sure that your libraries are available by adding the necessary commands. # .scrutinizer.yml before_commands: - sudo pip install abc # Python2 - sudo pip3 install abc # Python3 Tip: We are currently not using virtualenv to run pylint, when installing your modules make sure to use the command for the correct version. 2. Missing __init__.py files This error could also result from missing `__init__.py` files in your module folders. Make sure that you place one file in each sub-folder. Loading history...
4	3	from pathlib import Path
5
6
7	3	EXT_TO_MIME_TYPE = {
8		'.gz': 'application/x-gzip',
9		'.whl': 'application/zip',
10		'.zip': 'application/zip',
11		}
12
13
14	3	@attr.s
15	3	class GitHubApi(object):
16	3	ISSUE_ENDPOINT = (
17		'https://api.github.com/repos{/owner}{/repo}/issues{/number}'
18		)
19	3	LABELS_ENDPOINT = (
20		'https://api.github.com/repos{/owner}{/repo}/labels'
21		)
22	3	RELEASES_ENDPOINT = (
23		'https://api.github.com/repos{/owner}{/repo}/releases'
24		)
25
26	3	repository = attr.ib()
27
28	3	@property
29		def owner(self):
30	3	return self.repository.owner
31
32	3	@property
33		def repo(self):
34	3	return self.repository.repo
35
36	3	@property
37		def auth_token(self):
38	3	return self.repository.auth_token
39
40	3	def pull_request(self, pr_num):
41	3	pull_request_api_url = uritemplate.expand(
42		self.ISSUE_ENDPOINT,
43		dict(
44		owner=self.owner,
45		repo=self.repo,
46		number=pr_num
47		),
48		)
49
50	3	return requests.get(
51		pull_request_api_url,
52		headers={
53		'Authorization': 'token {}'.format(self.auth_token)
54		},
55		).json()
56
57	3	def labels(self):
58	3	labels_api_url = uritemplate.expand(
59		self.LABELS_ENDPOINT,
60		dict(
61		owner=self.owner,
62		repo=self.repo,
63		),
64		)
65
66	3	return requests.get(
67		labels_api_url,
68		headers={
69		'Authorization': 'token {}'.format(self.auth_token)
70		},
71		).json()
72
73	3	def create_release(self, release, uploads=None):
74	3	params = {
75		'tag_name': release.version,
76		'name': release.name,
77		'body': release.description,
78		# 'prerelease': True,
79		}
80
81	3	releases_api_url = uritemplate.expand(
82		self.RELEASES_ENDPOINT,
83		dict(
84		owner=self.owner,
85		repo=self.repo,
86		)
87		)
88
89	3	return requests.post(
90		releases_api_url,
91		headers={
92		'Authorization': 'token {}'.format(self.auth_token)
93		},
94		# auth=(self.auth_token, 'x-oauth-basic'),
95		json=params,
96		).json()
97
98		upload_responses = []
99		upload_url = response['upload_url']
		0 ignored issues – show Comprehensibility Best Practice introduced 2017-10-31 07:14 UTC by Report Bug Copy Issue Report Undefined variable 'response' Loading history...
100		for upload in uploads:
101		upload = Path(upload)
102		upload_responses.append(requests.post(
103		uritemplate.expand(
104		upload_url,
105		dict(name=upload.name)
106		),
107		# auth=(gh_token, 'x-oauth-basic'),
108		headers={
109		'authorization': 'token {}'.format(self.auth_token),
110		'content-type': EXT_TO_MIME_TYPE[distribution.ext],
		0 ignored issues – show Comprehensibility Best Practice introduced 2017-10-31 07:14 UTC by Report Bug Copy Issue Report Undefined variable 'distribution' Loading history...
111		},
112		data=upload.read_bytes(),
		0 ignored issues – show Bug introduced 2017-10-31 07:14 UTC by Report Bug Copy Issue Report The Instance of `Path` does not seem to have a member named `read_bytes`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
113		verify=False,
114		))
115
116		return response, upload_responses
		0 ignored issues – show Comprehensibility Best Practice introduced 2017-10-31 07:14 UTC by Report Bug Copy Issue Report Undefined variable 'response' Loading history...
117

Pull Request — master (#127)

GitHubApi.auth_token() A

Complexity

Size

Duplication

Code Coverage

Importance

1. Missing Dependencies

2. Missing init.py files

1. Missing Dependencies

2. Missing init.py files

michaeljoseph / changes

Pull Request — master (#127)

GitHubApi.auth_token() A

Complexity

Size

Duplication

Code Coverage

Importance

1. Missing Dependencies

2. Missing __init__.py files

1. Missing Dependencies

2. Missing __init__.py files

Duplication Side-by-Side

Filter issues like

2. Missing init.py files

2. Missing init.py files