add console printer

also add python-requests as explicit dependency and escape symbols in
repository name for badges in default tempate

tests/ahriman/core/formatters/conftest.py

@@ -0,0 +1,67 @@
+import pytest
+
+from ahriman.core.formatters.aur_printer import AurPrinter
+from ahriman.core.formatters.configuration_printer import ConfigurationPrinter
+from ahriman.core.formatters.package_printer import PackagePrinter
+from ahriman.core.formatters.status_printer import StatusPrinter
+from ahriman.core.formatters.string_printer import StringPrinter
+from ahriman.core.formatters.update_printer import UpdatePrinter
+from ahriman.models.aur_package import AURPackage
+from ahriman.models.build_status import BuildStatus
+from ahriman.models.package import Package
+
+
+@pytest.fixture
+def aur_package_ahriman_printer(aur_package_ahriman: AURPackage) -> AurPrinter:
+    """
+    fixture for AUR package printer
+    :param aur_package_ahriman: AUR package fixture
+    :return: AUR package printer test instance
+    """
+    return AurPrinter(aur_package_ahriman)
+
+
+@pytest.fixture
+def configuration_printer() -> ConfigurationPrinter:
+    """
+    fixture for configuration printer
+    :return: configuration printer test instance
+    """
+    return ConfigurationPrinter("section", {"key_one": "value_one", "key_two": "value_two"})
+
+
+@pytest.fixture
+def package_ahriman_printer(package_ahriman: Package) -> PackagePrinter:
+    """
+    fixture for package printer
+    :param package_ahriman: package fixture
+    :return: package printer test instance
+    """
+    return PackagePrinter(package_ahriman, BuildStatus())
+
+
+@pytest.fixture
+def status_printer() -> StatusPrinter:
+    """
+    fixture for build status printer
+    :return: build status printer test instance
+    """
+    return StatusPrinter(BuildStatus())
+
+
+@pytest.fixture
+def string_printer() -> StringPrinter:
+    """
+    fixture for any string printer
+    :return: any string printer test instance
+    """
+    return StringPrinter("hello, world")
+
+
+@pytest.fixture
+def update_printer(package_ahriman: Package) -> UpdatePrinter:
+    """
+    fixture for build status printer
+    :return: build status printer test instance
+    """
+    return UpdatePrinter(package_ahriman, None)

tests/ahriman/core/formatters/test_aur_printer.py

@@ -0,0 +1,15 @@
+from ahriman.core.formatters.aur_printer import AurPrinter
+
+
+def test_properties(aur_package_ahriman_printer: AurPrinter) -> None:
+    """
+    must return non empty properties list
+    """
+    assert aur_package_ahriman_printer.properties()
+
+
+def test_title(aur_package_ahriman_printer: AurPrinter) -> None:
+    """
+    must return non empty title
+    """
+    assert aur_package_ahriman_printer.title() is not None

tests/ahriman/core/formatters/test_build_printer.py

@@ -0,0 +1,36 @@
+import pytest
+
+from ahriman.core.formatters.build_printer import BuildPrinter
+from ahriman.models.package import Package
+
+
+def test_properties(package_ahriman: Package) -> None:
+    """
+    must return empty properties list
+    """
+    assert not BuildPrinter(package_ahriman, is_success=True, use_utf=False).properties()
+
+
+def test_sign_ascii(package_ahriman: Package) -> None:
+    """
+    must correctly generate sign in ascii
+    """
+    BuildPrinter(package_ahriman, is_success=True, use_utf=False).title().encode("ascii")
+    BuildPrinter(package_ahriman, is_success=False, use_utf=False).title().encode("ascii")
+
+
+def test_sign_utf8(package_ahriman: Package) -> None:
+    """
+    must correctly generate sign in ascii
+    """
+    with pytest.raises(UnicodeEncodeError):
+        BuildPrinter(package_ahriman, is_success=True, use_utf=True).title().encode("ascii")
+    with pytest.raises(UnicodeEncodeError):
+        BuildPrinter(package_ahriman, is_success=False, use_utf=True).title().encode("ascii")
+
+
+def test_title(package_ahriman: Package) -> None:
+    """
+    must return non empty title
+    """
+    assert BuildPrinter(package_ahriman, is_success=True, use_utf=False).title() is not None

