From 8851deb565fc0f88a0f7c1bf9aa08e83c3f8ab9e Mon Sep 17 00:00:00 2001 From: Pavla Kratochvilova Date: Mon, 23 Sep 2024 09:06:08 +0200 Subject: [PATCH] Set the destdir option --- pyanaconda/modules/payloads/payload/dnf/dnf_manager.py | 5 +---- .../payload/test_module_payload_dnf5_manager.py | 10 +--------- 2 files changed, 2 insertions(+), 13 deletions(-) diff --git a/pyanaconda/modules/payloads/payload/dnf/dnf_manager.py b/pyanaconda/modules/payloads/payload/dnf/dnf_manager.py index f456171348e..b10dc9db2b2 100644 --- a/pyanaconda/modules/payloads/payload/dnf/dnf_manager.py +++ b/pyanaconda/modules/payloads/payload/dnf/dnf_manager.py @@ -634,10 +634,7 @@ def set_download_location(self, path): :param path: a path to the package directory """ - # FIXME: Reimplement the assignment. - # for repo in self._base.repos.iter_enabled(): - # repo.pkgdir = path - + self._base.get_config().destdir = path self._download_location = path def download_packages(self, callback): diff --git a/tests/unit_tests/pyanaconda_tests/modules/payloads/payload/test_module_payload_dnf5_manager.py b/tests/unit_tests/pyanaconda_tests/modules/payloads/payload/test_module_payload_dnf5_manager.py index 0eacbd14c8c..dfdb34c3706 100644 --- a/tests/unit_tests/pyanaconda_tests/modules/payloads/payload/test_module_payload_dnf5_manager.py +++ b/tests/unit_tests/pyanaconda_tests/modules/payloads/payload/test_module_payload_dnf5_manager.py @@ -538,18 +538,10 @@ def _install_packages_failed(self, base, transaction, progress): """Simulate the failed installation of packages.""" progress.error("The p1 package couldn't be installed!") - @pytest.mark.skip("Not implemented") def test_set_download_location(self): """Test the set_download_location method.""" - r1 = self._add_repo("r1") - r2 = self._add_repo("r2") - r3 = self._add_repo("r3") - self.dnf_manager.set_download_location("/my/download/location") - - assert r1.pkgdir == "/my/download/location" - assert r2.pkgdir == "/my/download/location" - assert r3.pkgdir == "/my/download/location" + assert self._get_configuration().destdir == "/my/download/location" def test_download_location(self): """Test the download_location property."""