Merge ppa-dev-tools:tests-dashboard into ppa-dev-tools:main

[Bryce Harrington]

I am still running a version of terminator that does not support the
ansi hyperlink functionality, and for that reason was planning on
implementing something to test the terminal for support, and make
`ppa tests` automatically format links.

However, in researching into it, I see that once I upgrade to jammy's
version of Terminator I'll have the support, and a lot of other terminal
programs also already include it now. Furthermore, I've failed to find
if there's a provided way to check for support, other than just looking
up the terminal program's version number against a known-good list.

So rather than bother with all that logic (that'll become less and less
necessary over time), I'll just follow lp-test-ppa's solution of relying
on commandline params / config values for users to manually select what
they want, and leave it at that.

Anyway, by and large this is derivative of lp-test-ppa's output
formatting, but I've added some refinements. First, I've structured
the bulleting to follow the style we use in changelog entries (I like it).
I've attempted to condense the amount of output by eliminating redundant
text, re-ordering how things are displayed, and filtering out stuff I
think users won't care about. Likely, this will all require further
refinement, but the ultimate goal is to provide a convenient
dashboard-like view.

Beyond that, this also includes a variety of random fixes and cleanups
discovered during the preparation of this branch. Hopefully nothing
that causes too much distraction. Most notable of these is a fix for
the rather rare situation we're in where the old devel release is no
longer under development, but the next development release hasn't
opened.

81e5433... by Bryce Harrington on 2022-10-25

tests: Implement the --architectures option for test result display

[Athos Ribeiro (athos-ribeiro)]

Hi Bryce! Nice work :)

This LGTM. I added a few inline comments below. As usual, feel free to ignore/act upon them as you please, since they are mostly minor nitpicks.

For large patch sets, it would be nice to have refactoring changes in their own commits.
For instance, bfd3865 is described as a test change/introduction, but also refactors a few code snippets to change a param name. Splitting this in different commits could help reviewers or any future change analysis that may be needed.

review: Approve

[Bryce Harrington (bryce)]

Noted on the refactoring change. I often squash things together in hopes it makes stuff easier to review, so it's good to know when it doesn't.

With bfd3865, it wasn't just a parameter name change, the type also changed from a string to a list. Normally I would have just pluralized series but since it's the same word singular or plural that would just be confusing, thus the name change. In hindsight, probably better to rename series->release and then to releases.

Thanks again!

diff --git a/NEWS.md b/NEWS.md
index d134499..533c92f 100644
--- a/NEWS.md
+++ b/NEWS.md
@@ -1,3 +1,10 @@
+# Unreleased #
+
+Autopkgtest action URLs are printed for packages in the PPA when running
+the `ppa tests` command.  These can be loaded in a web browser by
+someone with core-dev permissions to start the test runs.  `ppa tests`
+can then be re-run to check on the tests status and results.
+
 # 0.2.1 Release #
 
 This corrects some packaging issues when generating .deb and .snap
diff --git a/README.md b/README.md
index d26e1a4..99f10ee 100644
--- a/README.md
+++ b/README.md
@@ -57,3 +57,27 @@ Delete the PPA
 ```
 $ ppa destroy ppa:my-name/my-ppa
 ```
+
+Auto-linked Autopkgtest Trigger URLS
+------------------------------------
+
+By default, `ppa tests` will display autopkgtest triggers as hyperlinked
+text.  The hyperlinking feature is supported on many newer terminal
+programs but as it's a relatively recent VTE function it may not yet be
+available in the terminal you use.  The following terminal programs are
+believed to support it:
+
+  - iTerm2 3.1
+  - DomTerm 1.0.2
+  - hTerm 1.76
+  - Terminology 1.3?
+  - Gnome Terminal 3.26 (VTE 0.50)
+  - Guake 3.2.1 (VTE 0.50)
+  - TOXTerm 3.5.1 (VTE 0.50)
+  - Tilix 3.26 (VTE 0.50)
+  - Terminator 2.0
+
+This is not a comprehensive list, and likely will lengthen swiftly.
+Meanwhile, if you have a non-supporting browser, the --show-urls option
+can be passed to `ppa tests` to make it display the raw URLs that can be
+manually cut and pasted into your web browser.
diff --git a/ppa/debug.py b/ppa/debug.py
index 009ead4..e6fac49 100644
--- a/ppa/debug.py
+++ b/ppa/debug.py
@@ -10,25 +10,31 @@
 
 import sys
 import pprint
+import textwrap
 
 DEBUGGING = False
 
 
-def dbg(msg):
+def dbg(msg, wrap=0, prefix=None, indent=''):
     """Prints information if debugging is enabled"""
     if DEBUGGING:
         if type(msg) is str:
