Remove sync_default_files from method signatures

synced_files has been removed. files_to_sync is always initialized with the default files path. If default files have not to be synchronized then they have to be removed from files_to_sync list.
packit · Dec 3, 2024 · f8a993c · f8a993c
1 parent fc00865
commit f8a993c
Show file tree

Hide file tree

Showing 5 changed files with 0 additions and 12 deletions.
diff --git a/packit_service/worker/handlers/distgit.py b/packit_service/worker/handlers/distgit.py
@@ -266,7 +266,6 @@ def sync_branch(
                 "create_pr": True,
                 "local_pr_branch_suffix": branch_suffix,
                 "use_downstream_specfile": is_pull_from_upstream_job,
-                "sync_default_files": not is_pull_from_upstream_job,
                 "add_pr_instructions": True,
                 "resolved_bugs": self.get_resolved_bugs(),
                 "release_monitoring_project_id": self.data.event_dict.get(

diff --git a/tests/integration/test_new_hotness_update.py b/tests/integration/test_new_hotness_update.py
@@ -215,7 +215,6 @@ def test_new_hotness_update(new_hotness_update, sync_release_model):
         create_pr=True,
         local_pr_branch_suffix="update-pull_from_upstream",
         use_downstream_specfile=True,
-        sync_default_files=False,
         add_pr_instructions=True,
         resolved_bugs=["rhbz#2106196"],
         release_monitoring_project_id=4181,
@@ -391,7 +390,6 @@ def test_new_hotness_update_non_git(new_hotness_update, sync_release_model_non_g
         create_pr=True,
         local_pr_branch_suffix="update-pull_from_upstream",
         use_downstream_specfile=True,
-        sync_default_files=False,
         add_pr_instructions=True,
         resolved_bugs=["rhbz#2106196"],
         release_monitoring_project_id=4181,

diff --git a/tests/integration/test_pr_comment.py b/tests/integration/test_pr_comment.py
@@ -2727,7 +2727,6 @@ def _get_project(url, *_, **__):
         create_pr=True,
         local_pr_branch_suffix="update-pull_from_upstream",
         use_downstream_specfile=True,
-        sync_default_files=False,
         add_pr_instructions=True,
         resolved_bugs=["rhbz#123", "rhbz#124"],
         release_monitoring_project_id=None,
@@ -2894,7 +2893,6 @@ def _get_project(url, *_, **__):
         create_pr=True,
         local_pr_branch_suffix="update-pull_from_upstream",
         use_downstream_specfile=True,
-        sync_default_files=False,
         add_pr_instructions=True,
         resolved_bugs=["rhbz#123", "rhbz#124"],
         release_monitoring_project_id=None,

diff --git a/tests/integration/test_release_event.py b/tests/integration/test_release_event.py
@@ -215,7 +215,6 @@ def test_dist_git_push_release_handle(
         create_pr=True,
         local_pr_branch_suffix="update-propose_downstream",
         use_downstream_specfile=False,
-        sync_default_files=True,
         add_pr_instructions=True,
         resolved_bugs=[],
         release_monitoring_project_id=None,
@@ -357,7 +356,6 @@ def test_dist_git_push_release_handle_multiple_branches(
             create_pr=True,
             local_pr_branch_suffix="update-propose_downstream",
             use_downstream_specfile=False,
-            sync_default_files=True,
             add_pr_instructions=True,
             resolved_bugs=[],
             release_monitoring_project_id=None,
@@ -507,7 +505,6 @@ def test_dist_git_push_release_handle_one_failed(
                 create_pr=True,
                 local_pr_branch_suffix="update-propose_downstream",
                 use_downstream_specfile=False,
-                sync_default_files=True,
                 add_pr_instructions=True,
                 resolved_bugs=[],
                 release_monitoring_project_id=None,
@@ -532,7 +529,6 @@ def test_dist_git_push_release_handle_one_failed(
                 create_pr=True,
                 local_pr_branch_suffix="update-propose_downstream",
                 use_downstream_specfile=False,
-                sync_default_files=True,
                 add_pr_instructions=True,
                 resolved_bugs=[],
                 release_monitoring_project_id=None,
@@ -785,7 +781,6 @@ def test_retry_propose_downstream_task(
         create_pr=True,
         local_pr_branch_suffix="update-propose_downstream",
         use_downstream_specfile=False,
-        sync_default_files=True,
         add_pr_instructions=True,
         resolved_bugs=[],
         release_monitoring_project_id=None,
@@ -901,7 +896,6 @@ def test_dont_retry_propose_downstream_task(
         create_pr=True,
         local_pr_branch_suffix="update-propose_downstream",
         use_downstream_specfile=False,
-        sync_default_files=True,
         add_pr_instructions=True,
         resolved_bugs=[],
         release_monitoring_project_id=None,

diff --git a/tests/unit/test_steve.py b/tests/unit/test_steve.py
@@ -186,7 +186,6 @@ def test_process_message(event, private, enabled_private_namespaces, success):
         create_pr=True,
         local_pr_branch_suffix="update-propose_downstream",
         use_downstream_specfile=False,
-        sync_default_files=True,
         add_pr_instructions=True,
         resolved_bugs=[],
         release_monitoring_project_id=None,