Merge pull request #793 from storybuilder-org/StoryNodeItemLogging

Story node item logging

StoryCADLib/ViewModels/StoryNodeItem.cs

@@ -328,7 +328,7 @@ public StoryNodeItem(ILogService logger, StoryElement node, StoryNodeItem parent
 
     // Overloaded constructor without logger
     public StoryNodeItem(StoryElement node, StoryNodeItem parent,
-        StoryItemType type = StoryItemType.Unknown) : this(null, node, parent, type) 
+        StoryItemType type = StoryItemType.Unknown) : this(Ioc.Default.GetRequiredService<ILogService>(), node, parent, type)
     {
     }
 
@@ -408,13 +408,13 @@ public StoryNodeItem(ILogService logger, StoryNodeItem parent, IXmlNode node)
     }
 
     // Overloaded constructor without logger
-    public StoryNodeItem(StoryNodeItem parent, IXmlNode node) : this(null, parent, node)
+    public StoryNodeItem(StoryNodeItem parent, IXmlNode node) : this(Ioc.Default.GetRequiredService<ILogService>(), parent, node)
     {
     }
 
     #endregion
 
-        public void Delete(StoryViewType storyView)
+	public void Delete(StoryViewType storyView)
     {
         _logger.Log(LogLevel.Trace, $"Starting to delete element {Name} ({Uuid}) from {storyView}");
         //Sanity check

StoryCADTests/NodeItemTests.cs

@@ -0,0 +1,27 @@
+using CommunityToolkit.Mvvm.DependencyInjection;
+using Microsoft.VisualStudio.TestTools.UnitTesting;
+using StoryCAD.Models;
+using StoryCAD.ViewModels;
+
+namespace StoryCADTests;
+
+[TestClass]
+class NodeItemTests
+{
+	/// <summary>
+	/// Creates a new node and tries to delete it.
+	/// </summary>
+	[TestMethod]
+	public void TryDelete()
+	{
+		ShellViewModel Shell = Ioc.Default.GetRequiredService<ShellViewModel>();
+		StoryModel model = new();
+		ProblemModel Overview = new("Overview", model);
+		StoryNodeItem overview = new(null, Overview, null);
+		ProblemModel Problem = new("Test",model);
+		StoryNodeItem Prob = new(null, Problem, overview);
+
+		Shell.StoryModel = model;
+		Prob.Delete(StoryViewType.ExplorerView);
+	}
+}