-            sys.stderr.write("{}\n".format(msg))
+            if wrap == 0 and indent != '':
+                wrap = 72
+            if wrap > 0:
+                if prefix is None and len(indent)>0:
+                    prefix = indent
+                msg = textwrap.fill(msg, width=wrap, initial_indent=prefix, subsequent_indent=indent)
+            sys.stderr.write(f"{msg}\n")
         else:
             pprint.pprint(msg)
 
 
 def warn(msg):
-    """Prints message to stderr"""
-    sys.stderr.write("Warning: {}\n".format(msg))
+    """Prints warning message to stderr."""
+    sys.stderr.write(f"Warning: {msg}\n")
 
 
-def die(msg, code=1):
-    """Prints message to stderr and exits with given code"""
-    sys.stderr.write("Error: {}\n".format(msg))
-    sys.exit(code)
+def error(msg):
+    """Prints error message to stderr."""
+    sys.stderr.write(f"Error: {msg}\n")
diff --git a/ppa/job.py b/ppa/job.py
index 91f015c..00dd92b 100755
--- a/ppa/job.py
+++ b/ppa/job.py
@@ -78,7 +78,7 @@ class Job:
         return f"{URL_AUTOPKGTEST}/request.cgi?{rel_str}{arch_str}{pkg_str}{trigger_str}"
 
 
-def get_running(response, series=None, ppa=None) -> Iterator[Job]:
+def get_running(response, releases=None, ppa=None) -> Iterator[Job]:
     """Returns iterator currently running autopkgtests for given criteria
 
     Filters the list of running autopkgtest jobs by the given series
@@ -88,12 +88,12 @@ def get_running(response, series=None, ppa=None) -> Iterator[Job]:
     results for that series or ppa.
 
     :param HTTPResponse response: Context manager; the response from urlopen()
-    :param str series: The Ubuntu release codename criteria, or None.
+    :param list[str] releases: The Ubuntu series codename(s), or None.
     :param str ppa: The PPA address criteria, or None.
     :rtype: Iterator[Job]
     :returns: Currently running jobs, if any, or an empty list on error
     """
-    for pkg, jobs in json.loads(response.read().decode('utf-8')).items():
+    for pkg, jobs in json.loads(response.read().decode('utf-8') or '{}').items():
         for handle in jobs:
             for codename in jobs[handle]:
                 for arch, jobinfo in jobs[handle][codename].items():
@@ -101,12 +101,12 @@ def get_running(response, series=None, ppa=None) -> Iterator[Job]:
                     ppas = jobinfo[0].get('ppas', None)
                     submit_time = jobinfo[1]
                     job = Job(0, submit_time, pkg, codename, arch, triggers, ppas)
-                    if (series and (series != job.series)) or (ppa and (ppa not in job.ppas)):
+                    if (releases and (job.series not in releases)) or (ppa and (ppa not in job.ppas)):
                         continue
                     yield job
 
 
-def get_waiting(response, series=None, ppa=None) -> Iterator[Job]:
+def get_waiting(response, releases=None, ppa=None) -> Iterator[Job]:
     """Returns iterator of queued autopkgtests for given criteria
 
     Filters the list of autopkgtest jobs waiting for execution by the
@@ -116,12 +116,12 @@ def get_waiting(response, series=None, ppa=None) -> Iterator[Job]:
     available results for that series or ppa.
 
     :param HTTPResponse response: Context manager; the response from urlopen()
-    :param str series: The Ubuntu release codename criteria, or None.
+    :param list[str] releases: The Ubuntu series codename(s), or None.
     :param str ppa: The PPA address criteria, or None.
     :rtype: Iterator[Job]
     :returns: Currently waiting jobs, if any, or an empty list on error
     """
-    for _, queue in json.loads(response.read().decode('utf-8')).items():
+    for _, queue in json.loads(response.read().decode('utf-8') or '{}').items():
         for codename in queue:
             for arch in queue[codename]:
                 n = 0
@@ -134,42 +134,42 @@ def get_waiting(response, series=None, ppa=None) -> Iterator[Job]:
                     triggers = jobinfo.get('triggers', None)
                     ppas = jobinfo.get('ppas', None)
                     job = Job(n, None, pkg, codename, arch, triggers, ppas)
-                    if (series and (series != job.series)) or (ppa and (ppa not in job.ppas)):
+                    if (releases and (job.series not in releases)) or (ppa and (ppa not in job.ppas)):
                         continue
                     yield job
 
 
 def show_running(jobs):
     """Prints the active (running and waiting) tests"""
-    rformat = "    %-8s %-40s %-8s %-8s %-40s %s"
+    rformat = "%-8s %-40s %-8s %-8s %-40s %s"
 
     n = 0
     for n, e in enumerate(jobs, start=1):
         if n == 1:
-            print("Running:")
+            print("* Running:")
             ppa_str = ','.join(e.ppas)
             trig_str = ','.join(e.triggers)
