Release 0.18.0

also satisfy pylint with too big method with too much variables

package/archlinux/PKGBUILD

@@ -1,7 +1,7 @@
 # Maintainer: Evgeniy Alekseev
 
 pkgname='ahriman'
-pkgver=0.17.0
+pkgver=0.18.0
 pkgrel=1
 pkgdesc="ArcHlinux ReposItory MANager"
 arch=('any')
@@ -23,7 +23,7 @@ optdepends=('aws-cli: sync to s3'
 source=("https://github.com/arcan1s/ahriman/releases/download/$pkgver/$pkgname-$pkgver-src.tar.xz"
         'ahriman.sysusers'
         'ahriman.tmpfiles')
-sha512sums=('0e29a7a075cbca0259a2f6374a5d1d09285112f6221d62b1c24b56ad23807133001d56ed045307866f9c7cde772f54adda234bdfcaaa272ac7d363c7da6d1f9f'
+sha512sums=('8acc57f937d587ca665c29092cadddbaf3ba0b80e870b80d1551e283aba8f21306f9030a26fec8c71ab5863316f5f5f061b7ddc63cdff9e6d5a885f28ef1893d'
             '13718afec2c6786a18f0b223ef8e58dccf0688bca4cdbe203f14071f5031ed20120eb0ce38b52c76cfd6e8b6581a9c9eaa2743eb11abbaca637451a84c33f075'
             '55b20f6da3d66e7bbf2add5d95a3b60632df121717d25a993e56e737d14f51fe063eb6f1b38bd81cc32e05db01c0c1d80aaa720c45cde87f238d8b46cdb8cbc4')
 backup=('etc/ahriman.ini'

src/ahriman/application/ahriman.py

@@ -23,8 +23,14 @@ import sys
 import ahriman.application.handlers as handlers
 import ahriman.version as version
 
+from ahriman.models.build_status import BuildStatusEnum
+
+
+# pylint thinks it is bad idea, but get the fuck off
+# pylint: disable=protected-access
+SubParserAction = argparse._SubParsersAction
+
 
-# pylint: disable=too-many-statements
 def _parser() -> argparse.ArgumentParser:
     """
     command line parser generator
@@ -43,73 +49,207 @@ def _parser() -> argparse.ArgumentParser:
 
     subparsers = parser.add_subparsers(title="command", help="command to run", dest="command", required=True)
 
-    add_parser = subparsers.add_parser("add", description="add package")
-    add_parser.add_argument("package", help="package base/name or archive path", nargs="+")
-    add_parser.add_argument("--without-dependencies", help="do not add dependencies", action="store_true")
-    add_parser.set_defaults(handler=handlers.Add)
+    _set_add_parser(subparsers)
+    _set_check_parser(subparsers)
+    _set_clean_parser(subparsers)
+    _set_config_parser(subparsers)
+    _set_rebuild_parser(subparsers)
+    _set_remove_parser(subparsers)
+    _set_report_parser(subparsers)
+    _set_setup_parser(subparsers)
+    _set_sign_parser(subparsers)
+    _set_status_parser(subparsers)
+    _set_status_update_parser(subparsers)
+    _set_sync_parser(subparsers)
+    _set_update_parser(subparsers)
+    _set_web_parser(subparsers)
 
-    check_parser = subparsers.add_parser("check", description="check for updates. Same as update --dry-run --no-manual")
-    check_parser.add_argument("package", help="filter check by package base", nargs="*")
-    check_parser.add_argument("--no-vcs", help="do not check VCS packages", action="store_true")
-    check_parser.set_defaults(handler=handlers.Update, no_aur=False, no_manual=True, dry_run=True)
+    return parser
 
-    clean_parser = subparsers.add_parser("clean", description="clear all local caches")
-    clean_parser.add_argument("--no-build", help="do not clear directory with package sources", action="store_true")
-    clean_parser.add_argument("--no-cache", help="do not clear directory with package caches", action="store_true")
-    clean_parser.add_argument("--no-chroot", help="do not clear build chroot", action="store_true")
-    clean_parser.add_argument("--no-manual", help="do not clear directory with manually added packages",
-                              action="store_true")
-    clean_parser.add_argument("--no-packages", help="do not clear directory with built packages", action="store_true")
-    clean_parser.set_defaults(handler=handlers.Clean, unsafe=True)
 
-    config_parser = subparsers.add_parser("config", description="dump configuration for specified architecture")
-    config_parser.set_defaults(handler=handlers.Dump, lock=None, no_report=True, unsafe=True)
+def _set_add_parser(root: SubParserAction) -> argparse.ArgumentParser:
+    """
+    add parser for add subcommand
+    :param root: subparsers for the commands
+    :return: created argument parser
+    """
+    parser = root.add_parser("add", description="add package")
+    parser.add_argument("package", help="package base/name or archive path", nargs="+")
+    parser.add_argument("--without-dependencies", help="do not add dependencies", action="store_true")
+    parser.set_defaults(handler=handlers.Add)
+    return parser
 
-    rebuild_parser = subparsers.add_parser("rebuild", description="rebuild whole repository")
-    rebuild_parser.set_defaults(handler=handlers.Rebuild)
 
-    remove_parser = subparsers.add_parser("remove", description="remove package")
-    remove_parser.add_argument("package", help="package name or base", nargs="+")
-    remove_parser.set_defaults(handler=handlers.Remove)
+def _set_check_parser(root: SubParserAction) -> argparse.ArgumentParser:
+    """
+    add parser for check subcommand
+    :param root: subparsers for the commands
+    :return: created argument parser
+    """
+    parser = root.add_parser("check", description="check for updates. Same as update --dry-run --no-manual")
+    parser.add_argument("package", help="filter check by package base", nargs="*")
+    parser.add_argument("--no-vcs", help="do not check VCS packages", action="store_true")
+    parser.set_defaults(handler=handlers.Update, no_aur=False, no_manual=True, dry_run=True)
+    return parser
 
-    report_parser = subparsers.add_parser("report", description="generate report")
-    report_parser.add_argument("target", help="target to generate report", nargs="*")
-    report_parser.set_defaults(handler=handlers.Report)
 
-    setup_parser = subparsers.add_parser("setup", description="create initial service configuration, requires root")
-    setup_parser.add_argument("--build-command", help="build command prefix", default="ahriman")
-    setup_parser.add_argument("--from-config", help="path to default devtools pacman configuration",
-                              default="/usr/share/devtools/pacman-extra.conf")
-    setup_parser.add_argument("--no-multilib", help="do not add multilib repository", action="store_true")
-    setup_parser.add_argument("--packager", help="packager name and email", required=True)
-    setup_parser.add_argument("--repository", help="repository name", default="aur-clone")
-    setup_parser.set_defaults(handler=handlers.Setup, lock=None, no_report=True, unsafe=True)
+def _set_clean_parser(root: SubParserAction) -> argparse.ArgumentParser:
+    """
+    add parser for clean subcommand
+    :param root: subparsers for the commands
+    :return: created argument parser
+    """
+    parser = root.add_parser("clean", description="clear all local caches")
+    parser.add_argument("--no-build", help="do not clear directory with package sources", action="store_true")
+    parser.add_argument("--no-cache", help="do not clear directory with package caches", action="store_true")
+    parser.add_argument("--no-chroot", help="do not clear build chroot", action="store_true")
+    parser.add_argument("--no-manual", help="do not clear directory with manually added packages", action="store_true")
+    parser.add_argument("--no-packages", help="do not clear directory with built packages", action="store_true")
+    parser.set_defaults(handler=handlers.Clean, unsafe=True)
+    return parser
 
-    sign_parser = subparsers.add_parser("sign", description="(re-)sign packages and repository database")
-    sign_parser.add_argument("package", help="sign only specified packages", nargs="*")
-    sign_parser.set_defaults(handler=handlers.Sign)
 
-    status_parser = subparsers.add_parser("status", description="request status of the package")
-    status_parser.add_argument("--ahriman", help="get service status itself", action="store_true")
-    status_parser.add_argument("package", help="filter status by package base", nargs="*")
-    status_parser.set_defaults(handler=handlers.Status, lock=None, no_report=True, unsafe=True)
+def _set_config_parser(root: SubParserAction) -> argparse.ArgumentParser:
+    """
+    add parser for config subcommand
+    :param root: subparsers for the commands
+    :return: created argument parser
+    """
+    parser = root.add_parser("config", description="dump configuration for specified architecture")
+    parser.set_defaults(handler=handlers.Dump, lock=None, no_report=True, unsafe=True)
+    return parser
 
-    sync_parser = subparsers.add_parser("sync", description="sync packages to remote server")
-    sync_parser.add_argument("target", help="target to sync", nargs="*")
-    sync_parser.set_defaults(handler=handlers.Sync)
 
-    update_parser = subparsers.add_parser("update", description="run updates")
-    update_parser.add_argument("package", help="filter check by package base", nargs="*")
-    update_parser.add_argument("--dry-run", help="just perform check for updates, same as check command",
-                               action="store_true")
-    update_parser.add_argument("--no-aur", help="do not check for AUR updates. Implies --no-vcs", action="store_true")
-    update_parser.add_argument("--no-manual", help="do not include manual updates", action="store_true")
-    update_parser.add_argument("--no-vcs", help="do not check VCS packages", action="store_true")
-    update_parser.set_defaults(handler=handlers.Update)
+def _set_rebuild_parser(root: SubParserAction) -> argparse.ArgumentParser:
+    """
+    add parser for rebuild subcommand
+    :param root: subparsers for the commands
+    :return: created argument parser
+    """
+    parser = root.add_parser("rebuild", description="rebuild whole repository")
+    parser.set_defaults(handler=handlers.Rebuild)
+    return parser
 
-    web_parser = subparsers.add_parser("web", description="start web server")
-    web_parser.set_defaults(handler=handlers.Web, lock=None, no_report=True)
 
+def _set_remove_parser(root: SubParserAction) -> argparse.ArgumentParser:
+    """
+    add parser for remove subcommand
+    :param root: subparsers for the commands
+    :return: created argument parser
+    """
+    parser = root.add_parser("remove", description="remove package")
+    parser.add_argument("package", help="package name or base", nargs="+")
+    parser.set_defaults(handler=handlers.Remove)
+    return parser
+
+
+def _set_report_parser(root: SubParserAction) -> argparse.ArgumentParser:
+    """
+    add parser for report subcommand
+    :param root: subparsers for the commands
+    :return: created argument parser
+    """
+    parser = root.add_parser("report", description="generate report")
+    parser.add_argument("target", help="target to generate report", nargs="*")
+    parser.set_defaults(handler=handlers.Report)
+    return parser
+
+
+def _set_setup_parser(root: SubParserAction) -> argparse.ArgumentParser:
+    """
+    add parser for setup subcommand
+    :param root: subparsers for the commands
+    :return: created argument parser
+    """
+    parser = root.add_parser("setup", description="create initial service configuration, requires root")
+    parser.add_argument("--build-command", help="build command prefix", default="ahriman")
+    parser.add_argument("--from-config", help="path to default devtools pacman configuration",
+                        default="/usr/share/devtools/pacman-extra.conf")
+    parser.add_argument("--no-multilib", help="do not add multilib repository", action="store_true")
+    parser.add_argument("--packager", help="packager name and email", required=True)
+    parser.add_argument("--repository", help="repository name", default="aur-clone")
+    parser.set_defaults(handler=handlers.Setup, lock=None, no_report=True, unsafe=True)
+    return parser
+
+
+def _set_sign_parser(root: SubParserAction) -> argparse.ArgumentParser:
+    """
+    add parser for sign subcommand
+    :param root: subparsers for the commands
+    :return: created argument parser
+    """
+    parser = root.add_parser("sign", description="(re-)sign packages and repository database")
+    parser.add_argument("package", help="sign only specified packages", nargs="*")
+    parser.set_defaults(handler=handlers.Sign)
+    return parser
+
+
+def _set_status_parser(root: SubParserAction) -> argparse.ArgumentParser:
+    """
+    add parser for status subcommand
+    :param root: subparsers for the commands
+    :return: created argument parser
+    """
+    parser = root.add_parser("status", description="request status of the package")
+    parser.add_argument("--ahriman", help="get service status itself", action="store_true")
+    parser.add_argument("package", help="filter status by package base", nargs="*")
+    parser.set_defaults(handler=handlers.Status, lock=None, no_report=True, unsafe=True)
+    return parser
+
+
+def _set_status_update_parser(root: SubParserAction) -> argparse.ArgumentParser:
+    """
+    add parser for status update subcommand
+    :param root: subparsers for the commands
+    :return: created argument parser
+    """
+    parser = root.add_parser("status-update", description="request status of the package")
+    parser.add_argument(
+        "package",
+        help="set status for specified packages. If no packages supplied, service status will be updated",
+        nargs="*")
+    parser.add_argument("--status", help="new status", choices=[value.value for value in BuildStatusEnum],
+                        default="success")
+    parser.set_defaults(handler=handlers.StatusUpdate, lock=None, no_report=True, unsafe=True)
+    return parser
+
+
+def _set_sync_parser(root: SubParserAction) -> argparse.ArgumentParser:
+    """
+    add parser for sync subcommand
+    :param root: subparsers for the commands
+    :return: created argument parser
+    """
+    parser = root.add_parser("sync", description="sync packages to remote server")
+    parser.add_argument("target", help="target to sync", nargs="*")
+    parser.set_defaults(handler=handlers.Sync)
+    return parser
+
+
+def _set_update_parser(root: SubParserAction) -> argparse.ArgumentParser:
+    """
+    add parser for update subcommand
+    :param root: subparsers for the commands
+    :return: created argument parser
+    """
+    parser = root.add_parser("update", description="run updates")
+    parser.add_argument("package", help="filter check by package base", nargs="*")
+    parser.add_argument("--dry-run", help="just perform check for updates, same as check command", action="store_true")
+    parser.add_argument("--no-aur", help="do not check for AUR updates. Implies --no-vcs", action="store_true")
+    parser.add_argument("--no-manual", help="do not include manual updates", action="store_true")
+    parser.add_argument("--no-vcs", help="do not check VCS packages", action="store_true")
+    parser.set_defaults(handler=handlers.Update)
+    return parser
+
+
+def _set_web_parser(root: SubParserAction) -> argparse.ArgumentParser:
+    """
+    add parser for web subcommand
+    :param root: subparsers for the commands
+    :return: created argument parser
+    """
+    parser = root.add_parser("web", description="start web server")
+    parser.set_defaults(handler=handlers.Web, lock=None, no_report=True)
     return parser
 
 

src/ahriman/application/handlers/__init__.py

@@ -28,6 +28,7 @@ from ahriman.application.handlers.report import Report
 from ahriman.application.handlers.setup import Setup
 from ahriman.application.handlers.sign import Sign
 from ahriman.application.handlers.status import Status
+from ahriman.application.handlers.status_update import StatusUpdate
 from ahriman.application.handlers.sync import Sync
 from ahriman.application.handlers.update import Update
 from ahriman.application.handlers.web import Web

src/ahriman/application/handlers/setup.py

@@ -124,7 +124,7 @@ class Setup(Handler):
         config.set(repository, "SigLevel", "Optional TrustAll")  # we don't care
         config.set(repository, "Server", f"file://{paths.repository}")
 
-        target = source.parent / f"pacman-{prefix}.conf"
+        target = source.parent / f"pacman-{prefix}-{architecture}.conf"
         with target.open("w") as devtools_config:
             config.write(devtools_config)
 

src/ahriman/application/handlers/status_update.py

@@ -0,0 +1,51 @@
+#
+# Copyright (c) 2021 ahriman team.
+#
+# This file is part of ahriman
+# (see https://github.com/arcan1s/ahriman).
+#
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program. If not, see <http://www.gnu.org/licenses/>.
+#
+import argparse
+
+from typing import Type
+
+from ahriman.application.application import Application
+from ahriman.application.handlers.handler import Handler
+from ahriman.core.configuration import Configuration
+from ahriman.models.build_status import BuildStatusEnum
+
+
+class StatusUpdate(Handler):
+    """
+    status update handler
+    """
+
+    @classmethod
+    def run(cls: Type[Handler], args: argparse.Namespace, architecture: str, config: Configuration) -> None:
+        """
+        callback for command line
+        :param args: command line args
+        :param architecture: repository architecture
+        :param config: configuration instance
+        """
+        client = Application(architecture, config).repository.reporter
+        status = BuildStatusEnum(args.status)
+        if args.package:
+            # update packages statuses
+            for package in args.package:
+                client.update(package, status)
+        else:
+            # update service status
+            client.update_self(status)

src/ahriman/core/report/html.py

@@ -23,7 +23,7 @@ from typing import Callable, Dict, Iterable
 
 from ahriman.core.configuration import Configuration
 from ahriman.core.report.report import Report
-from ahriman.core.util import pretty_size, pretty_datetime
+from ahriman.core.util import pretty_datetime, pretty_size
 from ahriman.models.package import Package
 from ahriman.models.sign_settings import SignSettings
 
@@ -38,7 +38,18 @@ class HTML(Report):
         link_path - prefix fo packages to download, string, required
         has_package_signed - True in case if package sign enabled, False otherwise, required
         has_repo_signed - True in case if repository database sign enabled, False otherwise, required
-        packages - sorted list of packages properties: archive_size, build_date, filename, installed_size, name, version. Required
+        packages - sorted list of packages properties, required
+                   * architecture, string
+                   * archive_size, pretty printed size, string
+                   * build_date, pretty printed datetime, string
+                   * description, string
+                   * filename, string,
+                   * groups, sorted list of strings
+                   * installed_size, pretty printed datetime, string
+                   * licenses, sorted list of strings
+                   * name, string
+                   * url, string
+                   * version, string
         pgp_key - default PGP key ID, string, optional
         repository - repository name, string, required
 
@@ -48,7 +59,7 @@ class HTML(Report):
     :ivar pgp_key: default PGP key
     :ivar report_path: output path to html report
     :ivar sign_targets: targets to sign enabled in configuration
-    :ivar tempate_path: path to directory with jinja templates
+    :ivar template_path: path to directory with jinja templates
     """
 
     def __init__(self, architecture: str, config: Configuration) -> None:
@@ -83,11 +94,16 @@ class HTML(Report):
 
         content = [
             {
+                "architecture": properties.architecture or "",
                 "archive_size": pretty_size(properties.archive_size),
                 "build_date": pretty_datetime(properties.build_date),
+                "description": properties.description or "",
                 "filename": properties.filename,
+                "groups": properties.groups,
                 "installed_size": pretty_size(properties.installed_size),
+                "licenses": properties.licenses,
                 "name": package,
+                "url": properties.url or "",
                 "version": base.version
             } for base in packages for package, properties in base.packages.items()
         ]

src/ahriman/models/package.py

@@ -59,6 +59,13 @@ class Package:
         """
         return f"{self.aur_url}/{self.base}.git"
 
+    @property
+    def groups(self) -> List[str]:
+        """
+        :return: sum of groups per each package
+        """
+        return sorted(set(sum([package.groups for package in self.packages.values()], start=[])))
+
     @property
     def is_single_package(self) -> bool:
         """
@@ -78,6 +85,13 @@ class Package:
             or self.base.endswith("-hg")\
             or self.base.endswith("-svn")
 
+    @property
+    def licenses(self) -> List[str]:
+        """
+        :return: sum of licenses per each package
+        """
+        return sorted(set(sum([package.licenses for package in self.packages.values()], start=[])))
+
     @property
     def web_url(self) -> str:
         """
@@ -95,8 +109,8 @@ class Package:
         :return: package properties
         """
         package = pacman.handle.load_pkg(str(path))
-        properties = PackageDescription(package.size, package.builddate, path.name, package.isize)
-        return cls(package.base, package.version, aur_url, {package.name: properties})
+        return cls(package.base, package.version, aur_url,
+                   {package.name: PackageDescription.from_package(package, path)})
 
     @classmethod
     def from_aur(cls: Type[Package], name: str, aur_url: str) -> Package:

src/ahriman/models/package_desciption.py

@@ -17,25 +17,38 @@
 # You should have received a copy of the GNU General Public License
 # along with this program. If not, see <http://www.gnu.org/licenses/>.
 #
-from dataclasses import dataclass
+from __future__ import annotations
+
+from dataclasses import dataclass, field
 from pathlib import Path
-from typing import Optional
+from pyalpm import Package  # type: ignore
+from typing import List, Optional, Type
 
 
 @dataclass
 class PackageDescription:
     """
     package specific properties
+    :ivar architecture: package architecture
     :ivar archive_size: package archive size
     :ivar build_date: package build date
+    :ivar description: package description
     :ivar filename: package archive name
+    :ivar groups: package groups
     :ivar installed_size: package installed size
+    :ivar licenses: package licenses list
+    :ivar url: package url
     """
 
+    architecture: Optional[str] = None
     archive_size: Optional[int] = None
     build_date: Optional[int] = None
+    description: Optional[str] = None
     filename: Optional[str] = None
+    groups: List[str] = field(default_factory=list)
     installed_size: Optional[int] = None
+    licenses: List[str] = field(default_factory=list)
+    url: Optional[str] = None
 
     @property
     def filepath(self) -> Optional[Path]:
@@ -43,3 +56,22 @@ class PackageDescription:
         :return: path object for current filename
         """
         return Path(self.filename) if self.filename is not None else None
+
+    @classmethod
+    def from_package(cls: Type[PackageDescription], package: Package, path: Path) -> PackageDescription:
+        """
+        construct class from alpm package class
+        :param package: alpm generated object
+        :param path: path to package archive
+        :return: package properties based on tarball
+        """
+        return PackageDescription(
+            architecture=package.arch,
+            archive_size=package.size,
+            build_date=package.builddate,
+            description=package.desc,
+            filename=path.name,
+            groups=package.groups,
+            installed_size=package.isize,
+            licenses=package.licenses,
+            url=package.url)

src/ahriman/version.py

@@ -17,4 +17,4 @@
 # You should have received a copy of the GNU General Public License
 # along with this program. If not, see <http://www.gnu.org/licenses/>.
 #
-__version__ = "0.17.0"
+__version__ = "0.18.0"

src/ahriman/web/views/index.py

@@ -34,10 +34,20 @@ class IndexView(BaseView):
     It uses jinja2 templates for report generation, the following variables are allowed:
 
         architecture - repository architecture, string, required
-        packages - sorted list of packages properties: base, packages (sorted list), status,
-                   timestamp, version, web_url. Required
+        packages - sorted list of packages properties, required
+                   * base, string
+                   * groups, sorted list of strings
+                   * licenses, sorted list of strings
+                   * packages, sorted list of strings
+                   * status, string based on enum value
+                   * timestamp, pretty printed datetime, string
+                   * version, string
+                   * web_url, string
         repository - repository name, string, required
-        service - service status properties: status, status_color, timestamp. Required
+        service - service status properties, required
+                   * status, string based on enum value
+                   * status_color, string based on enum value
+                   * timestamp, pretty printed datetime, string
         version - ahriman version, string, required
     """
 
@@ -51,6 +61,8 @@ class IndexView(BaseView):
         packages = [
             {
                 "base": package.base,
+                "groups": package.groups,
+                "licenses": package.licenses,
                 "packages": list(sorted(package.packages)),
                 "status": status.status.value,
                 "timestamp": pretty_datetime(status.timestamp),

tests/ahriman/application/handlers/test_handler_status_update.py

@@ -0,0 +1,40 @@
+import argparse
+
+from pytest_mock import MockerFixture
+
+from ahriman.application.handlers import StatusUpdate
+from ahriman.core.configuration import Configuration
+from ahriman.models.build_status import BuildStatusEnum
+from ahriman.models.package import Package
+
+
+def _default_args(args: argparse.Namespace) -> argparse.Namespace:
+    args.status = BuildStatusEnum.Success.value
+    args.package = None
+    return args
+
+
+def test_run(args: argparse.Namespace, configuration: Configuration, mocker: MockerFixture) -> None:
+    """
+    must run command
+    """
+    args = _default_args(args)
+    mocker.patch("pathlib.Path.mkdir")
+    update_self_mock = mocker.patch("ahriman.core.status.client.Client.update_self")
+
+    StatusUpdate.run(args, "x86_64", configuration)
+    update_self_mock.assert_called_once()
+
+
+def test_run_packages(args: argparse.Namespace, configuration: Configuration, package_ahriman: Package,
+                      mocker: MockerFixture) -> None:
+    """
+    must run command with specified packages
+    """
+    args = _default_args(args)
+    args.package = [package_ahriman.base]
+    mocker.patch("pathlib.Path.mkdir")
+    update_mock = mocker.patch("ahriman.core.status.client.Client.update")
+
+    StatusUpdate.run(args, "x86_64", configuration)
+    update_mock.assert_called_once()

tests/ahriman/application/test_ahriman.py

@@ -64,6 +64,16 @@ def test_subparsers_status(parser: argparse.ArgumentParser) -> None:
     assert args.unsafe
 
 
+def test_subparsers_status_update(parser: argparse.ArgumentParser) -> None:
+    """
+    status-update command must imply lock, no_report and unsafe
+    """
+    args = parser.parse_args(["-a", "x86_64", "status-update"])
+    assert args.lock is None
+    assert args.no_report
+    assert args.unsafe
+
+
 def test_subparsers_web(parser: argparse.ArgumentParser) -> None:
     """
     web command must imply lock and no_report

tests/ahriman/conftest.py

@@ -58,28 +58,43 @@ def package_python_schedule(
 @pytest.fixture
 def package_description_ahriman() -> PackageDescription:
     return PackageDescription(
+        architecture="x86_64",
         archive_size=4200,
         build_date=42,
+        description="ArcHlinux ReposItory MANager",
         filename="ahriman-0.12.1-1-any.pkg.tar.zst",
-        installed_size=4200000)
+        groups=[],
+        installed_size=4200000,
+        licenses=["GPL3"],
+        url="https://github.com/arcan1s/ahriman")
 
 
 @pytest.fixture
 def package_description_python_schedule() -> PackageDescription:
     return PackageDescription(
+        architecture="x86_64",
         archive_size=4201,
         build_date=421,
+        description="Python job scheduling for humans.",
         filename="python-schedule-1.0.0-2-any.pkg.tar.zst",
-        installed_size=4200001)
+        groups=[],
+        installed_size=4200001,
+        licenses=["MIT"],
+        url="https://github.com/dbader/schedule")
 
 
 @pytest.fixture
 def package_description_python2_schedule() -> PackageDescription:
     return PackageDescription(
+        architecture="x86_64",
         archive_size=4202,
         build_date=422,
+        description="Python job scheduling for humans.",
         filename="python2-schedule-1.0.0-2-any.pkg.tar.zst",
-        installed_size=4200002)
+        groups=[],
+        installed_size=4200002,
+        licenses=["MIT"],
+        url="https://github.com/dbader/schedule")
 
 
 @pytest.fixture

tests/ahriman/models/test_package.py

@@ -14,6 +14,17 @@ def test_git_url(package_ahriman: Package) -> None:
     assert package_ahriman.base in package_ahriman.git_url
 
 
+def test_groups(package_ahriman: Package) -> None:
+    """
+    must return list of groups for each package
+    """
+    assert all(
+        all(group in package_ahriman.groups for group in package.groups)
+        for package in package_ahriman.packages.values()
+    )
+    assert sorted(package_ahriman.groups) == package_ahriman.groups
+
+
 def test_is_single_package_false(package_python_schedule: Package) -> None:
     """
     python-schedule must not be single package
@@ -42,6 +53,17 @@ def test_is_vcs_true(package_tpacpi_bat_git: Package) -> None:
     assert package_tpacpi_bat_git.is_vcs
 
 
+def test_licenses(package_ahriman: Package) -> None:
+    """
+    must return list of licenses for each package
+    """
+    assert all(
+        all(lic in package_ahriman.licenses for lic in package.licenses)
+        for package in package_ahriman.packages.values()
+    )
+    assert sorted(package_ahriman.licenses) == package_ahriman.licenses
+
+
 def test_web_url(package_ahriman: Package) -> None:
     """
     must generate valid web url