tests/ahriman/core/formatters/test_configuration_printer.py

@@ -0,0 +1,22 @@
+from ahriman.core.formatters.configuration_printer import ConfigurationPrinter
+
+
+def test_properties(configuration_printer: ConfigurationPrinter) -> None:
+    """
+    must return non empty properties list
+    """
+    assert configuration_printer.properties()
+
+
+def test_properties_required(configuration_printer: ConfigurationPrinter) -> None:
+    """
+    must return all properties as required
+    """
+    assert all(prop.is_required for prop in configuration_printer.properties())
+
+
+def test_title(configuration_printer: ConfigurationPrinter) -> None:
+    """
+    must return non empty title
+    """
+    assert configuration_printer.title() == "[section]"

tests/ahriman/core/formatters/test_package_printer.py

@@ -0,0 +1,15 @@
+from ahriman.core.formatters.package_printer import PackagePrinter
+
+
+def test_properties(package_ahriman_printer: PackagePrinter) -> None:
+    """
+    must return non empty properties list
+    """
+    assert package_ahriman_printer.properties()
+
+
+def test_title(package_ahriman_printer: PackagePrinter) -> None:
+    """
+    must return non empty title
+    """
+    assert package_ahriman_printer.title() is not None

tests/ahriman/core/formatters/test_printer.py

@@ -0,0 +1,45 @@
+from unittest.mock import MagicMock
+
+from ahriman.core.formatters.package_printer import PackagePrinter
+from ahriman.core.formatters.printer import Printer
+
+
+def test_print(package_ahriman_printer: PackagePrinter) -> None:
+    """
+    must print content
+    """
+    log_mock = MagicMock()
+    package_ahriman_printer.print(verbose=False, log_fn=log_mock)
+    log_mock.assert_called()
+
+
+def test_print_empty() -> None:
+    """
+    must not print empty object
+    """
+    log_mock = MagicMock()
+    Printer().print(verbose=True, log_fn=log_mock)
+    log_mock.assert_not_called()
+
+
+def test_print_verbose(package_ahriman_printer: PackagePrinter) -> None:
+    """
+    must print content with increased verbosity
+    """
+    log_mock = MagicMock()
+    package_ahriman_printer.print(verbose=True, log_fn=log_mock)
+    log_mock.assert_called()
+
+
+def test_properties() -> None:
+    """
+    must return empty properties list
+    """
+    assert Printer().properties() == []
+
+
+def test_title() -> None:
+    """
+    must return empty title
+    """
+    assert Printer().title() is None

tests/ahriman/core/formatters/test_status_printer.py

@@ -0,0 +1,15 @@
+from ahriman.core.formatters.status_printer import StatusPrinter
+
+
+def test_properties(status_printer: StatusPrinter) -> None:
+    """
+    must return empty properties list
+    """
+    assert not status_printer.properties()
+
+
+def test_title(status_printer: StatusPrinter) -> None:
+    """
+    must return non empty title
+    """
+    assert status_printer.title() is not None

tests/ahriman/core/formatters/test_string_printer.py

@@ -0,0 +1,15 @@
+from ahriman.core.formatters.string_printer import StringPrinter
+
+
+def test_properties(string_printer: StringPrinter) -> None:
+    """
+    must return empty properties list
+    """
+    assert not string_printer.properties()
+
+
+def test_title(string_printer: StringPrinter) -> None:
+    """
+    must return non empty title
+    """
+    assert string_printer.title() is not None

tests/ahriman/core/formatters/test_update_printer.py

@@ -0,0 +1,15 @@
+from ahriman.core.formatters.update_printer import UpdatePrinter
+
+
+def test_properties(update_printer: UpdatePrinter) -> None:
+    """
+    must return empty properties list
+    """
+    assert update_printer.properties()
+
+
+def test_title(update_printer: UpdatePrinter) -> None:
+    """
+    must return non empty title
+    """
+    assert update_printer.title() is not None

tests/ahriman/core/report/test_console.py