-            print(rformat % ("time", "pkg", "release", "arch", "ppa", "trigger"))
-        print(rformat % (str(e.submit_time), e.source_package, e.series, e.arch, ppa_str, trig_str))
+            print("  # " + rformat % ("time", "pkg", "release", "arch", "ppa", "trigger"))
+        print("  - " + rformat % (str(e.submit_time), e.source_package, e.series, e.arch, ppa_str, trig_str))
     if n == 0:
-        print("Running: (none)")
+        print("* Running: (none)")
 
 
 def show_waiting(jobs):
     """Prints the active (running and waiting) tests"""
-    rformat = "    %-8s %-40s %-8s %-8s %-40s %s"
+    rformat = "%-8s %-40s %-8s %-8s %-40s %s"
 
     n = 0
     for n, e in enumerate(jobs, start=1):
         if n == 1:
-            print("Waiting:")
-            print(rformat % ("Q-num", "pkg", "release", "arch", "ppa", "trigger"))
+            print("* Waiting:")
+            print("  # " + rformat % ("Q-num", "pkg", "release", "arch", "ppa", "trigger"))
 
         ppa_str = ','.join(e.ppas)
         trig_str = ','.join(e.triggers)
-        print(rformat % (e.number, e.source_package, e.series, e.arch, ppa_str, trig_str))
+        print("  - " + rformat % (e.number, e.source_package, e.series, e.arch, ppa_str, trig_str))
     if n == 0:
-        print("Waiting: (none)")
+        print("* Waiting: (none)")
 
 
 if __name__ == "__main__":
@@ -202,11 +202,11 @@ if __name__ == "__main__":
 
     print("running:")
     response = urlopen(f"file://{root_dir}/tests/data/running-20220822.json")
-    for job in get_running(response, 'kinetic', ppa):
+    for job in get_running(response, ['kinetic'], ppa):
         print(job)
     print()
 
     print("waiting:")
     response = urlopen(f"file://{root_dir}/tests/data/queues-20220822.json")
-    for job in get_waiting(response, 'kinetic', ppa):
+    for job in get_waiting(response, ['kinetic'], ppa):
         print(job)
diff --git a/ppa/ppa.py b/ppa/ppa.py
index 4140873..7b6f16f 100755
--- a/ppa/ppa.py
+++ b/ppa/ppa.py
@@ -8,6 +8,8 @@
 # Released under GNU GPLv2 or later, read the file 'LICENSE.GPLv2+' for
 # more information.
 
+"""A wrapper around a Launchpad Personal Package Archive object."""
+
 import re
 
 from textwrap import indent
@@ -32,7 +34,7 @@ class PpaDoesNotExist(BaseException):
         """Printable error message.
 
         :rtype str:
-        :return: Error message about the failure
+        :return: Error message about the failure.
         """
         if self.message:
             return self.message
@@ -40,7 +42,7 @@ class PpaDoesNotExist(BaseException):
 
 
 class Ppa:
-    """Encapsulates data needed to access and conveniently wrap a PPA
+    """Encapsulates data needed to access and conveniently wrap a PPA.
 
     This object proxies a PPA, allowing lazy initialization and caching
     of data from the remote.
@@ -88,6 +90,7 @@ class Ppa:
         return f"{self.team_name}/{self.name}"
 
     @property
+    @lru_cache
     def archive(self):
         """Retrieves the LP Archive object from the Launchpad service.
 
@@ -101,6 +104,15 @@ class Ppa:
         except NotFound:
             raise PpaDoesNotExist(self.ppa_name, self.team_name)
 
+    @lru_cache
+    def exists(self) -> bool:
+        """Returns true if the PPA exists in Launchpad."""
+        try:
+            self.archive
+            return True
+        except PpaDoesNotExist:
+            return False
+
     @property
     @lru_cache
     def address(self):
@@ -157,6 +169,7 @@ class Ppa:
         return retval and self.archive.description == description
 
     @property
+    @lru_cache
     def architectures(self):
         """Returns the architectures configured to build packages in the PPA.
 
@@ -408,7 +421,7 @@ def get_das(distro, series_name, arch_name):
 
 
 def get_ppa(lp, config):
-    """Load the specified PPA from Launchpad
+    """Load the specified PPA from Launchpad.
 
     :param Lp lp: The Launchpad wrapper object.
     :param dict config: Configuration param:value map.
diff --git a/ppa/ppa_group.py b/ppa/ppa_group.py
index ce1c0d1..622cf76 100755
--- a/ppa/ppa_group.py
+++ b/ppa/ppa_group.py
@@ -8,12 +8,14 @@
 # Released under GNU GPLv2 or later, read the file 'LICENSE.GPLv2+' for
 # more information.
 
-from .ppa import Ppa
-from .text import o2str
+"""A team or person that owns one or more PPAs in Launchpad."""
 
 from functools import lru_cache
 from lazr.restfulclient.errors import BadRequest
 
+from .ppa import Ppa
+from .text import o2str
+
 
 class PpaAlreadyExists(BaseException):
     """Exception indicating a PPA operation could not be performed."""
