| |
@@ -1,5 +1,5 @@
|
| |
"""
|
| |
- Unit tests for `toddler.plugins.pdc_unretire_packages`
|
| |
+ Unit tests for `toddler.plugins.unretire_packages`
|
| |
"""
|
| |
|
| |
import datetime
|
| |
@@ -12,16 +12,16 @@
|
| |
import pytest
|
| |
|
| |
from toddlers.exceptions import ValidationError
|
| |
- import toddlers.plugins.pdc_unretire_packages as pdc_unretire_packages
|
| |
+ import toddlers.plugins.unretire_packages as unretire_packages
|
| |
|
| |
|
| |
class TestAcceptTopic:
|
| |
"""
|
| |
- Test class for `toddler.plugins.pdc_unretire_packages.PDCUnretirePackages.accepts_topic`
|
| |
+ Test class for `toddler.plugins.unretire_packages.UnretirePackages.accepts_topic`
|
| |
method.
|
| |
"""
|
| |
|
| |
- toddler_cls = pdc_unretire_packages.PDCUnretirePackages
|
| |
+ toddler_cls = unretire_packages.UnretirePackages
|
| |
|
| |
def test_accepts_topic_invalid(self, toddler):
|
| |
"""
|
| |
@@ -38,18 +38,15 @@
|
| |
],
|
| |
)
|
| |
def test_accepts_topic_valid(self, topic, toddler):
|
| |
- """
|
| |
- Assert that valid topics are accepted.
|
| |
- """
|
| |
assert toddler.accepts_topic(topic)
|
| |
|
| |
|
| |
class TestProcess:
|
| |
"""
|
| |
- Test class for `toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages.process` method
|
| |
+ Test class for `toddlers.plugins.unretire_packages.UnretirePackages.process` method
|
| |
"""
|
| |
|
| |
- toddler_cls = pdc_unretire_packages.PDCUnretirePackages
|
| |
+ toddler_cls = unretire_packages.UnretirePackages
|
| |
|
| |
def test_process_invalid_project(self, caplog, toddler):
|
| |
"""
|
| |
@@ -64,7 +61,7 @@
|
| |
msg = IssueNewV1(body=body)
|
| |
|
| |
with patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages.process_ticket"
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages.process_ticket"
|
| |
) as mock_process_ticket:
|
| |
toddler.process({}, msg)
|
| |
mock_process_ticket.assert_not_called()
|
| |
@@ -81,7 +78,7 @@
|
| |
caplog.set_level(logging.INFO)
|
| |
|
| |
body = {
|
| |
- "project": {"fullname": pdc_unretire_packages.PROJECT_NAMESPACE},
|
| |
+ "project": {"fullname": unretire_packages.PROJECT_NAMESPACE},
|
| |
"issue": {
|
| |
"id": 100,
|
| |
"status": "Closed",
|
| |
@@ -92,7 +89,7 @@
|
| |
msg = IssueNewV1(body=body)
|
| |
|
| |
with patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages.process_ticket"
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages.process_ticket"
|
| |
) as mock_process_ticket:
|
| |
toddler.process({}, msg)
|
| |
|
| |
@@ -109,7 +106,7 @@
|
| |
caplog.set_level(logging.INFO)
|
| |
|
| |
body = {
|
| |
- "project": {"fullname": pdc_unretire_packages.PROJECT_NAMESPACE},
|
| |
+ "project": {"fullname": unretire_packages.PROJECT_NAMESPACE},
|
| |
"issue": {
|
| |
"id": 100,
|
| |
"status": "Open",
|
| |
@@ -120,7 +117,7 @@
|
| |
msg = IssueNewV1(body=body)
|
| |
|
| |
with patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages.process_ticket"
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages.process_ticket"
|
| |
) as mock_process_ticket:
|
| |
toddler.process({}, msg)
|
| |
|
| |
@@ -139,7 +136,7 @@
|
| |
caplog.set_level(logging.INFO)
|
| |
|
| |
body = {
|
| |
- "project": {"fullname": pdc_unretire_packages.PROJECT_NAMESPACE},
|
| |
+ "project": {"fullname": unretire_packages.PROJECT_NAMESPACE},
|
| |
"issue": {
|
| |
"id": 100,
|
| |
"status": "Open",
|
| |
@@ -150,7 +147,7 @@
|
| |
msg = IssueNewV1(body=body)
|
| |
|
| |
with patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages.process_ticket"
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages.process_ticket"
|
| |
) as mock_process_ticket:
|
| |
toddler.process({}, msg)
|
| |
|
| |
@@ -162,12 +159,13 @@
|
| |
"not a first word in title"
|
| |
)
|
| |
|
| |
+ @patch("toddlers.utils.bodhi.set_bodhi")
|
| |
@patch("toddlers.utils.pdc.pdc_client_for_config")
|
| |
@patch("toddlers.utils.pagure.set_pagure")
|
| |
@patch("toddlers.utils.bugzilla_system.set_bz")
|
| |
@patch("koji.ClientSession")
|
| |
def test_process_base_exception_appeared(
|
| |
- self, mock_koji, mock_bz, mock_pagure, mock_pdc, caplog, toddler
|
| |
+ self, mock_koji, mock_bz, mock_pagure, mock_pdc, mock_bodhi, caplog, toddler
|
| |
):
|
| |
"""
|
| |
Assert that if base exception appeared, it will be processed correctly.
|
| |
@@ -179,7 +177,7 @@
|
| |
"user": {"name": "zlopez"},
|
| |
}
|
| |
body = {
|
| |
- "project": {"fullname": pdc_unretire_packages.PROJECT_NAMESPACE},
|
| |
+ "project": {"fullname": unretire_packages.PROJECT_NAMESPACE},
|
| |
"issue": issue,
|
| |
}
|
| |
config = {
|
| |
@@ -193,7 +191,7 @@
|
| |
mock_pagure.return_value = mock_pagure_io
|
| |
|
| |
with patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages.process_ticket"
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages.process_ticket"
|
| |
) as mock_process_ticket:
|
| |
mock_process_ticket.side_effect = Exception("test")
|
| |
toddler.process(config, msg)
|
| |
@@ -201,11 +199,14 @@
|
| |
|
| |
mock_pagure_io.add_comment_to_issue.assert_called_once()
|
| |
|
| |
+ @patch("toddlers.utils.bodhi.set_bodhi")
|
| |
@patch("toddlers.utils.pdc.pdc_client_for_config")
|
| |
@patch("toddlers.utils.pagure.set_pagure")
|
| |
@patch("toddlers.utils.bugzilla_system.set_bz")
|
| |
@patch("koji.ClientSession")
|
| |
- def test_process(self, mock_koji, mock_bugzilla, mock_pagure, mock_pdc, toddler):
|
| |
+ def test_process(
|
| |
+ self, mock_koji, mock_bugzilla, mock_pagure, mock_pdc, mock_bodhi, toddler
|
| |
+ ):
|
| |
"""
|
| |
Assert that message toddler will be initialized correctly, if message passes
|
| |
initial processing.
|
| |
@@ -217,7 +218,7 @@
|
| |
"user": {"name": "zlopez"},
|
| |
}
|
| |
body = {
|
| |
- "project": {"fullname": pdc_unretire_packages.PROJECT_NAMESPACE},
|
| |
+ "project": {"fullname": unretire_packages.PROJECT_NAMESPACE},
|
| |
"issue": issue,
|
| |
}
|
| |
msg = IssueNewV1(body=body)
|
| |
@@ -237,7 +238,7 @@
|
| |
}
|
| |
|
| |
with patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages.process_ticket"
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages.process_ticket"
|
| |
) as mock_process_ticket:
|
| |
toddler.process(config, msg)
|
| |
|
| |
@@ -246,11 +247,12 @@
|
| |
mock_pdc.assert_called_with(config)
|
| |
mock_pagure.assert_called_with(config)
|
| |
mock_bugzilla.assert_called_with(config)
|
| |
+ mock_bodhi.assert_called_with(config)
|
| |
|
| |
|
| |
class TestProcessTicket:
|
| |
"""
|
| |
- Test class for `toddlers.plugin.pdc_unretire_packages.PDCUnretirePackages.process_ticket`
|
| |
+ Test class for `toddlers.plugin.unretire_packages.UnretirePackages.process_ticket`
|
| |
method.
|
| |
"""
|
| |
|
| |
@@ -258,36 +260,31 @@
|
| |
"""
|
| |
Initialize toddler
|
| |
"""
|
| |
- self.toddler = pdc_unretire_packages.PDCUnretirePackages()
|
| |
+ self.toddler = unretire_packages.UnretirePackages()
|
| |
self.toddler.temp_dir = Mock()
|
| |
self.toddler.git_repo = MagicMock()
|
| |
self.toddler.pagure_io = MagicMock()
|
| |
self.toddler.koji_session = MagicMock()
|
| |
+ self.toddler.bodhi = MagicMock()
|
| |
|
| |
- @patch("toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages._adjust_eol_pdc")
|
| |
- @patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages._unblock_tags_on_koji"
|
| |
- )
|
| |
+ @patch("toddlers.plugins.unretire_packages.UnretirePackages._unblock_tags_on_koji")
|
| |
@patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages"
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages"
|
| |
"._is_need_to_unblock_tags_on_koji",
|
| |
return_value=True,
|
| |
)
|
| |
@patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages._get_tags_to_unblock"
|
| |
- )
|
| |
- @patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages"
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages"
|
| |
"._is_package_ready_for_unretirement"
|
| |
)
|
| |
@patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages."
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages."
|
| |
"_get_branches_need_to_unretire"
|
| |
)
|
| |
- @patch("toddlers.plugins.pdc_unretire_packages.git.clone_repo")
|
| |
+ @patch("toddlers.plugins.unretire_packages.git.clone_repo")
|
| |
@patch("tempfile.TemporaryDirectory")
|
| |
@patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages._is_url_exist",
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages._is_url_exist",
|
| |
return_value=True,
|
| |
)
|
| |
def test_process_ticket_success(
|
| |
@@ -297,10 +294,8 @@
|
| |
mock_clone_repo,
|
| |
mock_get_branches,
|
| |
mock_is_package_ready,
|
| |
- mock_get_tags,
|
| |
mock_is_need_to_unblock_koji,
|
| |
mock_unblock_tags_koji,
|
| |
- mock_adjust_eol_pdc,
|
| |
caplog,
|
| |
):
|
| |
"""
|
| |
@@ -323,7 +318,8 @@
|
| |
package_url = "{0}/{1}/{2}.git".format(
|
| |
self.toddler.dist_git_base, namespace, repo
|
| |
)
|
| |
- tags_to_unblock = ["rawhide"]
|
| |
+ tags_to_unblock = ["rawhide", "f39", "f40", "f32", "f31"]
|
| |
+ current_releases = ["rawhide", "f39", "f40"]
|
| |
revert_message = f"Unretirement request: {issue['full_url']}"
|
| |
|
| |
mock_is_url_exist.return_value = True
|
| |
@@ -332,25 +328,23 @@
|
| |
self.toddler.git_repo.does_branch_exist.return_value = True
|
| |
self.toddler.git_repo.does_branch_contains_file.return_value = True
|
| |
mock_is_package_ready.return_value = True
|
| |
- # TODO learn weather i need it
|
| |
- mock_get_tags.return_value = tags_to_unblock
|
| |
mock_is_need_to_unblock_koji.return_value = True
|
| |
self.toddler.pagure_io.is_project_orphaned.return_value = True
|
| |
+ self.toddler.bodhi.get_active_branches.return_value = current_releases
|
| |
|
| |
self.toddler.process_ticket(issue)
|
| |
|
| |
mock_is_url_exist.assert_called_once_with(package_url)
|
| |
mock_get_branches.assert_called_once_with(issue_body=issue["content"])
|
| |
mock_is_package_ready.assert_called_once_with(
|
| |
- issue_id=issue["id"], issue_body=issue["content"], branches=tags_to_unblock
|
| |
+ issue_id=issue["id"], issue_body=issue["content"], branches=current_releases
|
| |
)
|
| |
- mock_get_tags.assert_called_once_with(issue_body=issue["content"])
|
| |
self.toddler.git_repo.revert_last_commit.assert_called_with(
|
| |
- message=revert_message, branch="rawhide"
|
| |
+ message=revert_message, branch="f40"
|
| |
)
|
| |
- mock_is_need_to_unblock_koji.assert_called_once_with(tags_to_unblock, repo)
|
| |
+ mock_is_need_to_unblock_koji.assert_called_once_with(current_releases, repo)
|
| |
mock_unblock_tags_koji.assert_called_once_with(
|
| |
- issue["id"], tags_to_unblock, repo
|
| |
+ issue["id"], current_releases, repo
|
| |
)
|
| |
self.toddler.pagure_io.is_project_orphaned.assert_called_once_with(
|
| |
namespace=namespace, repo=repo
|
| |
@@ -358,7 +352,6 @@
|
| |
self.toddler.pagure_io.assign_maintainer_to_project.assert_called_with(
|
| |
namespace=namespace, repo=repo, maintainer_fas=issue["user"]["name"]
|
| |
)
|
| |
- mock_adjust_eol_pdc.assert_called_once_with(namespace, repo, tags_to_unblock)
|
| |
assert caplog.records[-1].message == "everything was alright!"
|
| |
|
| |
# TODO mock _get_branches_need_to_unretire method
|
| |
@@ -384,7 +377,7 @@
|
| |
assert caplog.records[-1].message == msg
|
| |
self.toddler.pagure_io.close_issue.assert_called_once_with(
|
| |
issue["id"],
|
| |
- namespace=pdc_unretire_packages.PROJECT_NAMESPACE,
|
| |
+ namespace=unretire_packages.PROJECT_NAMESPACE,
|
| |
message=msg,
|
| |
reason=reason,
|
| |
)
|
| |
@@ -403,14 +396,14 @@
|
| |
msg = (
|
| |
"Unretire can only apply to rpm packages, please add `{0}`"
|
| |
" prefix before the package name"
|
| |
- ).format(pdc_unretire_packages.RPM_PREFIX)
|
| |
+ ).format(unretire_packages.RPM_PREFIX)
|
| |
reason = "Invalid"
|
| |
|
| |
self.toddler.process_ticket(issue)
|
| |
assert caplog.records[-1].message == msg
|
| |
self.toddler.pagure_io.close_issue.assert_called_once_with(
|
| |
issue["id"],
|
| |
- namespace=pdc_unretire_packages.PROJECT_NAMESPACE,
|
| |
+ namespace=unretire_packages.PROJECT_NAMESPACE,
|
| |
message=msg,
|
| |
reason=reason,
|
| |
)
|
| |
@@ -436,13 +429,13 @@
|
| |
assert caplog.records[-1].message == msg
|
| |
self.toddler.pagure_io.close_issue.assert_called_once_with(
|
| |
issue["id"],
|
| |
- namespace=pdc_unretire_packages.PROJECT_NAMESPACE,
|
| |
+ namespace=unretire_packages.PROJECT_NAMESPACE,
|
| |
message=msg,
|
| |
reason=reason,
|
| |
)
|
| |
|
| |
@patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages._is_url_exist",
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages._is_url_exist",
|
| |
return_value=False,
|
| |
)
|
| |
def test_process_ticket_package_url_doesnt_exist(self, mock_is_url_exist, caplog):
|
| |
@@ -465,15 +458,15 @@
|
| |
assert caplog.records[-1].message == msg
|
| |
self.toddler.pagure_io.close_issue.assert_called_once_with(
|
| |
issue["id"],
|
| |
- namespace=pdc_unretire_packages.PROJECT_NAMESPACE,
|
| |
+ namespace=unretire_packages.PROJECT_NAMESPACE,
|
| |
message=msg,
|
| |
reason=reason,
|
| |
)
|
| |
|
| |
- @patch("toddlers.plugins.pdc_unretire_packages.git.clone_repo")
|
| |
+ @patch("toddlers.plugins.unretire_packages.git.clone_repo")
|
| |
@patch("tempfile.TemporaryDirectory")
|
| |
@patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages._is_url_exist",
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages._is_url_exist",
|
| |
return_value=True,
|
| |
)
|
| |
def test_process_ticket_git_error_happened(
|
| |
@@ -498,28 +491,25 @@
|
| |
assert caplog.records[-1].message == msg
|
| |
self.toddler.pagure_io.close_issue.assert_called_once_with(
|
| |
issue["id"],
|
| |
- namespace=pdc_unretire_packages.PROJECT_NAMESPACE,
|
| |
+ namespace=unretire_packages.PROJECT_NAMESPACE,
|
| |
message=msg,
|
| |
reason=reason,
|
| |
)
|
| |
|
| |
@patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages._get_tags_to_unblock"
|
| |
- )
|
| |
- @patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages."
|
| |
- "PDCUnretirePackages._is_package_ready_for_unretirement",
|
| |
+ "toddlers.plugins.unretire_packages."
|
| |
+ "UnretirePackages._is_package_ready_for_unretirement",
|
| |
return_value=False,
|
| |
)
|
| |
@patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages."
|
| |
- "PDCUnretirePackages._get_branches_need_to_unretire",
|
| |
+ "toddlers.plugins.unretire_packages."
|
| |
+ "UnretirePackages._get_branches_need_to_unretire",
|
| |
return_value=["rawhide"],
|
| |
)
|
| |
- @patch("toddlers.plugins.pdc_unretire_packages.git.clone_repo")
|
| |
+ @patch("toddlers.plugins.unretire_packages.git.clone_repo")
|
| |
@patch("tempfile.TemporaryDirectory")
|
| |
@patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages._is_url_exist",
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages._is_url_exist",
|
| |
return_value=True,
|
| |
)
|
| |
def test_process_ticket_package_is_not_ready_for_unretirement(
|
| |
@@ -529,7 +519,6 @@
|
| |
mock_clone_repo,
|
| |
mock_get_branches,
|
| |
mock_is_package_ready,
|
| |
- mock_get_tags,
|
| |
caplog,
|
| |
):
|
| |
issue = {
|
| |
@@ -543,13 +532,13 @@
|
| |
}
|
| |
|
| |
self.toddler.process_ticket(issue)
|
| |
- mock_get_tags.assert_not_called()
|
| |
+ self.toddler.git_repo.revert_last_commit.assert_not_called()
|
| |
|
| |
|
| |
# TODO: create class for _get_branches_need_to_unretire method
|
| |
class TestIsPackageReadyForUnretirement:
|
| |
"""
|
| |
- Test class for `toddlers.plugin.pdc_unretire_packages.PDCUnreturePackages
|
| |
+ Test class for `toddlers.plugin.unretire_packages.UnreturePackages
|
| |
._is_package_ready_for_unretirement` method.
|
| |
"""
|
| |
|
| |
@@ -557,15 +546,15 @@
|
| |
"""
|
| |
Initialize toddler
|
| |
"""
|
| |
- self.toddler = pdc_unretire_packages.PDCUnretirePackages()
|
| |
+ self.toddler = unretire_packages.UnretirePackages()
|
| |
self.toddler.pagure_io = Mock()
|
| |
|
| |
@patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages."
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages."
|
| |
"_verify_bugzilla_need_to_be_checked"
|
| |
)
|
| |
@patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages."
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages."
|
| |
"_verify_package_not_retired_for_reason"
|
| |
)
|
| |
def test_is_package_ready_for_unretirement_success(
|
| |
@@ -599,7 +588,7 @@
|
| |
err_msg = "Package was retired for a reason: legal of license issue."
|
| |
|
| |
with patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages"
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages"
|
| |
"._verify_package_not_retired_for_reason",
|
| |
side_effect=ValidationError(err_msg),
|
| |
):
|
| |
@@ -611,13 +600,13 @@
|
| |
)
|
| |
self.toddler.pagure_io.close_issue.assert_called_with(
|
| |
issue_id=issue_id,
|
| |
- namespace=pdc_unretire_packages.PROJECT_NAMESPACE,
|
| |
+ namespace=unretire_packages.PROJECT_NAMESPACE,
|
| |
message=err_msg,
|
| |
reason="Invalid",
|
| |
)
|
| |
|
| |
@patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages."
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages."
|
| |
"_verify_package_not_retired_for_reason"
|
| |
)
|
| |
def test_is_package_ready_for_unretirement_date_issue(self, verify_reason_mock):
|
| |
@@ -630,7 +619,7 @@
|
| |
err_msg = "Couldn't get a date of the retire commit."
|
| |
|
| |
with patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages"
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages"
|
| |
"._verify_bugzilla_need_to_be_checked",
|
| |
side_effect=ValidationError(err_msg),
|
| |
):
|
| |
@@ -642,7 +631,7 @@
|
| |
)
|
| |
self.toddler.pagure_io.close_issue.assert_called_with(
|
| |
issue_id=issue_id,
|
| |
- namespace=pdc_unretire_packages.PROJECT_NAMESPACE,
|
| |
+ namespace=unretire_packages.PROJECT_NAMESPACE,
|
| |
message=err_msg,
|
| |
reason="Invalid",
|
| |
)
|
| |
@@ -650,7 +639,7 @@
|
| |
|
| |
class TestVerifyPackageNotRetiredForReason:
|
| |
"""
|
| |
- Test class for `toddlers.plugin.pdc_unretire_packages.PDCUnreturePackages
|
| |
+ Test class for `toddlers.plugin.unretire_packages.UnreturePackages
|
| |
._verify_package_not_retired_for_reason` method.
|
| |
"""
|
| |
|
| |
@@ -658,7 +647,7 @@
|
| |
"""
|
| |
Initialize toddler.
|
| |
"""
|
| |
- self.toddler = pdc_unretire_packages.PDCUnretirePackages()
|
| |
+ self.toddler = unretire_packages.UnretirePackages()
|
| |
self.toddler.git_repo = Mock()
|
| |
|
| |
def test_verify_package_not_retired_for_reason_success(self):
|
| |
@@ -687,7 +676,7 @@
|
| |
|
| |
class TestVerifyBugzillaNeedToBeChecked:
|
| |
"""
|
| |
- Test class for `toddlers.plugin.pdc_unretire_packages.PDCUnreturePackages
|
| |
+ Test class for `toddlers.plugin.unretire_packages.UnreturePackages
|
| |
._verify_bugzilla_need_to_be_checked` method.
|
| |
"""
|
| |
|
| |
@@ -695,11 +684,11 @@
|
| |
"""
|
| |
Initialize toddler.
|
| |
"""
|
| |
- self.toddler = pdc_unretire_packages.PDCUnretirePackages()
|
| |
+ self.toddler = unretire_packages.UnretirePackages()
|
| |
self.toddler.git_repo = Mock()
|
| |
|
| |
@patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages._verify_bugzilla_ticket"
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages._verify_bugzilla_ticket"
|
| |
)
|
| |
def test_verify_bugzilla_need_to_be_checked_success(self, mock_verify_bz):
|
| |
"""
|
| |
@@ -718,7 +707,7 @@
|
| |
mock_verify_bz.assert_not_called()
|
| |
|
| |
@patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages._verify_bugzilla_ticket"
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages._verify_bugzilla_ticket"
|
| |
)
|
| |
def test_verify_bugzilla_need_to_be_checked_date_is_none(self, mock_verify_bz):
|
| |
"""
|
| |
@@ -738,7 +727,7 @@
|
| |
mock_verify_bz.assert_not_called()
|
| |
|
| |
@patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages._verify_bugzilla_ticket"
|
| |
+ "toddlers.plugins.unretire_packages.UnretirePackages._verify_bugzilla_ticket"
|
| |
)
|
| |
def test_verify_bugzilla_need_to_be_checked_need_to_check_bugzilla(
|
| |
self, mock_verify_bz
|
| |
@@ -757,7 +746,7 @@
|
| |
|
| |
class TestVerifyBugzillaTicket:
|
| |
"""
|
| |
- Test class for `toddlers.plugin.pdc_unretire_packages.PDCUnreturePackages
|
| |
+ Test class for `toddlers.plugin.unretire_packages.UnreturePackages
|
| |
._verify_bugzilla_ticket` method.
|
| |
"""
|
| |
|
| |
@@ -765,9 +754,9 @@
|
| |
"""
|
| |
Initialize toddler.
|
| |
"""
|
| |
- self.toddler = pdc_unretire_packages.PDCUnretirePackages()
|
| |
+ self.toddler = unretire_packages.UnretirePackages()
|
| |
|
| |
- @patch("toddlers.plugins.pdc_unretire_packages.bugzilla_system.get_bug")
|
| |
+ @patch("toddlers.plugins.unretire_packages.bugzilla_system.get_bug")
|
| |
def test_verify_bugzilla_ticket_success(self, get_bug_mock):
|
| |
"""
|
| |
Assert that method process correctly.
|
| |
@@ -796,7 +785,7 @@
|
| |
self.toddler._verify_bugzilla_ticket(issue_body)
|
| |
|
| |
@patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.bugzilla_system.get_bug",
|
| |
+ "toddlers.plugins.unretire_packages.bugzilla_system.get_bug",
|
| |
side_effect=Exception("bug error"),
|
| |
)
|
| |
def test_verify_bugzilla_ticket_get_bug_error(self, mock_get_bug):
|
| |
@@ -813,7 +802,7 @@
|
| |
self.toddler._verify_bugzilla_ticket(issue_body)
|
| |
|
| |
@patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.bugzilla_system.get_bug",
|
| |
+ "toddlers.plugins.unretire_packages.bugzilla_system.get_bug",
|
| |
return_value=None,
|
| |
)
|
| |
def test_verify_bugzilla_ticket_bug_not_found(self, get_bug_mock):
|
| |
@@ -826,7 +815,7 @@
|
| |
with pytest.raises(ValidationError, match=err_msg):
|
| |
self.toddler._verify_bugzilla_ticket(issue_body)
|
| |
|
| |
- @patch("toddlers.plugins.pdc_unretire_packages.bugzilla_system.get_bug")
|
| |
+ @patch("toddlers.plugins.unretire_packages.bugzilla_system.get_bug")
|
| |
def test_verify_bugzilla_ticket_bug_product_is_not_fedora(self, mock_get_bug):
|
| |
"""
|
| |
Assert that method will raise error if the bug product is not `Fedora`.
|
| |
@@ -844,7 +833,7 @@
|
| |
with pytest.raises(ValidationError, match=err_msg):
|
| |
self.toddler._verify_bugzilla_ticket(issue_body)
|
| |
|
| |
- @patch("toddlers.plugins.pdc_unretire_packages.bugzilla_system.get_bug")
|
| |
+ @patch("toddlers.plugins.unretire_packages.bugzilla_system.get_bug")
|
| |
def test_verify_bugzilla_ticket_bug_does_not_have_required_flag(self, get_bug_mock):
|
| |
"""
|
| |
Assert that method will raise Exception if bug doesn't have required flag.
|
| |
@@ -862,7 +851,7 @@
|
| |
with pytest.raises(ValidationError, match=err_msg):
|
| |
self.toddler._verify_bugzilla_ticket(issue_body)
|
| |
|
| |
- @patch("toddlers.plugins.pdc_unretire_packages.bugzilla_system.get_bug")
|
| |
+ @patch("toddlers.plugins.unretire_packages.bugzilla_system.get_bug")
|
| |
def test_verify_bugzilla_ticket_wrong_flag_status(self, get_bug_mock):
|
| |
"""
|
| |
Assert that method will raise Exception if flag has wrong status.
|
| |
@@ -885,15 +874,15 @@
|
| |
|
| |
class TestBranchesNeedToUnretire:
|
| |
"""
|
| |
- Test class for `toddlers.plugin.pdc_unretire_packages.
|
| |
- PDCUnreturePackages._get_branches_need_to_unretire` method.
|
| |
+ Test class for `toddlers.plugin.unretire_packages.
|
| |
+ UnreturePackages._get_branches_need_to_unretire` method.
|
| |
"""
|
| |
|
| |
def setup_method(self):
|
| |
"""
|
| |
Initialize toddler.
|
| |
"""
|
| |
- self.toddler = pdc_unretire_packages.PDCUnretirePackages()
|
| |
+ self.toddler = unretire_packages.UnretirePackages()
|
| |
|
| |
def test_get_branches_need_to_unretire_no_branch_names_in_issue_body(self):
|
| |
"""
|
| |
@@ -912,122 +901,9 @@
|
| |
assert self.toddler._get_branches_need_to_unretire(issue_body) == result
|
| |
|
| |
|
| |
- class TestGetTagsToUnblock:
|
| |
- """
|
| |
- Test class for `toddlers.plugin.pdc_unretire_packages.PDCUnreturePackages._get_tags_to_unblock`
|
| |
- method.
|
| |
- """
|
| |
-
|
| |
- def setup_method(self):
|
| |
- """
|
| |
- Initialize toddler.
|
| |
- """
|
| |
- self.toddler = pdc_unretire_packages.PDCUnretirePackages()
|
| |
-
|
| |
- @patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages"
|
| |
- "._get_last_fedora_version_branch_name",
|
| |
- return_value="f38",
|
| |
- )
|
| |
- def test_get_tags_to_unblock_no_tags_in_issue_body(self, get_last_branch_mock):
|
| |
- """
|
| |
- Assert that get tags to unblock process correctly if the last branch is rawhide
|
| |
- """
|
| |
- issue_body = "some body"
|
| |
-
|
| |
- assert self.toddler._get_tags_to_unblock(issue_body) == ["f38"]
|
| |
-
|
| |
- @patch(
|
| |
- "toddlers.plugins.pdc_unretire_packages.PDCUnretirePackages"
|
| |
- "._get_last_fedora_version_branch_name",
|
| |
- return_value="f38",
|
| |
- )
|
| |
- def test_get_tags_to_unblock_there_are_tags_in_issue_body(
|
| |
- self, get_last_branch_mock
|
| |
- ):
|
| |
- """
|
| |
- Assert that get tags to unblock process correctly if there are some tags in issue body
|
| |
- """
|
| |
- issue_body = "f35, f36, f37, rawhide"
|
| |
-
|
| |
- assert self.toddler._get_tags_to_unblock(issue_body) == [
|
| |
- "f35",
|
| |
- "f36",
|
| |
- "f37",
|
| |
- "rawhide",
|
| |
- "f38",
|
| |
- ]
|
| |
-
|
| |
-
|
| |
- class TestGetLastFedoraVersionBranchName:
|
| |
- """
|
| |
- Test class for `toddlers.plugin.pdc_unretire_packages.PDCUnreturePackages
|
| |
- ._get_last_fedora_version_branch_name` method.
|
| |
- """
|
| |
-
|
| |
- def setup_method(self):
|
| |
- """
|
| |
- Initialize toddler.
|
| |
- """
|
| |
- self.toddler = pdc_unretire_packages.PDCUnretirePackages()
|
| |
-
|
| |
- @patch("requests.get")
|
| |
- def test_get_last_fedora_version_branch_name_no_rawhide_branch(
|
| |
- self, get_request_mock
|
| |
- ):
|
| |
- """
|
| |
- Assert error during processing if func can't find the rawhide branch on bodhi.
|
| |
- """
|
| |
- response = Mock()
|
| |
- response.json.return_value = {"total": 2, "releases": []}
|
| |
- get_request_mock.return_value = response
|
| |
- err_msg = "Not able to find a rawhide branch."
|
| |
-
|
| |
- with pytest.raises(ValidationError, match=err_msg):
|
| |
- self.toddler._get_last_fedora_version_branch_name()
|
| |
-
|
| |
- @patch("requests.get")
|
| |
- def test_get_last_fedora_version_branch_name_too_many_rawhide_branches(
|
| |
- self, get_request_mock
|
| |
- ):
|
| |
- """
|
| |
- Assert error during processing if func found too many rawhide branches.
|
| |
- """
|
| |
- response = Mock()
|
| |
- response.json.return_value = {
|
| |
- "total": 2,
|
| |
- "releases": [
|
| |
- {"dist_tag": "f37", "state": "pending", "branch": "rawhide"},
|
| |
- {"dist_tag": "f38", "state": "pending", "branch": "rawhide"},
|
| |
- ],
|
| |
- }
|
| |
- get_request_mock.return_value = response
|
| |
- err_msg = "Not able to find a rawhide branch."
|
| |
-
|
| |
- with pytest.raises(ValidationError, match=err_msg):
|
| |
- self.toddler._get_last_fedora_version_branch_name()
|
| |
-
|
| |
- @patch("requests.get")
|
| |
- def test_get_last_fedora_version_branch_name(self, get_request_mock):
|
| |
- """
|
| |
- Assert that get last fedora version branch process correctly.
|
| |
- """
|
| |
- response = Mock()
|
| |
- response.json.return_value = {
|
| |
- "total": 2,
|
| |
- "releases": [
|
| |
- {"dist_tag": "f37", "state": "pending", "branch": "pupipup"},
|
| |
- {"dist_tag": "f38", "state": "pending", "branch": "rawhide"},
|
| |
- ],
|
| |
- }
|
| |
- get_request_mock.return_value = response
|
| |
-
|
| |
- assert self.toddler._get_last_fedora_version_branch_name() == "f38"
|
| |
-
|
| |
-
|
| |
class TestIsNeedToUnblockTagsOnKoji:
|
| |
"""
|
| |
- Test class for `toddlers.plugin.pdc_unretire_packages.PDCUnreturePackages
|
| |
+ Test class for `toddlers.plugin.unretire_packages.UnreturePackages
|
| |
._is_need_to_unblock_tags_on_koji` method.
|
| |
"""
|
| |
|
| |
@@ -1035,7 +911,7 @@
|
| |
"""
|
| |
Initialize toddler.
|
| |
"""
|
| |
- self.toddler = pdc_unretire_packages.PDCUnretirePackages()
|
| |
+ self.toddler = unretire_packages.UnretirePackages()
|
| |
self.toddler.koji_session = Mock()
|
| |
|
| |
def test_is_need_to_unblock_tags_on_koji_package_does_not_have_tags(self):
|
| |
pdc import should be removed