@@ -0,0 +1,20 @@
+from pytest_mock import MockerFixture
+from unittest import mock
+
+from ahriman.core.configuration import Configuration
+from ahriman.core.report.console import Console
+from ahriman.models.package import Package
+from ahriman.models.result import Result
+
+
+def test_generate(configuration: Configuration, result: Result, package_python_schedule: Package,
+                  mocker: MockerFixture) -> None:
+    """
+    must print result to stdout
+    """
+    print_mock = mocker.patch("ahriman.core.formatters.printer.Printer.print")
+    result.add_failed(package_python_schedule)
+    report = Console("x86_64", configuration, "console")
+
+    report.generate([], result)
+    print_mock.assert_has_calls([mock.call(verbose=True), mock.call(verbose=True)])

tests/ahriman/core/report/test_email.py

@@ -5,6 +5,7 @@ from pytest_mock import MockerFixture
 from ahriman.core.configuration import Configuration
 from ahriman.core.report.email import Email
 from ahriman.models.package import Package
+from ahriman.models.result import Result
 
 
 def test_send(configuration: Configuration, mocker: MockerFixture) -> None:
@@ -92,24 +93,26 @@ def test_generate(configuration: Configuration, package_ahriman: Package, mocker
     send_mock = mocker.patch("ahriman.core.report.email.Email._send")
 
     report = Email("x86_64", configuration, "email")
-    report.generate([package_ahriman], [])
+    report.generate([package_ahriman], Result())
     send_mock.assert_called_once_with(pytest.helpers.anyvar(int), {})
 
 
-def test_generate_with_built(configuration: Configuration, package_ahriman: Package, mocker: MockerFixture) -> None:
+def test_generate_with_built(configuration: Configuration, package_ahriman: Package, result: Result,
+                             mocker: MockerFixture) -> None:
     """
     must generate report with built packages
     """
     send_mock = mocker.patch("ahriman.core.report.email.Email._send")
 
     report = Email("x86_64", configuration, "email")
-    report.generate([package_ahriman], [package_ahriman])
+    report.generate([package_ahriman], result)
     send_mock.assert_called_once_with(pytest.helpers.anyvar(int), {})
 
 
 def test_generate_with_built_and_full_path(
         configuration: Configuration,
         package_ahriman: Package,
+        result: Result,
         mocker: MockerFixture) -> None:
     """
     must generate report with built packages
@@ -118,7 +121,7 @@ def test_generate_with_built_and_full_path(
 
     report = Email("x86_64", configuration, "email")
     report.full_template_path = report.template_path
-    report.generate([package_ahriman], [package_ahriman])
+    report.generate([package_ahriman], result)
     send_mock.assert_called_once_with(pytest.helpers.anyvar(int), pytest.helpers.anyvar(int))
 
 
@@ -130,11 +133,11 @@ def test_generate_no_empty(configuration: Configuration, package_ahriman: Packag
     send_mock = mocker.patch("ahriman.core.report.email.Email._send")
 
     report = Email("x86_64", configuration, "email")
-    report.generate([package_ahriman], [])
+    report.generate([package_ahriman], Result())
     send_mock.assert_not_called()
 
 
-def test_generate_no_empty_with_built(configuration: Configuration, package_ahriman: Package,
+def test_generate_no_empty_with_built(configuration: Configuration, package_ahriman: Package, result: Result,
                                       mocker: MockerFixture) -> None:
     """
     must generate report with built packages if no_empty_report is set
@@ -143,5 +146,5 @@ def test_generate_no_empty_with_built(configuration: Configuration, package_ahri
     send_mock = mocker.patch("ahriman.core.report.email.Email._send")
 
     report = Email("x86_64", configuration, "email")
-    report.generate([package_ahriman], [package_ahriman])
+    report.generate([package_ahriman], result)
     send_mock.assert_called_once_with(pytest.helpers.anyvar(int), {})

tests/ahriman/core/report/test_jinja_template.py

@@ -1,6 +1,7 @@
 from ahriman.core.configuration import Configuration
 from ahriman.core.report.jinja_template import JinjaTemplate
 from ahriman.models.package import Package
+from ahriman.models.result import Result
 
 
 def test_generate(configuration: Configuration, package_ahriman: Package) -> None:
@@ -9,4 +10,4 @@ def test_generate(configuration: Configuration, package_ahriman: Package) -> Non
     """
     path = configuration.getpath("html", "template_path")
     report = JinjaTemplate("html", configuration)
-    assert report.make_html([package_ahriman], path)
+    assert report.make_html(Result(success=[package_ahriman]), path)

tests/ahriman/core/report/test_report.py

@@ -6,6 +6,7 @@ from ahriman.core.configuration import Configuration
 from ahriman.core.exceptions import ReportFailed
 from ahriman.core.report.report import Report
 from ahriman.models.report_settings import ReportSettings
+from ahriman.models.result import Result
 
 
 def test_report_failure(configuration: Configuration, mocker: MockerFixture) -> None:
@@ -14,32 +15,41 @@ def test_report_failure(configuration: Configuration, mocker: MockerFixture) ->
     """
     mocker.patch("ahriman.core.report.html.HTML.generate", side_effect=Exception())
     with pytest.raises(ReportFailed):
-        Report.load("x86_64", configuration, "html").run([], [])
+        Report.load("x86_64", configuration, "html").run([], Result())
 
 
-def test_report_dummy(configuration: Configuration, mocker: MockerFixture) -> None:
+def test_report_dummy(configuration: Configuration, result: Result, mocker: MockerFixture) -> None:
     """
     must construct dummy report class
     """
     mocker.patch("ahriman.models.report_settings.ReportSettings.from_option", return_value=ReportSettings.Disabled)
     report_mock = mocker.patch("ahriman.core.report.report.Report.generate")
-    Report.load("x86_64", configuration, "disabled").run([], [])
-    report_mock.assert_called_once_with([], [])
+    Report.load("x86_64", configuration, "disabled").run([], result)
+    report_mock.assert_called_once_with([], result)
 
 
-def test_report_email(configuration: Configuration, mocker: MockerFixture) -> None:
+def test_report_console(configuration: Configuration, result: Result, mocker: MockerFixture) -> None:
+    """
+    must generate console report
+    """
+    report_mock = mocker.patch("ahriman.core.report.console.Console.generate")
+    Report.load("x86_64", configuration, "console").run([], result)
+    report_mock.assert_called_once_with([], result)
+
+
+def test_report_email(configuration: Configuration, result: Result, mocker: MockerFixture) -> None:
     """
     must generate email report
     """
     report_mock = mocker.patch("ahriman.core.report.email.Email.generate")
-    Report.load("x86_64", configuration, "email").run([], [])
-    report_mock.assert_called_once_with([], [])
+    Report.load("x86_64", configuration, "email").run([], result)
+    report_mock.assert_called_once_with([], result)
 
 
-def test_report_html(configuration: Configuration, mocker: MockerFixture) -> None:
+def test_report_html(configuration: Configuration, result: Result, mocker: MockerFixture) -> None:
     """
     must generate html report
     """
     report_mock = mocker.patch("ahriman.core.report.html.HTML.generate")
-    Report.load("x86_64", configuration, "html").run([], [])
-    report_mock.assert_called_once_with([], [])
+    Report.load("x86_64", configuration, "html").run([], result)
+    report_mock.assert_called_once_with([], result)

tests/ahriman/core/repository/test_executor.py

@@ -35,7 +35,6 @@ def test_process_build(executor: Executor, package_ahriman: Package, mocker: Moc
     mocker.patch("ahriman.core.build_tools.task.Task.init")
     move_mock = mocker.patch("shutil.move")
     status_client_mock = mocker.patch("ahriman.core.status.client.Client.set_building")
-    built_packages_mock = mocker.patch("ahriman.core.repository.executor.Executor.packages_built")
 
     executor.process_build([package_ahriman])
     # must move files (once)
@@ -45,8 +44,6 @@ def test_process_build(executor: Executor, package_ahriman: Package, mocker: Moc
     # must clear directory
     from ahriman.core.repository.cleaner import Cleaner
     Cleaner.clear_build.assert_called_once_with()
-    # must return build packages after all
-    built_packages_mock.assert_called_once_with()
 
 
 def test_process_build_failure(executor: Executor, package_ahriman: Package, mocker: MockerFixture) -> None: