From 581401d60f53303f4023b1c7959858c7fd969e03 Mon Sep 17 00:00:00 2001 From: Evgeniy Alekseev Date: Tue, 10 Aug 2021 23:25:12 +0300 Subject: [PATCH] skip update process if no update supplied --- src/ahriman/application/application.py | 2 ++ tests/ahriman/application/test_application.py | 4 ++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/src/ahriman/application/application.py b/src/ahriman/application/application.py index a88b85b0..eb73dc37 100644 --- a/src/ahriman/application/application.py +++ b/src/ahriman/application/application.py @@ -211,6 +211,8 @@ class Application: :param updates: list of packages to update """ def process_update(paths: Iterable[Path]) -> None: + if not paths: + return # don't need to process if no update supplied updated = [Package.load(path, self.repository.pacman, self.repository.aur_url) for path in paths] self.repository.process_update(paths) self._finalize(updated) diff --git a/tests/ahriman/application/test_application.py b/tests/ahriman/application/test_application.py index 3150ba17..8913ec42 100644 --- a/tests/ahriman/application/test_application.py +++ b/tests/ahriman/application/test_application.py @@ -299,5 +299,5 @@ def test_update(application: Application, package_ahriman: Package, mocker: Mock application.update([package_ahriman]) build_mock.assert_called_once() - update_mock.assert_has_calls([mock.call([]), mock.call(paths)]) - finalize_mock.assert_has_calls([mock.call([]), mock.call([package_ahriman])]) + update_mock.assert_called_with(paths) + finalize_mock.assert_called_with([package_ahriman])