add ability to run only speicifed triggers from command line

This commit also restores repo-report and repo-sync subcommands

tests/ahriman/application/handlers/test_handler_triggers.py

@@ -4,15 +4,48 @@ from pytest_mock import MockerFixture
 
 from ahriman.application.handlers import Triggers
 from ahriman.core.configuration import Configuration
+from ahriman.models.package import Package
 from ahriman.models.result import Result
 
 
+def _default_args(args: argparse.Namespace) -> argparse.Namespace:
+    """
+    default arguments for these test cases
+
+    Args:
+        args(argparse.Namespace): command line arguments fixture
+
+    Returns:
+        argparse.Namespace: generated arguments for these test cases
+    """
+    args.trigger = []
+    return args
+
+
 def test_run(args: argparse.Namespace, configuration: Configuration, mocker: MockerFixture) -> None:
     """
     must run command
     """
+    args = _default_args(args)
     mocker.patch("ahriman.models.repository_paths.RepositoryPaths.tree_create")
     application_mock = mocker.patch("ahriman.core.repository.Repository.process_triggers")
 
     Triggers.run(args, "x86_64", configuration, True, False)
     application_mock.assert_called_once_with(Result())
+
+
+def test_run_trigger(args: argparse.Namespace, configuration: Configuration, package_ahriman: Package,
+                     mocker: MockerFixture) -> None:
+    """
+    must run triggers specified by command line
+    """
+    args = _default_args(args)
+    args.trigger = ["ahriman.core.report.ReportTrigger"]
+    mocker.patch("ahriman.core.repository.Repository.packages", return_value=[package_ahriman])
+    mocker.patch("ahriman.models.repository_paths.RepositoryPaths.tree_create")
+    report_mock = mocker.patch("ahriman.core.report.ReportTrigger.run")
+    upload_mock = mocker.patch("ahriman.core.upload.UploadTrigger.run")
+
+    Triggers.run(args, "x86_64", configuration, True, False)
+    report_mock.assert_called_once_with(Result(), [package_ahriman])
+    upload_mock.assert_not_called()

tests/ahriman/application/test_ahriman.py