@@ -28,10 +30,10 @@ class PpaAlreadyExists(BaseException):
         self.message = message
 
     def __str__(self):
-        """Printable error message
+        """Printable error message.
 
         :rtype str:
-        :return: Error message about the failure
+        :return: Error message about the failure.
         """
         if self.message:
             return self.message
@@ -51,7 +53,11 @@ class PpaGroup:
         :param launchpadlib.service service: The Launchpad service object.
         :param str name: Launchpad username or team name.
         """
-        assert service is not None
+        if not service:
+            raise ValueError("undefined service.")
+        if not name:
+            raise ValueError("undefined name.")
+
         self.service = service
         self.name = name
 
@@ -116,9 +122,10 @@ class PpaGroup:
     @property
     @lru_cache
     def ppas(self):
-        """Generator to access the PPAs in this group
+        """Generator to access the PPAs in this group.
+
         :rtype: Iterator[ppa.Ppa]
-        :returns: Each PPA in the group as a ppa.Ppa object
+        :returns: Each PPA in the group as a ppa.Ppa object.
         """
         for lp_ppa in self.team.ppas:
             if '-deletedppa' in lp_ppa.name:
@@ -129,6 +136,7 @@ class PpaGroup:
     @lru_cache
     def get(self, ppa_name):
         """Provides a Ppa for the named ppa.
+
         :rtype: ppa.Ppa
         :returns: A Ppa object describing the named ppa.
         """
diff --git a/ppa/result.py b/ppa/result.py
index 012165d..db0af48 100755
--- a/ppa/result.py
+++ b/ppa/result.py
@@ -13,7 +13,7 @@
 
 import re
 import urllib.request
-from functools import lru_cache, cached_property
+from functools import lru_cache
 from typing import List, Iterator
 import gzip
 import time
@@ -65,14 +65,16 @@ class Result:
         :rtype: str
         :returns: Printable summary of the object.
         """
-        return f"{self.source} on {self.series} for {self.arch} @ {self.timestamp}"
+        pad = ' ' * (1 + abs(len('ppc64el') - len(self.arch)))
+        return f"{self.source} on {self.series} for {self.arch}{pad}@ {self.timestamp}"
 
     @property
     def timestamp(self) -> str:
         """Formats the result's completion time as a string."""
         return time.strftime("%d.%m.%y %H:%M:%S", self.time)
 
-    @cached_property
+    @property
+    @lru_cache
     def log(self) -> str:
         """Returns log contents for results, downloading if necessary.
 
@@ -103,7 +105,8 @@ class Result:
             return None
 
     # TODO: Merge triggers and get_triggers()
-    @cached_property
+    @property
+    @lru_cache
     def triggers(self) -> List[str]:
         """Returns package/version parameters used to generate this Result.
 
@@ -165,7 +168,8 @@ class Result:
             subtests.append(Subtest(line))
         return subtests
 
-    @cached_property
+    @property
+    @lru_cache
     def status(self) -> str:
         """Returns overall status of all subtests
 
@@ -187,7 +191,8 @@ class Result:
                 return 'FAIL'
         return 'PASS'
 
-    @cached_property
+    @property
+    @lru_cache
     def status_icon(self) -> str:
         """Unicode symbol corresponding to test's overall status.
 
diff --git a/ppa/subtest.py b/ppa/subtest.py
index 2f18190..8d871c4 100755
--- a/ppa/subtest.py
+++ b/ppa/subtest.py
@@ -11,7 +11,7 @@
 
 """An individual DEP8 test run"""
 
-from functools import cached_property
+from functools import lru_cache
 
 
 class Subtest:
@@ -55,7 +55,8 @@ class Subtest:
         """
         return f"{self.desc:25} {self.status:6} {self.status_icon}"
 
-    @cached_property
+    @property
+    @lru_cache
     def desc(self) -> str:
         """The descriptive text for the given subtest.
 
@@ -64,7 +65,8 @@ class Subtest:
         """
         return next(iter(self._line.split()), '')
 
-    @cached_property
+    @property
+    @lru_cache
     def status(self) -> str:
         """The success or failure of the given subtest.
 
@@ -76,7 +78,8 @@ class Subtest:
                 return k
         return 'UNKNOWN'
 
-    @cached_property
+    @property
+    @lru_cache
     def status_icon(self) -> str:
         """Unicode symbol corresponding to subtest's status.
 
diff --git a/ppa/text.py b/ppa/text.py
index b1e1453..ebd3ec3 100644
--- a/ppa/text.py
+++ b/ppa/text.py
@@ -69,8 +69,8 @@ def o2str(obj):
 
 @lru_cache
 def to_bool(value):
-    """
-    Converts 'something' to boolean. Raises exception for invalid formats
+    """Converts 'something' to boolean. Raises exception for invalid formats.
+
     Possible True  values: 1, True, '1', 'TRue', 'yes', 'y', 't'
     Possible False values: 0, False, None, [], {}, '', '0', 'faLse', 'no', 'n', 'f', 0.0
     """
@@ -116,31 +116,16 @@ def o2float(value):
         raise
 
 
+def ansi_hyperlink(url, text):
+    """Formats text into a hyperlink using ANSI escape codes.
+
+    :param str url: The linked action to load in a web browser.
+    :param str text: The visible text to show.
+    :rtype: str
+    :returns: ANSI escape code sequence to display the hyperlink.
+    """
+    return f"\u001b]8;;{url}\u001b\\{text}\u001b]8;;\u001b\\"
+
+
 if __name__ == "__main__":
-    test_cases = [
-        ('true', True),
-        ('t', True),
-        ('yes', True),
-        ('y', True),
-        ('1', True),
-        ('false', False),
-        ('f', False),
-        ('no', False),
-        ('n', False),
-        ('0', False),
-        ('', False),
-        (1, True),
-        (0, False),
-        (1.0, True),
-        (0.0, False),
-        ([], False),
-        ({}, False),
-        ((), False),
-        ([1], True),
-        ({1: 2}, True),
-        ((1,), True),
-        (None, False),
-        (object(), True),
-    ]
-    for test, expected in test_cases:
-        assert to_bool(test) == expected, f"to_bool({test}) failed to return {expected}"
+    print(ansi_hyperlink("https://launchpad.net/ppa-dev-tools", "ppa-dev-tools"))
diff --git a/ppa/trigger.py b/ppa/trigger.py
index ae00b44..72f8ba6 100755
--- a/ppa/trigger.py
+++ b/ppa/trigger.py
@@ -11,7 +11,7 @@
 
 """A directive to run a DEP8 test against a source package"""
 
-from functools import cached_property
+from functools import lru_cache
 from urllib.parse import urlencode
 
 from .constants import URL_AUTOPKGTEST
@@ -60,7 +60,8 @@ class Trigger:
         """
         return f"{self.package}/{self.version}"
 
-    @cached_property
+    @property
+    @lru_cache
     def history_url(self) -> str:
         """Renders the trigger as a URL to the job history.
 
@@ -74,7 +75,8 @@ class Trigger:
         pkg_str = f"{prefix}/{self.package}"
         return f"{URL_AUTOPKGTEST}/packages/{pkg_str}/{self.series}/{self.arch}"
 
