Improve unsatisfied output (#31)

src/packse/view.py

@@ -64,14 +64,19 @@ def render_versions(
         for_requirement: Requirement | None = None,
     ):
         versions = packages[package].versions
+        if for_requirement:
+            versions = [
+                version
+                for version in versions
+                if for_requirement.specifier.contains(version)
+            ]
+
+            if not versions:
+                yield prefix + last + "unsatisfied: no matching version"
+                return
 
         pointers = [tee] * (len(versions) - 1) + [last]
-        satisfied = False
         for pointer, version in zip(pointers, versions):
-            if for_requirement and not for_requirement.specifier.contains(version):
-                continue
-
-            satisfied = True
             message = "satisfied by " if for_requirement else ""
             yield prefix + pointer + message + f"{package}-{version}"
 
@@ -81,9 +86,6 @@ def render_versions(
                     versions[version].requires, prefix=prefix + extension
                 )
 
-        if for_requirement and not satisfied:
-            yield prefix + last + "unsatisfied"
-
     def render_requirements(requirements: list[str], prefix: str = ""):
         pointers = [tee] * (len(requirements) - 1) + [last]
         for pointer, requirement in zip(pointers, sorted(requirements)):
@@ -99,7 +101,7 @@ def render_requirements(requirements: list[str], prefix: str = ""):
                     for_requirement=parsed_requirement,
                 )
             else:
-                yield prefix + space + last + "unsatisfied"
+                yield prefix + space + last + "unsatisfied: no versions for package"
 
     # Print the root package first
     pointer = tee if scenario.packages else last

tests/__snapshots__/test_scenarios.ambr

@@ -631,12 +631,12 @@
       requires-package-does-not-exist-e4666012
       └── root
           └── requires a
-              └── unsatisfied
+              └── unsatisfied: no versions for package
 
       requires-exact-version-does-not-exist-c640da4b
       ├── root
       │   └── requires a==2.0.0
-      │       └── unsatisfied
+      │       └── unsatisfied: no matching version
       └── a
           └── a-1.0.0
 
@@ -651,7 +651,7 @@
       requires-less-version-does-not-exist-14de847d
       ├── root
       │   └── requires a<2.0.0
-      │       └── unsatisfied
+      │       └── unsatisfied: no matching version
       └── a
           ├── a-2.0.0
           ├── a-3.0.0
@@ -664,7 +664,7 @@
       └── a
           └── a-1.0.0
               └── requires b
-                  └── unsatisfied
+                  └── unsatisfied: no versions for package
 
 
     ''',
@@ -686,7 +686,7 @@
       requires-direct-incompatible-versions-3a64108d
       ├── root
       │   ├── requires a==1.0.0
-      │   │   ├── satisfied by a-1.0.0
+      │   │   └── satisfied by a-1.0.0
       │   └── requires a==2.0.0
       │       └── satisfied by a-2.0.0
       └── a
@@ -698,7 +698,7 @@
       │   ├── requires a
       │   │   └── satisfied by a-1.0.0
       │   └── requires b==1.0.0
-      │       ├── satisfied by b-1.0.0
+      │       └── satisfied by b-1.0.0
       ├── a
       │   └── a-1.0.0
       │       └── requires b==2.0.0
@@ -716,7 +716,7 @@
       ├── a
       │   └── a-1.0.0
       │       └── requires c==1.0.0
-      │           ├── satisfied by c-1.0.0
+      │           └── satisfied by c-1.0.0
       ├── b
       │   └── b-1.0.0
       │       └── requires c==2.0.0

tests/__snapshots__/test_view.ambr

@@ -17,7 +17,7 @@
           ├── b-1.0.0
           ├── b-2.0.0
           │   └── requires c
-          │       └── unsatisfied
+          │       └── unsatisfied: no versions for package
           └── b-3.0.0