From 49d433ab3729e4bda684f2894f98e8800d9e8bf6 Mon Sep 17 00:00:00 2001 From: Evgenii Alekseev Date: Wed, 11 Feb 2026 14:26:48 +0200 Subject: [PATCH] remove duplicate code --- src/ahriman/core/database/migrations/m016_archive.py | 3 ++- tests/ahriman/core/archive/test_archive_tree.py | 12 ------------ 2 files changed, 2 insertions(+), 13 deletions(-) diff --git a/src/ahriman/core/database/migrations/m016_archive.py b/src/ahriman/core/database/migrations/m016_archive.py index 27beb82f..376c17ce 100644 --- a/src/ahriman/core/database/migrations/m016_archive.py +++ b/src/ahriman/core/database/migrations/m016_archive.py @@ -25,6 +25,7 @@ from sqlite3 import Connection from ahriman.application.handlers.handler import Handler from ahriman.core.alpm.pacman import Pacman from ahriman.core.configuration import Configuration +from ahriman.core.utils import symlink_relative from ahriman.models.package import Package from ahriman.models.pacman_synchronization import PacmanSynchronization from ahriman.models.repository_paths import RepositoryPaths @@ -81,4 +82,4 @@ def move_packages(repository_paths: RepositoryPaths, pacman: Pacman) -> None: source.rename(target) # create symlink to the archive - source.symlink_to(target.relative_to(source.parent, walk_up=True)) + symlink_relative(source, target) diff --git a/tests/ahriman/core/archive/test_archive_tree.py b/tests/ahriman/core/archive/test_archive_tree.py index c95c816c..399ab59d 100644 --- a/tests/ahriman/core/archive/test_archive_tree.py +++ b/tests/ahriman/core/archive/test_archive_tree.py @@ -7,18 +7,6 @@ from ahriman.core.utils import utcnow from ahriman.models.package import Package -def test_symlinks_create_empty_filename(archive_tree: ArchiveTree, package_ahriman: Package, - mocker: MockerFixture) -> None: - """ - must skip symlinks creation if filename is not set - """ - package_ahriman.packages[package_ahriman.base].filename = None - symlinks_mock = mocker.patch("pathlib.Path.symlink_to") - - archive_tree.symlinks_create([package_ahriman]) - symlinks_mock.assert_not_called() - - def test_repo(archive_tree: ArchiveTree) -> None: """ must return correct repository object