-    @cached_property
+    @property
+    @lru_cache
     def action_url(self) -> str:
         """Renders the trigger as a URL to start running the test.
 
diff --git a/scripts/ppa b/scripts/ppa
index 04abd4c..09ba1e7 100755
--- a/scripts/ppa
+++ b/scripts/ppa
@@ -51,7 +51,7 @@ import sys
 import time
 import argparse
 from inspect import currentframe
-from distro_info import UbuntuDistroInfo
+from distro_info import UbuntuDistroInfo, DistroDataOutdated
 
 try:
     from ruamel import yaml
@@ -88,11 +88,11 @@ from ppa.result import (
     Result,
     get_results
 )
-from ppa.text import o2str
+from ppa.text import o2str, ansi_hyperlink
 from ppa.trigger import Trigger
 
 import ppa.debug
-from ppa.debug import dbg, warn
+from ppa.debug import dbg, warn, error
 
 
 def UNIMPLEMENTED():
@@ -192,6 +192,18 @@ def create_arg_parser():
     tests_parser.add_argument('ppa_name', metavar='ppa-name',
                               action='store',
                               help="Name of the PPA to view tests")
+    tests_parser.add_argument('-a', '--arches', '--arch', '--architectures',
+                              dest="architectures", action='store',
+                              help="Comma-separated list of hardware architectures to include in triggers")
+    tests_parser.add_argument('-r', '--releases', '--release',
+                              dest="releases", action='store',
+                              help="Comma-separated list of Ubuntu release codenames to show triggers for")
+    tests_parser.add_argument('-p', '--packages', '--package',
+                              dest="packages", action='store',
+                              help="Comma-separated list of source package names to show triggers for")
+    tests_parser.add_argument('-L', '--show-urls',
+                              dest='show_urls', action='store_true',
+                              help="Display unformatted trigger action URLs")
 
     # Wait Command
     wait_parser = subparser.add_parser('wait', help='wait help')
@@ -260,6 +272,13 @@ def create_config(lp, args):
             else:
                 warn(f"Invalid architectures '{args.architectures}'")
                 return None
+    elif args.command == 'tests':
+        if args.releases is not None:
+            if args.releases:
+                config['releases'] = args.releases.split(',')
+            else:
+                warn(f"Invalid releases '{args.releases}'")
+                return None
 
     return config
 
@@ -550,49 +569,88 @@ def command_tests(lp, config):
     if not lp:
         return 1
 
-    # Show tests only from the current development release
-    udi = UbuntuDistroInfo()
-    release = udi.devel()
+    releases = config.get('releases', None)
+    if releases is None:
+        udi = UbuntuDistroInfo()
+        try:
+            # Show tests only from the current development release
+            releases = [ udi.devel() ]
+        except DistroDataOutdated as e:
+            # If no development release defined, use the current active release
+            warn(f"Devel release is undefined; assuming stable release instead.")
+            dbg(f"({e})", wrap=72, prefix='  - ', indent='    ')
+            releases = [ udi.stable() ]
+
+    packages = config.get('packages', None)
+
+    ppa = get_ppa(lp, config)
+    if not ppa.exists():
+        error(f"PPA {ppa.name} does not exist for user {ppa.team_name}")
+        return 1
+
+    architectures = config.get('architectures', ARCHES_AUTOPKGTEST)
 
     try:
-        ppa = get_ppa(lp, config)
+        # Triggers
+        print("* Triggers:")
         for source_pub in ppa.get_source_publications():
-            # Triggers
-            for arch in ARCHES_AUTOPKGTEST:
-                trigger = Trigger(
-                    package=source_pub.source_package_name,
-                    version=source_pub.source_package_version,
-                    arch=arch,
-                    series=source_pub.distro_series.name,
-                    ppa=ppa)
-                print(trigger.action_url)
+            series = source_pub.distro_series.name
+            if series not in releases:
+                continue
+            pkg = source_pub.source_package_name
+            if packages and (pkg not in packages):
+                continue
+            ver = source_pub.source_package_version
+            url = f"https://launchpad.net/ubuntu/+source/{pkg}/{ver}"
+            source_hyperlink = ansi_hyperlink(url, f"{pkg}/{ver}")
+            print(f"  - Source {source_hyperlink}: {source_pub.status}")
+            triggers = [Trigger(pkg, ver, arch, series, ppa) for arch in architectures]
+
+            if config.get("show_urls"):
+                for trigger in triggers:
+                    print(f"    + {trigger.arch}: {trigger.action_url}♻️ ")
+                for trigger in triggers:
+                    print(f"    + {trigger.arch}: {trigger.action_url}💍")
+
+            else:
+                for trigger in triggers:
+                    pad = ' ' * (1 + abs(len('ppc64el') - len(trigger.arch)))
+                    basic_trig = ansi_hyperlink(trigger.action_url, f"Trigger basic @{trigger.arch}♻️ ")
+                    all_proposed_trig = ansi_hyperlink(trigger.action_url + "&all-proposed=1",
+                                                       f"Trigger all-proposed @{trigger.arch}💍")
+                    print(f"    + " + pad.join([basic_trig, all_proposed_trig]))
 
         # Results
-        base_results_fmt = f"{URL_AUTOPKGTEST}/results/autopkgtest-%s-%s-%s/"
-        base_results_url = base_results_fmt % (release, ppa.team_name, ppa.name)
-        url = f"{base_results_url}?format=plain"
-        response = open_url(url)
-        if response:
-            for result in get_results(response, base_results_url, arches=ARCHES_AUTOPKGTEST):
-                print(f"* {result} {result.status_icon}")
-                print(f"  - Triggers: " + ', '.join([str(r) for r in result.get_triggers()]))
-                if result.status != 'PASS':
-                    print(f"  - Status: {result.status}")
-                    print(f"  - Log: {result.url}")
-                    for subtest in result.get_subtests():
-                        print(f"  - {subtest}")
-                print()
+        print("* Results:")
+        for release in releases:
+            base_results_fmt = f"{URL_AUTOPKGTEST}/results/autopkgtest-%s-%s-%s/"
+            base_results_url = base_results_fmt % (release, ppa.team_name, ppa.name)
+            url = f"{base_results_url}?format=plain"
+            response = open_url(url)
+            if response:
+                trigger_sets = {}
+                for result in get_results(response, base_results_url, arches=architectures):
+                    trigger = ', '.join([str(r) for r in result.get_triggers()])
+                    trigger_sets.setdefault(trigger, '')
+                    trigger_sets[trigger] += f"    + {result.status_icon} {result}\n"
+                    if result.status != 'PASS':
+                        trigger_sets[trigger] += f"      • Status: {result.status}\n"
+                        trigger_sets[trigger] += f"      • Log: {result.url}\n"
+                        for subtest in result.get_subtests():
+                            trigger_sets[trigger] += f"      • {subtest}\n"
+                for trigger, result in trigger_sets.items():
+                    print(f"  - {trigger}\n{result}")
 
         # Running Queue
         response = open_url(f"{URL_AUTOPKGTEST}/static/running.json", "running autopkgtests")
         if response:
-            show_running(sorted(get_running(response, series=release, ppa=str(ppa)),
-                               key=lambda k: str(k.submit_time)))
+            show_running(sorted(get_running(response, releases=releases, ppa=str(ppa)),
+                                key=lambda k: str(k.submit_time)))
 
         # Waiting Queue
         response = open_url(f"{URL_AUTOPKGTEST}/queues.json", "waiting autopkgtests")
         if response:
-            show_waiting(get_waiting(response, series=release, ppa=str(ppa)))
+            show_waiting(get_waiting(response, releases=releases, ppa=str(ppa)))
 
         return os.EX_OK
     except KeyboardInterrupt:
@@ -640,7 +698,7 @@ def main(args):
             return func(lp, config, param)
         return func(lp, config)
     except KeyError:
-        parser.error("No such command {}".format(args.command))
+        parser.error(f"No such command {args.command}")
         return 1
 
 
diff --git a/tests/test_job.py b/tests/test_job.py
index 78a12c7..1d1708b 100644
--- a/tests/test_job.py
+++ b/tests/test_job.py
@@ -81,7 +81,7 @@ def test_get_running():
                  '"Log Output Here"'
                  '] } } } }')
     fake_response = RequestResponseMock(json_text)
-    job = next(get_running(fake_response, series='focal', ppa='ppa:me/myppa'))
+    job = next(get_running(fake_response, releases=['focal'], ppa='ppa:me/myppa'))
     assert repr(job) == "Job(source_package='mypackage', series='focal', arch='arm64')"
     assert job.triggers == ["yourpackage/1.2.3"]
     assert job.ppas == ["ppa:me/myppa"]
@@ -100,7 +100,7 @@ def test_get_waiting():
                  '      \\"triggers\\":    [ \\"c/3.2-1\\", \\"d/2-2\\" ] }"'
                  '] } } }')
     fake_response = RequestResponseMock(json_text)
-    job = next(get_waiting(fake_response, series='focal', ppa='ppa:me/myppa'))
+    job = next(get_waiting(fake_response, releases=['focal'], ppa='ppa:me/myppa'))
     assert job
     assert job.source_package == "b"
     assert job.ppas == ['ppa:me/myppa']
diff --git a/tests/test_ppa_group.py b/tests/test_ppa_group.py
index 63907fd..f8aa8db 100644
--- a/tests/test_ppa_group.py
+++ b/tests/test_ppa_group.py
@@ -23,9 +23,14 @@ from tests.helpers import PersonMock, LaunchpadMock, LpServiceMock
 
 def test_object():
     """Checks that PpaGroup objects can be instantiated."""
-    ppa_group = PpaGroup(service=LpServiceMock(), name=None)
+    ppa_group = PpaGroup(service=LpServiceMock(), name='test-ppa')
     assert ppa_group
 
+    with pytest.raises(ValueError):
+        ppa_group = PpaGroup(service=LpServiceMock(), name=None)
+    with pytest.raises(ValueError):
+        ppa_group = PpaGroup(service=None, name='test-ppa')
+
 
 def test_create_ppa():
     """Checks that PpaGroups can create PPAs."""
diff --git a/tests/test_result.py b/tests/test_result.py
index 8c498a4..d928efb 100644
--- a/tests/test_result.py
+++ b/tests/test_result.py
@@ -41,7 +41,7 @@ def test_str():
     """Checks Result object textual presentation."""
     timestamp = time.strptime('20030201_040506', "%Y%m%d_%H%M%S")
     result = Result('url', timestamp, 'b', 'c', 'd')
-    assert f"{result}" == 'd on b for c @ 01.02.03 04:05:06'
+    assert f"{result}" == 'd on b for c       @ 01.02.03 04:05:06'
 
 
 def test_timestamp():
diff --git a/tests/test_scripts_ppa.py b/tests/test_scripts_ppa.py
index 812b4b5..51a0755 100644
--- a/tests/test_scripts_ppa.py
+++ b/tests/test_scripts_ppa.py
@@ -112,7 +112,7 @@ def test_create_arg_parser():
     args.debug = None
 
     # Check -q, --dry-run
-    args = parser.parse_args(['cmd', '--dry-run'])
+    args = parser.parse_args(['--dry-run', 'status', 'test-ppa'])
     assert args.dry_run == True
     args.dry_run = None
 
@@ -159,11 +159,18 @@ def test_create_arg_parser_create():
     args = parser.parse_args(['create', 'test-ppa'])
     assert args.ppa_name == 'test-ppa'
 
-    # Check -a, --arch, --arches, --architectures
+    # Check that command args can come before or after the ppa name
     args = parser.parse_args(['create', 'test-ppa', '-a', 'x'])
     assert args.architectures == 'x'
     args.architectures = None
+    args = parser.parse_args(['create', '-a', 'x', 'test-ppa'])
+    assert args.architectures == 'x'
+    args.architectures = None
 
+    # Check -a, --arch, --arches, --architectures
+    args = parser.parse_args(['create', 'test-ppa', '-a', 'x'])
+    assert args.architectures == 'x'
+    args.architectures = None
     args = parser.parse_args(['create', 'test-ppa', '--arch', 'x'])
     assert args.architectures == 'x'
     args.architectures = None
@@ -175,6 +182,72 @@ def test_create_arg_parser_create():
     args.architectures = None
 
 
+def test_create_arg_parser_tests():
+    """Checks argument parsing for the 'tests' command."""
+    parser = script.create_arg_parser()
+
+    # Check ppa_name
+    args = parser.parse_args(['tests', 'test-ppa'])
+    assert args.ppa_name == 'test-ppa'
+
+    # Check -a, --arch, --arches, --architectures
+    args = parser.parse_args(['tests', 'test-ppa', '-a', 'x'])
+    assert args.architectures == 'x'
+    args.architectures = None
+
+    args = parser.parse_args(['tests', 'test-ppa', '--arch', 'x'])
+    assert args.architectures == 'x'
+    args.architectures = None
+    args = parser.parse_args(['tests', 'test-ppa', '--arches', 'x'])
+    assert args.architectures == 'x'
+    args.architectures = None
+    args = parser.parse_args(['tests', 'test-ppa', '--architectures', 'a,b,c'])
+    assert args.architectures == 'a,b,c'
+    args.architectures = None
+
+    # Check -r, --release, --releases
+    args = parser.parse_args(['tests', 'test-ppa', '-r', 'x'])
+    assert args.releases == 'x'
+    args.releases = None
+    args = parser.parse_args(['tests', 'test-ppa', '--release', 'x'])
+    assert args.releases == 'x'
+    args.releases = None
+    args = parser.parse_args(['tests', 'test-ppa', '--releases', 'x'])
+    assert args.releases == 'x'
+    args.releases = None
+    args = parser.parse_args(['tests', 'test-ppa', '--releases', 'x,y,z'])
+    assert args.releases == 'x,y,z'
+    args.releases = None
+
+    # Check -p, --package, --packages
+    args = parser.parse_args(['tests', 'tests-ppa', '-p', 'x'])
+    assert args.packages == 'x'
+    args.packages = None
+    args = parser.parse_args(['tests', 'tests-ppa', '--package', 'x'])
+    assert args.packages == 'x'
+    args.packages = None
+    args = parser.parse_args(['tests', 'tests-ppa', '--packages', 'x'])
+    assert args.packages == 'x'
+    args.packages = None
+    args = parser.parse_args(['tests', 'tests-ppa', '--packages', 'x,y,z'])
+    assert args.packages == 'x,y,z'
+    args.packages = None
+
+    # Check --show-urls, --show-url, -L
+    args = parser.parse_args(['tests', 'tests-ppa'])
+    assert args.show_urls == False
+    args.show_urls = None
+    args = parser.parse_args(['tests', 'tests-ppa', '--show-urls'])
+    assert args.show_urls == True
+    args.show_urls = None
+    args = parser.parse_args(['tests', 'tests-ppa', '--show-url'])
+    assert args.show_urls == True
+    args.show_urls = None
+    args = parser.parse_args(['tests', 'tests-ppa', '-L'])
+    assert args.show_urls == True
+    args.show_urls = None
+
+
 @pytest.mark.xfail(reason="Unimplemented")
 def test_create_config():
     # args = []
diff --git a/tests/test_text.py b/tests/test_text.py
new file mode 100644
index 0000000..57ce979
--- /dev/null
+++ b/tests/test_text.py
@@ -0,0 +1,54 @@
+#!/usr/bin/env python3
+# -*- Mode: Python; coding: utf-8; indent-tabs-mode: nil; tab-width: 4 -*-
+
+# Author:  Bryce Harrington <[e-mail]>
+#
+# Copyright (C) 2022 Bryce W. Harrington
+#
+# Released under GNU GPLv2 or later, read the file 'LICENSE.GPLv2+' for
+# more information.
+
+import os
+import sys
+
+import pytest
+
+sys.path.insert(0, os.path.realpath(
+    os.path.join(os.path.dirname(os.path.realpath(__file__)), "..")))
+
+import ppa.text
+
[e-mail]('input, expected', [
+    ('true', True),
+    ('t', True),
+    ('yes', True),
+    ('y', True),
+    ('1', True),
+    ('false', False),
+    ('f', False),
+    ('no', False),
+    ('n', False),
+    ('0', False),
+    ('', False),
+    (1, True),
+    (0, False),
+    (1.0, True),
+    (0.0, False),
+    ((), False),
+    ((1,), True),
+    (None, False),
+    (object(), True),
+])
+def test_to_bool(input, expected):
+    """Check that the given input produces the expected true/false result.
+
+    :param * input: Any available type to be converted to boolean.
+    :param bool expected: The True or False result to expect.
+    """
+    assert ppa.text.to_bool(input) == expected
+
+
+def test_ansi_hyperlink():
+    """Check that text can be linked with a url."""
+    assert ppa.text.ansi_hyperlink("xxx", "yyy") == "\u001b]8;;xxx\u001b\\yyy\u001b]8;;\u001b\\"
+