From 536d040a6a20aec9f6c13e9be0e7dfc64dcc62b9 Mon Sep 17 00:00:00 2001 From: Evgenii Alekseev Date: Tue, 17 Feb 2026 01:46:58 +0200 Subject: [PATCH] feat: handle only unknownpackageerror on aur load --- .../application/application/application_repository.py | 3 ++- .../application/application/test_application_repository.py | 5 +++-- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/src/ahriman/application/application/application_repository.py b/src/ahriman/application/application/application_repository.py index 912ea3cc..8e18da31 100644 --- a/src/ahriman/application/application/application_repository.py +++ b/src/ahriman/application/application/application_repository.py @@ -22,6 +22,7 @@ from collections.abc import Iterable from ahriman.application.application.application_properties import ApplicationProperties from ahriman.application.application.workers import Updater from ahriman.core.build_tools.sources import Sources +from ahriman.core.exceptions import UnknownPackageError from ahriman.models.package import Package from ahriman.models.packagers import Packagers from ahriman.models.result import Result @@ -116,7 +117,7 @@ class ApplicationRepository(ApplicationProperties): for single in probe.packages: try: _ = Package.from_aur(single, None) - except Exception: + except UnknownPackageError: packages.append(single) return packages diff --git a/tests/ahriman/application/application/test_application_repository.py b/tests/ahriman/application/application/test_application_repository.py index de1a545d..138bb47d 100644 --- a/tests/ahriman/application/application/test_application_repository.py +++ b/tests/ahriman/application/application/test_application_repository.py @@ -5,6 +5,7 @@ from pytest_mock import MockerFixture from unittest.mock import call as MockCall from ahriman.application.application.application_repository import ApplicationRepository +from ahriman.core.exceptions import UnknownPackageError from ahriman.core.tree import Leaf, Tree from ahriman.models.changes import Changes from ahriman.models.package import Package @@ -135,7 +136,7 @@ def test_unknown_no_aur(application_repository: ApplicationRepository, package_a must return empty list in case if there is locally stored PKGBUILD """ mocker.patch("ahriman.core.repository.repository.Repository.packages", return_value=[package_ahriman]) - mocker.patch("ahriman.models.package.Package.from_aur", side_effect=Exception) + mocker.patch("ahriman.models.package.Package.from_aur", side_effect=UnknownPackageError(package_ahriman.base)) mocker.patch("ahriman.models.package.Package.from_build", return_value=package_ahriman) mocker.patch("pathlib.Path.is_dir", return_value=True) mocker.patch("ahriman.core.build_tools.sources.Sources.has_remotes", return_value=False) @@ -149,7 +150,7 @@ def test_unknown_no_aur_no_local(application_repository: ApplicationRepository, must return list of packages missing in aur and in local storage """ mocker.patch("ahriman.core.repository.repository.Repository.packages", return_value=[package_ahriman]) - mocker.patch("ahriman.models.package.Package.from_aur", side_effect=Exception) + mocker.patch("ahriman.models.package.Package.from_aur", side_effect=UnknownPackageError(package_ahriman.base)) mocker.patch("pathlib.Path.is_dir", return_value=False) packages = application_repository.unknown()