Merge pull request #13 from Nexus-Mods/deconstruct-tree-nodes

Add deconstruction to FileTreeNode for cleaner tests
Nexus-Mods · Aug 29, 2023 · 723c454 · 723c454
2 parents b3eda87 + 5edda07
commit 723c454
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 4 deletions.
diff --git a/src/NexusMods.Paths/FileTree/FileTreeNode.cs b/src/NexusMods.Paths/FileTree/FileTreeNode.cs
@@ -189,6 +189,18 @@ public static FileTreeNode<TPath, TValue> CreateTree(IEnumerable<KeyValuePair<TP
         return rootNode;
     }
 
+
+    /// <summary>
+    /// Deconstructs the node into its path and value.
+    /// </summary>
+    /// <param name="path"></param>
+    /// <param name="value"></param>
+    public void Deconstruct(out TPath path, out TValue? value)
+    {
+        path = Path;
+        value = Value;
+    }
+
     /// <summary>
     /// Populates the tree with the given collection of file entries.
     /// </summary>

diff --git a/tests/NexusMods.Paths.Tests/FileTree/AbsoluteFileTreeTests.cs b/tests/NexusMods.Paths.Tests/FileTree/AbsoluteFileTreeTests.cs
@@ -28,8 +28,9 @@ public void Test_FindNode(string path, bool isUnix, bool found, int value)
         if (found)
         {
             node.Should().NotBeNull();
-            node!.Path.Should().Be(CreateAbsPath(path, isUnix));
-            node!.Value.Should().Be(value);
+            var (testPath, testValue) = node!;
+            testPath.Should().Be(CreateAbsPath(path, isUnix));
+            testValue.Should().Be(value);
         }
         else
         {

diff --git a/tests/NexusMods.Paths.Tests/FileTree/RelativeFileTreeTests.cs b/tests/NexusMods.Paths.Tests/FileTree/RelativeFileTreeTests.cs
@@ -19,8 +19,9 @@ public void Test_FindNode(string path, bool found, int value)
         if (found)
         {
             node.Should().NotBeNull();
-            node!.Path.Should().Be((RelativePath)path);
-            node!.Value.Should().Be(value);
+            var (testPath, testValue) = node!;
+            testPath.Should().Be((RelativePath)path);
+            testValue.Should().Be(value);
         }
         else
         {