Fixes issue where nodes were not added to existing cluster (#16)

simonbs · web-flow · commit 7faa68a2d7b8 · 2022-12-18T13:01:20.000+01:00
* Fixes issue where nodes were not added to existing cluster

* Adds missing dependency
diff --git a/Package.swift b/Package.swift
@@ -45,6 +45,7 @@ let package = Package(
 
         // Sources/Library/Commands
         .target(name: "GraphCommand", dependencies: [
+            "DirectedGraphMapper",
             "DirectedGraphWriter",
             "PackageGraphBuilder",
             "PackageSwiftFileParser",
diff --git a/Sources/Library/Graphing/DirectedGraph/DirectedGraph.swift b/Sources/Library/Graphing/DirectedGraph/DirectedGraph.swift
@@ -27,7 +27,7 @@ public final class DirectedGraph: Equatable {
             for node in cluster.nodes {
                 existingCluster.addUniqueNode(node)
             }
-            return cluster
+            return existingCluster
         } else {
             clusters.append(cluster)
             return cluster

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ public final class DirectedGraph: Equatable {`
`27`	`27`	`for node in cluster.nodes {`
`28`	`28`	`existingCluster.addUniqueNode(node)`
`29`	`29`	`}`
`30`		`- return cluster`
	`30`	`+ return existingCluster`
`31`	`31`	`} else {`
`32`	`32`	`clusters.append(cluster)`
`33`	`33`	`return cluster`