Only include cyclic dependencies in error

1 files changed, 22 insertions, 0 deletions

diff --git a/src/test/java/org/shedlang/compiler/typechecker/NamespaceDependencyLineariserTests.java b/src/test/java/org/shedlang/compiler/typechecker/NamespaceDependencyLineariserTests.java
index 0a8e72bc..fba6d0e4 100644
--- a/src/test/java/org/shedlang/compiler/typechecker/NamespaceDependencyLineariserTests.java
+++ b/src/test/java/org/shedlang/compiler/typechecker/NamespaceDependencyLineariserTests.java
@@ -136,6 +136,28 @@ public class NamespaceDependencyLineariserTests {
         ));
     }
 
+    @Test
+    public void cycleOfDependenciesDoesNotIncludeParentDependenciesOutsideOfCycle() {
+        var namespaces = List.of(
+            namespaceNode("Z", List.of("A")),
+            namespaceNode("A", List.of("B")),
+            namespaceNode("B", List.of("C")),
+            namespaceNode("C", List.of("A"))
+        );
+
+        var error = assertThrows(
+            CircularNamespaceDependencyError.class,
+            () -> NamespaceDependencyLineariser.orderNamespaces(namespaces)
+        );
+
+        assertThat(error.namespaceNames(), isSequence(
+            equalTo(NamespaceName.of("A")),
+            equalTo(NamespaceName.of("B")),
+            equalTo(NamespaceName.of("C")),
+            equalTo(NamespaceName.of("A"))
+        ));
+    }
+
     private static UntypedNamespaceNode namespaceNode(
         String namespaceName,
         List<String> dependencies