@@ -348,6 +348,24 @@ def test_subparsers_repo_remove_unknown_architecture(parser: argparse.ArgumentPa
     assert args.architecture == ["x86_64"]
 
 
+def test_subparsers_repo_report(parser: argparse.ArgumentParser) -> None:
+    """
+    repo-report command must imply trigger
+    """
+    args = parser.parse_args(["repo-report"])
+    assert args.trigger == ["ahriman.core.report.ReportTrigger"]
+
+
+def test_subparsers_repo_report_architecture(parser: argparse.ArgumentParser) -> None:
+    """
+    repo-report command must correctly parse architecture list
+    """
+    args = parser.parse_args(["repo-report"])
+    assert args.architecture is None
+    args = parser.parse_args(["-a", "x86_64", "repo-report"])
+    assert args.architecture == ["x86_64"]
+
+
 def test_subparsers_repo_restore(parser: argparse.ArgumentParser) -> None:
     """
     repo-restore command must imply architecture list, lock, no-report and unsafe
@@ -436,6 +454,24 @@ def test_subparsers_repo_status_update_option_status(parser: argparse.ArgumentPa
     assert isinstance(args.status, BuildStatusEnum)
 
 
+def test_subparsers_repo_sync(parser: argparse.ArgumentParser) -> None:
+    """
+    repo-sync command must imply trigger
+    """
+    args = parser.parse_args(["repo-sync"])
+    assert args.trigger == ["ahriman.core.upload.UploadTrigger"]
+
+
+def test_subparsers_repo_sync_architecture(parser: argparse.ArgumentParser) -> None:
+    """
+    repo-sync command must correctly parse architecture list
+    """
+    args = parser.parse_args(["repo-report"])
+    assert args.architecture is None
+    args = parser.parse_args(["-a", "x86_64", "repo-report"])
+    assert args.architecture == ["x86_64"]
+
+
 def test_subparsers_repo_triggers_architecture(parser: argparse.ArgumentParser) -> None:
     """
     repo-triggers command must correctly parse architecture list

tests/ahriman/core/triggers/test_trigger_loader.py

@@ -13,7 +13,7 @@ def test_load_trigger_package(trigger_loader: TriggerLoader) -> None:
     """
     must load trigger from package
     """
-    assert trigger_loader._load_trigger("ahriman.core.report.ReportTrigger")
+    assert trigger_loader.load_trigger("ahriman.core.report.ReportTrigger")
 
 
 def test_load_trigger_package_invalid_import(trigger_loader: TriggerLoader, mocker: MockerFixture) -> None:
@@ -22,7 +22,7 @@ def test_load_trigger_package_invalid_import(trigger_loader: TriggerLoader, mock
     """
     mocker.patch("ahriman.core.triggers.trigger_loader.importlib.import_module", side_effect=ModuleNotFoundError())
     with pytest.raises(InvalidExtension):
-        trigger_loader._load_trigger("random.module")
+        trigger_loader.load_trigger("random.module")
 
 
 def test_load_trigger_package_not_trigger(trigger_loader: TriggerLoader) -> None:
@@ -30,7 +30,7 @@ def test_load_trigger_package_not_trigger(trigger_loader: TriggerLoader) -> None
     must raise InvalidExtension if imported module is not a type
     """
     with pytest.raises(InvalidExtension):
-        trigger_loader._load_trigger("ahriman.core.util.check_output")
+        trigger_loader.load_trigger("ahriman.core.util.check_output")
 
 
 def test_load_trigger_package_error_on_creation(trigger_loader: TriggerLoader, mocker: MockerFixture) -> None:
@@ -39,7 +39,7 @@ def test_load_trigger_package_error_on_creation(trigger_loader: TriggerLoader, m
     """
     mocker.patch("ahriman.core.triggers.trigger.Trigger.__init__", side_effect=Exception())
     with pytest.raises(InvalidExtension):
-        trigger_loader._load_trigger("ahriman.core.report.ReportTrigger")
+        trigger_loader.load_trigger("ahriman.core.report.ReportTrigger")
 
 
 def test_load_trigger_package_is_not_trigger(trigger_loader: TriggerLoader) -> None:
@@ -47,7 +47,7 @@ def test_load_trigger_package_is_not_trigger(trigger_loader: TriggerLoader) -> N
     must raise InvalidExtension if loaded class is not a trigger
     """
     with pytest.raises(InvalidExtension):
-        trigger_loader._load_trigger("ahriman.core.sign.gpg.GPG")
+        trigger_loader.load_trigger("ahriman.core.sign.gpg.GPG")
 
 
 def test_load_trigger_path(trigger_loader: TriggerLoader, resource_path_root: Path) -> None:
@@ -55,7 +55,7 @@ def test_load_trigger_path(trigger_loader: TriggerLoader, resource_path_root: Pa
     must load trigger from path
     """
     path = resource_path_root.parent.parent / "src" / "ahriman" / "core" / "report" / "report_trigger.py"
-    assert trigger_loader._load_trigger(f"{path}.ReportTrigger")
+    assert trigger_loader.load_trigger(f"{path}.ReportTrigger")
 
 
 def test_load_trigger_path_directory(trigger_loader: TriggerLoader, resource_path_root: Path) -> None:
@@ -64,7 +64,7 @@ def test_load_trigger_path_directory(trigger_loader: TriggerLoader, resource_pat
     """
     path = resource_path_root.parent.parent / "src" / "ahriman" / "core" / "report"
     with pytest.raises(InvalidExtension):
-        trigger_loader._load_trigger(f"{path}.ReportTrigger")
+        trigger_loader.load_trigger(f"{path}.ReportTrigger")
 
 
 def test_load_trigger_path_not_found(trigger_loader: TriggerLoader) -> None:
@@ -72,7 +72,7 @@ def test_load_trigger_path_not_found(trigger_loader: TriggerLoader) -> None:
     must raise InvalidExtension if file cannot be found
     """
     with pytest.raises(InvalidExtension):
-        trigger_loader._load_trigger("/some/random/path.py.SomeRandomModule")
+        trigger_loader.load_trigger("/some/random/path.py.SomeRandomModule")
 
 
 def test_process(trigger_loader: TriggerLoader, package_ahriman: Package, mocker: MockerFixture) -> None: