| |
@@ -27,7 +27,9 @@
|
| |
self.session, "orphan", "orphan", ["orphan@fedoraproject.org"]
|
| |
)
|
| |
repo = pagure.lib.query.get_authorized_project(
|
| |
- self.session, "test3", namespace="somenamespace",
|
| |
+ self.session,
|
| |
+ "test3",
|
| |
+ namespace="somenamespace",
|
| |
)
|
| |
token = pagure.lib.query.get_api_token(self.session, "aaabbbcccddd")
|
| |
token.project = repo
|
| |
@@ -36,8 +38,7 @@
|
| |
self._app.register_blueprint(plugin.DISTGIT_NS)
|
| |
|
| |
def test_token_missing_ACL(self):
|
| |
- """Test the orphan endpoint with an API token missing the `modify_project` ACL.
|
| |
- """
|
| |
+ """Test the orphan endpoint with an API token missing the `modify_project` ACL."""
|
| |
headers = {"Authorization": "token foo_token"}
|
| |
output = self.app.post(
|
| |
"/_dg/orphan/somenamespace/test3", headers=headers
|
| |
@@ -50,64 +51,76 @@
|
| |
headers = {"Authorization": "token BBBZZZOOO"}
|
| |
datainput = {}
|
| |
output = self.app.post(
|
| |
- "/_dg/orphan/somenamespace/test3", data=datainput, headers=headers,
|
| |
+ "/_dg/orphan/somenamespace/test3",
|
| |
+ data=datainput,
|
| |
+ headers=headers,
|
| |
)
|
| |
assert output.status_code == 401
|
| |
|
| |
def test_empty_form(self):
|
| |
- """Assert that empty form is accepted.
|
| |
- """
|
| |
+ """Assert that empty form is accepted."""
|
| |
headers = {"Authorization": "token aaabbbcccddd"}
|
| |
datainput = {}
|
| |
output = self.app.post(
|
| |
- "/_dg/orphan/somenamespace/test3", data=datainput, headers=headers,
|
| |
+ "/_dg/orphan/somenamespace/test3",
|
| |
+ data=datainput,
|
| |
+ headers=headers,
|
| |
)
|
| |
assert output.status_code == 200
|
| |
|
| |
def test_invalid_form(self):
|
| |
- """Assert that invalid form is not accepted.
|
| |
- """
|
| |
+ """Assert that invalid form is not accepted."""
|
| |
headers = {"Authorization": "token aaabbbcccddd"}
|
| |
datainput = {"orphan_reason": "bar"}
|
| |
output = self.app.post(
|
| |
- "/_dg/orphan/somenamespace/test3", data=datainput, headers=headers,
|
| |
+ "/_dg/orphan/somenamespace/test3",
|
| |
+ data=datainput,
|
| |
+ headers=headers,
|
| |
)
|
| |
assert output.status_code == 400
|
| |
|
| |
def test_already_orphaned_package(self):
|
| |
- """Assert that already orphaned package can't be orphaned again.
|
| |
- """
|
| |
+ """Assert that already orphaned package can't be orphaned again."""
|
| |
headers = {"Authorization": "token aaabbbcccddd"}
|
| |
datainput = {}
|
| |
orphan_user_obj = pagure.lib.query.get_user(self.session, "orphan")
|
| |
repo = pagure.lib.query.get_authorized_project(
|
| |
- self.session, "test3", namespace="somenamespace",
|
| |
+ self.session,
|
| |
+ "test3",
|
| |
+ namespace="somenamespace",
|
| |
)
|
| |
repo.user = orphan_user_obj
|
| |
self.session.add(repo)
|
| |
self.session.commit()
|
| |
output = self.app.post(
|
| |
- "/_dg/orphan/somenamespace/test3", data=datainput, headers=headers,
|
| |
+ "/_dg/orphan/somenamespace/test3",
|
| |
+ data=datainput,
|
| |
+ headers=headers,
|
| |
)
|
| |
assert output.status_code == 401
|
| |
|
| |
@patch("pagure_distgit.plugin.pagure.lib.notify.log")
|
| |
def test_orphan_package(self, mock_log):
|
| |
- """Assert that package is correctly orphaned.
|
| |
- """
|
| |
+ """Assert that package is correctly orphaned."""
|
| |
headers = {"Authorization": "token aaabbbcccddd"}
|
| |
datainput = {"orphan_reason": "Lack of time"}
|
| |
repo = pagure.lib.query.get_authorized_project(
|
| |
- self.session, "test3", namespace="somenamespace",
|
| |
+ self.session,
|
| |
+ "test3",
|
| |
+ namespace="somenamespace",
|
| |
)
|
| |
user = repo.user
|
| |
project_user = pagure.lib.model.ProjectUser(
|
| |
- project_id=repo.id, user_id=repo.user.id, access="admin",
|
| |
+ project_id=repo.id,
|
| |
+ user_id=repo.user.id,
|
| |
+ access="admin",
|
| |
)
|
| |
self.session.add(project_user)
|
| |
self.session.commit()
|
| |
output = self.app.post(
|
| |
- "/_dg/orphan/somenamespace/test3", data=datainput, headers=headers,
|
| |
+ "/_dg/orphan/somenamespace/test3",
|
| |
+ data=datainput,
|
| |
+ headers=headers,
|
| |
)
|
| |
assert output.status_code == 200
|
| |
data = json.loads(output.get_data(as_text=True))
|
| |
@@ -127,16 +140,19 @@
|
| |
|
| |
@patch("pagure_distgit.plugin.pagure.lib.notify.log")
|
| |
def test_orphan_package_repo_users_empty(self, mock_log):
|
| |
- """Assert that package is correctly orphaned.
|
| |
- """
|
| |
+ """Assert that package is correctly orphaned."""
|
| |
headers = {"Authorization": "token aaabbbcccddd"}
|
| |
datainput = {"orphan_reason": "Lack of time"}
|
| |
repo = pagure.lib.query.get_authorized_project(
|
| |
- self.session, "test3", namespace="somenamespace",
|
| |
+ self.session,
|
| |
+ "test3",
|
| |
+ namespace="somenamespace",
|
| |
)
|
| |
user = repo.user
|
| |
output = self.app.post(
|
| |
- "/_dg/orphan/somenamespace/test3", data=datainput, headers=headers,
|
| |
+ "/_dg/orphan/somenamespace/test3",
|
| |
+ data=datainput,
|
| |
+ headers=headers,
|
| |
)
|
| |
assert output.status_code == 200
|
| |
data = json.loads(output.get_data(as_text=True))
|
| |
@@ -155,8 +171,7 @@
|
| |
assert mock_log.call_count == 1
|
| |
|
| |
def test_orphan_package_repo_no_orphan_user(self):
|
| |
- """Assert that missing orphan user is correctly handled.
|
| |
- """
|
| |
+ """Assert that missing orphan user is correctly handled."""
|
| |
headers = {"Authorization": "token aaabbbcccddd"}
|
| |
datainput = {}
|
| |
orphan_user_obj = pagure.lib.query.get_user(self.session, "orphan")
|
| |
@@ -164,7 +179,9 @@
|
| |
self.session.commit()
|
| |
|
| |
output = self.app.post(
|
| |
- "/_dg/orphan/somenamespace/test3", data=datainput, headers=headers,
|
| |
+ "/_dg/orphan/somenamespace/test3",
|
| |
+ data=datainput,
|
| |
+ headers=headers,
|
| |
)
|
| |
assert output.status_code == 400
|
| |
|
| |
@@ -184,7 +201,9 @@
|
| |
self.session, "orphan", "orphan", ["orphan@fedoraproject.org"]
|
| |
)
|
| |
repo = pagure.lib.query.get_authorized_project(
|
| |
- self.session, "test3", namespace="somenamespace",
|
| |
+ self.session,
|
| |
+ "test3",
|
| |
+ namespace="somenamespace",
|
| |
)
|
| |
token = pagure.lib.query.get_api_token(self.session, "aaabbbcccddd")
|
| |
token.project = repo
|
| |
@@ -204,15 +223,18 @@
|
| |
self.assertDictEqual(data, exp)
|
| |
|
| |
def test_orphaned(self):
|
| |
- """ Assert that correct output is returned when package is orphaned.
|
| |
- """
|
| |
+ """Assert that correct output is returned when package is orphaned."""
|
| |
orphan_user_obj = pagure.lib.query.get_user(self.session, "orphan")
|
| |
repo = pagure.lib.query.get_authorized_project(
|
| |
- self.session, "test3", namespace="somenamespace",
|
| |
+ self.session,
|
| |
+ "test3",
|
| |
+ namespace="somenamespace",
|
| |
)
|
| |
repo.user = orphan_user_obj
|
| |
reason = model.PagureOrphanReason(
|
| |
- project_id=repo.id, reason="reason", reason_info="reason_info",
|
| |
+ project_id=repo.id,
|
| |
+ reason="reason",
|
| |
+ reason_info="reason_info",
|
| |
)
|
| |
self.session.add(reason)
|
| |
self.session.add(repo)
|
| |
@@ -229,11 +251,12 @@
|
| |
self.assertDictEqual(data, exp)
|
| |
|
| |
def test_orphaned_no_reason(self):
|
| |
- """ Assert that correct output is returned when package is orphaned.
|
| |
- """
|
| |
+ """Assert that correct output is returned when package is orphaned."""
|
| |
orphan_user_obj = pagure.lib.query.get_user(self.session, "orphan")
|
| |
repo = pagure.lib.query.get_authorized_project(
|
| |
- self.session, "test3", namespace="somenamespace",
|
| |
+ self.session,
|
| |
+ "test3",
|
| |
+ namespace="somenamespace",
|
| |
)
|
| |
repo.user = orphan_user_obj
|
| |
self.session.add(repo)
|
| |
@@ -285,7 +308,12 @@
|
| |
description="test project #4",
|
| |
hook_token="aaabbbcccdd",
|
| |
)
|
| |
- item.close_status = ["Invalid", "Insufficient data", "Fixed", "Duplicate"]
|
| |
+ item.close_status = [
|
| |
+ "Invalid",
|
| |
+ "Insufficient data",
|
| |
+ "Fixed",
|
| |
+ "Duplicate",
|
| |
+ ]
|
| |
self.session.add(item)
|
| |
self.session.flush()
|
| |
tests.create_locks(self.session, item)
|
| |
@@ -295,14 +323,18 @@
|
| |
pygit2.init_repository(repo_path, bare=True)
|
| |
|
| |
repo = pagure.lib.query.get_authorized_project(
|
| |
- self.session, "test4", namespace="rpms",
|
| |
+ self.session,
|
| |
+ "test4",
|
| |
+ namespace="rpms",
|
| |
)
|
| |
token = pagure.lib.query.get_api_token(self.session, "aaabbbcccddd")
|
| |
token.project = repo
|
| |
orphan_user_obj = pagure.lib.query.get_user(self.session, "orphan")
|
| |
repo.user = orphan_user_obj
|
| |
reason = model.PagureOrphanReason(
|
| |
- project_id=repo.id, reason="reason", reason_info="reason_info",
|
| |
+ project_id=repo.id,
|
| |
+ reason="reason",
|
| |
+ reason_info="reason_info",
|
| |
)
|
| |
self.session.add(reason)
|
| |
self.session.add(repo)
|
| |
@@ -316,9 +348,7 @@
|
| |
the `modify_project` ACL.
|
| |
"""
|
| |
headers = {"Authorization": "token foo_token"}
|
| |
- output = self.app.post(
|
| |
- "/_dg/take_orphan/rpms/test4", headers=headers
|
| |
- )
|
| |
+ output = self.app.post("/_dg/take_orphan/rpms/test4", headers=headers)
|
| |
# invalid token
|
| |
assert output.status_code == 401
|
| |
|
| |
@@ -326,23 +356,26 @@
|
| |
"""Test the take orphan endpoint with an invalid API token. """
|
| |
headers = {"Authorization": "token BBBZZZOOO"}
|
| |
output = self.app.post(
|
| |
- "/_dg/take_orphan/rpms/test4", headers=headers,
|
| |
+ "/_dg/take_orphan/somenamespace/test3",
|
| |
+ headers=headers,
|
| |
)
|
| |
assert output.status_code == 401
|
| |
|
| |
@patch("pagure_distgit.plugin._is_active_in_pdc")
|
| |
@patch("pagure_distgit.plugin.pagure.lib.notify.log")
|
| |
def test_take_orphan(self, mock_log, mock_pdc):
|
| |
- """Assert that package is correctly adopted.
|
| |
- """
|
| |
+ """Assert that package is correctly adopted."""
|
| |
mock_pdc.return_value = True
|
| |
headers = {"Authorization": "token aaabbbcccddd"}
|
| |
repo = pagure.lib.query.get_authorized_project(
|
| |
- self.session, "test4", namespace="rpms",
|
| |
+ self.session,
|
| |
+ "test4",
|
| |
+ namespace="rpms",
|
| |
)
|
| |
assert repo.orphan_reason.reason == "reason"
|
| |
output = self.app.post(
|
| |
- "/_dg/take_orphan/rpms/test4", headers=headers,
|
| |
+ "/_dg/take_orphan/rpms/test4",
|
| |
+ headers=headers,
|
| |
)
|
| |
assert output.status_code == 200
|
| |
data = json.loads(output.get_data(as_text=True))
|
| |
@@ -369,13 +402,16 @@
|
| |
mock_req.get.return_value = resp
|
| |
headers = {"Authorization": "token aaabbbcccddd"}
|
| |
repo = pagure.lib.query.get_authorized_project(
|
| |
- self.session, "test4", namespace="rpms",
|
| |
+ self.session,
|
| |
+ "test4",
|
| |
+ namespace="rpms",
|
| |
)
|
| |
reason = repo.orphan_reason
|
| |
self.session.delete(reason)
|
| |
self.session.commit()
|
| |
output = self.app.post(
|
| |
- "/_dg/take_orphan/rpms/test4", headers=headers,
|
| |
+ "/_dg/take_orphan/rpms/test4",
|
| |
+ headers=headers,
|
| |
)
|
| |
assert output.status_code == 200
|
| |
data = json.loads(output.get_data(as_text=True))
|
| |
@@ -391,5 +427,5 @@
|
| |
mock_req.get.assert_called_with(
|
| |
"invalid:/component-branches/?"
|
| |
"global_component=test4&name=rawhide&type=rpm",
|
| |
- timeout=(30, 30)
|
| |
+ timeout=(30, 30),
|
| |
)
|
| |
I dont think
PDC_BRANCH_OVERRIDES
is set, it is being used in tests but I couldn't find it in the normal code base though.