mirror of
https://github.com/khornberg/octokit.py
synced 2026-05-18 20:09:06 +03:00
Style: black
This commit is contained in:
Kyle Hornberg
+69
-208
@@ -20,12 +20,10 @@ class TestClientMethods(object):
|
||||
assert Octokit().activity.list_notifications.__doc__ == self.doc_string
|
||||
|
||||
def test_method_has_name_string(self):
|
||||
assert Octokit(
|
||||
).oauth_authorizations.list_grants.__name__ == "list_grants"
|
||||
assert Octokit().oauth_authorizations.list_grants.__name__ == "list_grants"
|
||||
|
||||
def test_method_is_accessible_also_by_snake_case_name(self):
|
||||
assert Octokit(
|
||||
).oauth_authorizations.list_your_grants.__name__ == "list_your_grants"
|
||||
assert Octokit().oauth_authorizations.list_your_grants.__name__ == "list_your_grants"
|
||||
|
||||
def test_method_calls_requests(self, mocker):
|
||||
mocker.patch("requests.get")
|
||||
@@ -38,8 +36,7 @@ class TestClientMethods(object):
|
||||
Octokit().oauth_authorizations.get_authorization()
|
||||
assert "authorization_id is a required parameter" == str(e1.value)
|
||||
with pytest.raises(errors.OctokitParameterError) as e2:
|
||||
Octokit().oauth_authorizations.get_authorization(
|
||||
authorization_id=None)
|
||||
Octokit().oauth_authorizations.get_authorization(authorization_id=None)
|
||||
assert "authorization_id must have a value" == str(e2.value)
|
||||
|
||||
def test_only_allows_valid_method_parameters(self):
|
||||
@@ -51,18 +48,16 @@ class TestClientMethods(object):
|
||||
mocker.patch("requests.get")
|
||||
Octokit().oauth_authorizations.get_authorization(authorization_id=100)
|
||||
requests.get.assert_called_once_with(
|
||||
"https://api.github.com/authorizations/100",
|
||||
params={},
|
||||
headers=Octokit().headers)
|
||||
"https://api.github.com/authorizations/100", params={}, headers=Octokit().headers
|
||||
)
|
||||
|
||||
def test_request_has_body_parameters(self, mocker):
|
||||
mocker.patch("requests.post")
|
||||
data = {"note": "remind me", "scopes": ["public_repo"]}
|
||||
create = Octokit().oauth_authorizations.create_authorization(**data)
|
||||
requests.post.assert_called_once_with(
|
||||
"https://api.github.com/authorizations",
|
||||
data=json.dumps(data, sort_keys=True),
|
||||
headers=create.headers)
|
||||
"https://api.github.com/authorizations", data=json.dumps(data, sort_keys=True), headers=create.headers
|
||||
)
|
||||
|
||||
def test_must_include_required_body_parameters(self):
|
||||
data = {"gist_id": "abc123", "note": "remind me"}
|
||||
@@ -70,8 +65,7 @@ class TestClientMethods(object):
|
||||
Octokit().oauth_authorizations.create_authorization(**data)
|
||||
assert "gist_id is not a valid parameter" == str(e.value)
|
||||
|
||||
def test_must_include_required_dictionary_sub_parameters_when_used(
|
||||
self, mocker):
|
||||
def test_must_include_required_dictionary_sub_parameters_when_used(self, mocker):
|
||||
mocker.patch("requests.get")
|
||||
data = {"owner": "owner", "repo": "repo", "name": "name"}
|
||||
with pytest.raises(errors.OctokitParameterError) as e:
|
||||
@@ -84,16 +78,9 @@ class TestClientMethods(object):
|
||||
data.update({"output": {"title": "blah", "summary": "that"}})
|
||||
Octokit().checks.create(**data)
|
||||
|
||||
def test_must_include_required_array_sub_parameters_when_used(
|
||||
self, mocker):
|
||||
def test_must_include_required_array_sub_parameters_when_used(self, mocker):
|
||||
mocker.patch("requests.get")
|
||||
data = {
|
||||
"owner": "owner",
|
||||
"repo": "repo",
|
||||
"name": "name",
|
||||
"head_sha": "master",
|
||||
"actions": []
|
||||
}
|
||||
data = {"owner": "owner", "repo": "repo", "name": "name", "head_sha": "master", "actions": []}
|
||||
with pytest.raises(errors.OctokitParameterError) as e:
|
||||
Octokit().checks.create(**data)
|
||||
assert "property is missing required items" == str(e.value)
|
||||
@@ -101,13 +88,7 @@ class TestClientMethods(object):
|
||||
with pytest.raises(errors.OctokitParameterError) as e:
|
||||
Octokit().checks.create(**data)
|
||||
assert "description is a required parameter" == str(e.value)
|
||||
data.update({
|
||||
"actions": [{
|
||||
"label": "blah",
|
||||
"description": "x",
|
||||
"identifier": "a"
|
||||
}]
|
||||
})
|
||||
data.update({"actions": [{"label": "blah", "description": "x", "identifier": "a"}]})
|
||||
Octokit().checks.create(**data)
|
||||
|
||||
def test_schema_types_must_match(self, mocker):
|
||||
@@ -117,21 +98,15 @@ class TestClientMethods(object):
|
||||
"repo": "repo",
|
||||
"name": "name",
|
||||
"head_sha": "master",
|
||||
"actions": {
|
||||
"desription": "blah"
|
||||
},
|
||||
"actions": {"desription": "blah"},
|
||||
}
|
||||
with pytest.raises(errors.OctokitParameterError) as e:
|
||||
Octokit().checks.create(**data)
|
||||
assert f'dict type does not match the schema type of array for the data of {data["actions"]}' == str(
|
||||
e.value)
|
||||
assert f'dict type does not match the schema type of array for the data of {data["actions"]}' == str(e.value)
|
||||
|
||||
def test_use_default_parameter_values(self, mocker):
|
||||
mocker.patch("requests.get")
|
||||
headers = {
|
||||
"Content-Type": "application/json",
|
||||
"accept": "application/vnd.github.v3+json"
|
||||
}
|
||||
headers = {"Content-Type": "application/json", "accept": "application/vnd.github.v3+json"}
|
||||
data = {
|
||||
"visibility": "all",
|
||||
"affiliation": "owner,collaborator,organization_member",
|
||||
@@ -141,15 +116,11 @@ class TestClientMethods(object):
|
||||
"page": 1,
|
||||
}
|
||||
Octokit().repos.list_for_authenticated_user()
|
||||
requests.get.assert_called_once_with(
|
||||
"https://api.github.com/user/repos", params=data, headers=headers)
|
||||
requests.get.assert_called_once_with("https://api.github.com/user/repos", params=data, headers=headers)
|
||||
|
||||
def test_deprecated_methods_are_available(self, mocker):
|
||||
mocker.patch("requests.get")
|
||||
headers = {
|
||||
"Content-Type": "application/json",
|
||||
"accept": "application/vnd.github.v3+json"
|
||||
}
|
||||
headers = {"Content-Type": "application/json", "accept": "application/vnd.github.v3+json"}
|
||||
data = {
|
||||
"visibility": "all",
|
||||
"affiliation": "owner,collaborator,organization_member",
|
||||
@@ -159,109 +130,63 @@ class TestClientMethods(object):
|
||||
"page": 1,
|
||||
}
|
||||
Octokit().repos.list()
|
||||
requests.get.assert_called_once_with(
|
||||
"https://api.github.com/user/repos", params=data, headers=headers)
|
||||
requests.get.assert_called_once_with("https://api.github.com/user/repos", params=data, headers=headers)
|
||||
|
||||
def test_use_passed_value_instead_of_default_parameter_values(
|
||||
self, mocker):
|
||||
def test_use_passed_value_instead_of_default_parameter_values(self, mocker):
|
||||
mocker.patch("requests.get")
|
||||
headers = {
|
||||
"Content-Type": "application/json",
|
||||
"accept": "application/vnd.github.v3+json"
|
||||
}
|
||||
headers = {"Content-Type": "application/json", "accept": "application/vnd.github.v3+json"}
|
||||
data = {"sort": "updated", "per_page": 30, "page": 1}
|
||||
Octokit().issues.list_comments_for_repo(owner="testUser",
|
||||
repo="testRepo",
|
||||
**data)
|
||||
Octokit().issues.list_comments_for_repo(owner="testUser", repo="testRepo", **data)
|
||||
requests.get.assert_called_once_with(
|
||||
"https://api.github.com/repos/testUser/testRepo/issues/comments",
|
||||
params=data,
|
||||
headers=headers)
|
||||
"https://api.github.com/repos/testUser/testRepo/issues/comments", params=data, headers=headers
|
||||
)
|
||||
|
||||
def test_validate_enum_values(self):
|
||||
with pytest.raises(errors.OctokitParameterError) as e:
|
||||
Octokit().issues.update(owner="testUser",
|
||||
repo="testRepo",
|
||||
issue_number=1,
|
||||
state="closeddddd")
|
||||
assert "closeddddd is not a valid option for state; must be one of ['open', 'closed']" == str(
|
||||
e.value)
|
||||
Octokit().issues.update(owner="testUser", repo="testRepo", issue_number=1, state="closeddddd")
|
||||
assert "closeddddd is not a valid option for state; must be one of ['open', 'closed']" == str(e.value)
|
||||
|
||||
def test_validate_boolean_values(self, mocker):
|
||||
mocker.patch("requests.post")
|
||||
Octokit().repos.create_deployment(owner="testUser",
|
||||
repo="testRepo",
|
||||
ref="abc123")
|
||||
Octokit().repos.create_deployment(owner="testUser", repo="testRepo", ref="abc123")
|
||||
data = '{"auto_merge": true, "description": "", "environment": "production", "payload": "", "ref": "abc123", "task": "deploy", "transient_environment": false}' # noqa E501
|
||||
headers = {
|
||||
"Content-Type": "application/json",
|
||||
"accept": "application/vnd.github.v3+json"
|
||||
}
|
||||
headers = {"Content-Type": "application/json", "accept": "application/vnd.github.v3+json"}
|
||||
requests.post.assert_called_once_with(
|
||||
"https://api.github.com/repos/testUser/testRepo/deployments",
|
||||
data=data,
|
||||
headers=headers)
|
||||
"https://api.github.com/repos/testUser/testRepo/deployments", data=data, headers=headers
|
||||
)
|
||||
|
||||
def test_non_default_params_not_in_the_url_for_get_requests_go_in_the_query_string(
|
||||
self, mocker):
|
||||
def test_non_default_params_not_in_the_url_for_get_requests_go_in_the_query_string(self, mocker):
|
||||
mocker.patch("requests.get")
|
||||
params = {"page": 2, "per_page": 30}
|
||||
Octokit().oauth_authorizations.list_grants(page=2)
|
||||
requests.get.assert_called_once_with(
|
||||
"https://api.github.com/applications/grants",
|
||||
params=params,
|
||||
headers=Octokit().headers)
|
||||
"https://api.github.com/applications/grants", params=params, headers=Octokit().headers
|
||||
)
|
||||
|
||||
def test_does_not_use_previous_values(self, mocker):
|
||||
mocker.patch("requests.patch")
|
||||
mocker.patch("requests.post")
|
||||
headers = {
|
||||
"accept": "application/vnd.github.v3+json",
|
||||
"Content-Type": "application/json"
|
||||
}
|
||||
headers = {"accept": "application/vnd.github.v3+json", "Content-Type": "application/json"}
|
||||
data = {"state": "closed"}
|
||||
issue = Octokit().issues.update(owner="testUser",
|
||||
repo="testRepo",
|
||||
issue_number=1,
|
||||
**data)
|
||||
issue = Octokit().issues.update(owner="testUser", repo="testRepo", issue_number=1, **data)
|
||||
requests.patch.assert_called_with(
|
||||
"https://api.github.com/repos/testUser/testRepo/issues/1",
|
||||
data=json.dumps(data),
|
||||
headers=headers)
|
||||
issue.pulls.create(owner="user",
|
||||
head="branch",
|
||||
base="master",
|
||||
title="Title",
|
||||
repo="testRepo")
|
||||
"https://api.github.com/repos/testUser/testRepo/issues/1", data=json.dumps(data), headers=headers
|
||||
)
|
||||
issue.pulls.create(owner="user", head="branch", base="master", title="Title", repo="testRepo")
|
||||
requests.post.assert_called_with(
|
||||
"https://api.github.com/repos/user/testRepo/pulls",
|
||||
data=json.dumps(
|
||||
{
|
||||
"base": "master",
|
||||
"head": "branch",
|
||||
"title": "Title"
|
||||
},
|
||||
sort_keys=True),
|
||||
headers={
|
||||
"Content-Type": "application/json",
|
||||
"accept": "application/vnd.github.v3+json"
|
||||
},
|
||||
data=json.dumps({"base": "master", "head": "branch", "title": "Title"}, sort_keys=True),
|
||||
headers={"Content-Type": "application/json", "accept": "application/vnd.github.v3+json"},
|
||||
)
|
||||
|
||||
def test_returned_object_is_not_self_but_a_copy_of_self(self, mocker):
|
||||
mocker.patch("requests.patch")
|
||||
headers = {
|
||||
"accept": "application/vnd.github.v3+json",
|
||||
"Content-Type": "application/json"
|
||||
}
|
||||
headers = {"accept": "application/vnd.github.v3+json", "Content-Type": "application/json"}
|
||||
octokit = Octokit()
|
||||
sut = octokit.issues.update(owner="testUser",
|
||||
repo="testRepo",
|
||||
issue_number=1)
|
||||
sut = octokit.issues.update(owner="testUser", repo="testRepo", issue_number=1)
|
||||
requests.patch.assert_called_once_with(
|
||||
"https://api.github.com/repos/testUser/testRepo/issues/1",
|
||||
data="{}",
|
||||
headers=headers)
|
||||
"https://api.github.com/repos/testUser/testRepo/issues/1", data="{}", headers=headers
|
||||
)
|
||||
assert sut.__class__.__name__ == "Octokit"
|
||||
assert sut != octokit
|
||||
|
||||
@@ -269,9 +194,7 @@ class TestClientMethods(object):
|
||||
patch = mocker.patch("requests.patch")
|
||||
Response = namedtuple("Response", ["json"])
|
||||
patch.return_value = Response(json=lambda: {})
|
||||
sut = Octokit().issues.update(owner="testUser",
|
||||
repo="testRepo",
|
||||
issue_number=1)
|
||||
sut = Octokit().issues.update(owner="testUser", repo="testRepo", issue_number=1)
|
||||
assert sut._response.__class__.__name__ == "Response"
|
||||
|
||||
def test_returned_object_has_json_attribute(self, mocker):
|
||||
@@ -279,13 +202,10 @@ class TestClientMethods(object):
|
||||
Request = namedtuple("Request", ["json"])
|
||||
patch.return_value = Request(json=lambda: data)
|
||||
data = {
|
||||
"documentation_url":
|
||||
"https://developer.github.com/v3/issues/#get-a-single-issue",
|
||||
"documentation_url": "https://developer.github.com/v3/issues/#get-a-single-issue",
|
||||
"message": "Not Found",
|
||||
}
|
||||
sut = Octokit().issues.get(owner="testUser",
|
||||
repo="testRepo",
|
||||
issue_number=1)
|
||||
sut = Octokit().issues.get(owner="testUser", repo="testRepo", issue_number=1)
|
||||
assert sut.json == data
|
||||
|
||||
def test_returned_object_has_response_attributes(self, mocker):
|
||||
@@ -295,21 +215,12 @@ class TestClientMethods(object):
|
||||
"number": 1347,
|
||||
"state": "open",
|
||||
"title": "Found a bug",
|
||||
"user": {
|
||||
"login": "octocat",
|
||||
"site_admin": False
|
||||
},
|
||||
"labels": [{
|
||||
"id": 208045946
|
||||
}, {
|
||||
"id": 208045947
|
||||
}],
|
||||
"user": {"login": "octocat", "site_admin": False},
|
||||
"labels": [{"id": 208045946}, {"id": 208045947}],
|
||||
}
|
||||
Request = namedtuple("Request", ["json"])
|
||||
patch.return_value = Request(json=lambda: data)
|
||||
sut = Octokit().issues.update(owner="testUser",
|
||||
repo="testRepo",
|
||||
issue_number=1)
|
||||
sut = Octokit().issues.update(owner="testUser", repo="testRepo", issue_number=1)
|
||||
assert sut.response.id == 1
|
||||
assert sut.response.number == 1347
|
||||
assert sut.response.state == "open"
|
||||
@@ -323,95 +234,53 @@ class TestClientMethods(object):
|
||||
data = [{"id": 208045946}, {"id": 208045947}]
|
||||
Request = namedtuple("Request", ["json"])
|
||||
patch.return_value = Request(json=lambda: data)
|
||||
sut = Octokit().issues.update(owner="testUser",
|
||||
repo="testRepo",
|
||||
issue_number=1)
|
||||
sut = Octokit().issues.update(owner="testUser", repo="testRepo", issue_number=1)
|
||||
assert sut.response[0].id == 208045946
|
||||
|
||||
def test_an_exception_with_json_is_replaced_by_the_raw_text(self, mocker):
|
||||
patch = mocker.patch("requests.patch")
|
||||
Request = namedtuple("Request", ["json"])
|
||||
patch.return_value = Request(json=lambda: "test")
|
||||
sut = Octokit().issues.update(owner="testUser",
|
||||
repo="testRepo",
|
||||
issue_number=1)
|
||||
sut = Octokit().issues.update(owner="testUser", repo="testRepo", issue_number=1)
|
||||
assert sut.json == "test"
|
||||
|
||||
def test_can_pass_in_optional_headers(self, mocker):
|
||||
mocker.patch("requests.get")
|
||||
headers = {
|
||||
"accept": "application/vnd.github.ant-man-preview+json",
|
||||
"Content-Type": "application/json"
|
||||
}
|
||||
headers = {"accept": "application/vnd.github.ant-man-preview+json", "Content-Type": "application/json"}
|
||||
Octokit().oauth_authorizations.get_authorization(
|
||||
authorization_id=100,
|
||||
headers={"accept": "application/vnd.github.ant-man-preview+json"})
|
||||
requests.get.assert_called_once_with(
|
||||
"https://api.github.com/authorizations/100",
|
||||
params={},
|
||||
headers=headers)
|
||||
authorization_id=100, headers={"accept": "application/vnd.github.ant-man-preview+json"}
|
||||
)
|
||||
requests.get.assert_called_once_with("https://api.github.com/authorizations/100", params={}, headers=headers)
|
||||
|
||||
def test_dictionary_keys_are_validated(self, mocker):
|
||||
mocker.patch("requests.put")
|
||||
headers = {
|
||||
"accept": "application/vnd.github.v3+json",
|
||||
"Content-Type": "application/json"
|
||||
}
|
||||
headers = {"accept": "application/vnd.github.v3+json", "Content-Type": "application/json"}
|
||||
data = {
|
||||
"required_status_checks": {
|
||||
"strict": True,
|
||||
"contexts": ["a", "b"]
|
||||
},
|
||||
"required_pull_request_reviews": {
|
||||
"dismiss_stale_reviews": True,
|
||||
"require_code_owner_reviews": True
|
||||
},
|
||||
"required_status_checks": {"strict": True, "contexts": ["a", "b"]},
|
||||
"required_pull_request_reviews": {"dismiss_stale_reviews": True, "require_code_owner_reviews": True},
|
||||
"enforce_admins": True,
|
||||
"restrictions": {
|
||||
"users": [],
|
||||
"teams": []
|
||||
},
|
||||
"restrictions": {"users": [], "teams": []},
|
||||
}
|
||||
Octokit().repos.update_branch_protection(owner="user",
|
||||
repo="repo",
|
||||
branch="branch",
|
||||
**data)
|
||||
Octokit().repos.update_branch_protection(owner="user", repo="repo", branch="branch", **data)
|
||||
requests.put.assert_called_with(
|
||||
"https://api.github.com/repos/user/repo/branches/branch/protection",
|
||||
data=json.dumps(data, sort_keys=True),
|
||||
headers=headers,
|
||||
)
|
||||
|
||||
def test_dictionary_keys_are_validated_multiple_times_in_a_row(
|
||||
self, mocker):
|
||||
def test_dictionary_keys_are_validated_multiple_times_in_a_row(self, mocker):
|
||||
mocker.patch("requests.put")
|
||||
headers = {
|
||||
"accept": "application/vnd.github.v3+json",
|
||||
"Content-Type": "application/json"
|
||||
}
|
||||
headers = {"accept": "application/vnd.github.v3+json", "Content-Type": "application/json"}
|
||||
data = {
|
||||
"required_status_checks": {
|
||||
"strict": True,
|
||||
"contexts": []
|
||||
},
|
||||
"required_pull_request_reviews": {
|
||||
"dismiss_stale_reviews": True
|
||||
},
|
||||
"required_status_checks": {"strict": True, "contexts": []},
|
||||
"required_pull_request_reviews": {"dismiss_stale_reviews": True},
|
||||
"enforce_admins": True,
|
||||
"restrictions": {
|
||||
"users": [],
|
||||
"teams": []
|
||||
},
|
||||
"restrictions": {"users": [], "teams": []},
|
||||
}
|
||||
for run in range(4):
|
||||
Octokit().repos.update_branch_protection(
|
||||
owner="user",
|
||||
repo="repo",
|
||||
branch="branch{}".format(run),
|
||||
**data)
|
||||
Octokit().repos.update_branch_protection(owner="user", repo="repo", branch="branch{}".format(run), **data)
|
||||
requests.put.assert_called_with(
|
||||
"https://api.github.com/repos/user/repo/branches/branch{}/protection"
|
||||
.format(run),
|
||||
"https://api.github.com/repos/user/repo/branches/branch{}/protection".format(run),
|
||||
data=json.dumps(data, sort_keys=True),
|
||||
headers=headers,
|
||||
)
|
||||
@@ -427,18 +296,10 @@ class TestMultipleValidations:
|
||||
"owner": "owner",
|
||||
"repo": "repo",
|
||||
"branch": "master",
|
||||
"required_status_checks": {
|
||||
"strict": True,
|
||||
"contexts": []
|
||||
},
|
||||
"required_pull_request_reviews": {
|
||||
"dismiss_stale_reviews": True
|
||||
},
|
||||
"required_status_checks": {"strict": True, "contexts": []},
|
||||
"required_pull_request_reviews": {"dismiss_stale_reviews": True},
|
||||
"enforce_admins": True,
|
||||
"restrictions": {
|
||||
"users": [],
|
||||
"teams": []
|
||||
},
|
||||
"restrictions": {"users": [], "teams": []},
|
||||
}
|
||||
octokit = Octokit()
|
||||
octokit.repos.update_branch_protection(**params)
|
||||
|
||||
Reference in New Issue
Block a user