[DotNetCore] Test node builders

Refactored the dependency node tests so they test the node builders
themselves. Previously the data object classes were being tested.
This is in preparation for new tests for multi-target projects.

1 files changed, 12 insertions, 4 deletions

diff --git a/main/src/addins/MonoDevelop.DotNetCore/MonoDevelop.DotNetCore.NodeBuilders/DependenciesNodeBuilder.cs b/main/src/addins/MonoDevelop.DotNetCore/MonoDevelop.DotNetCore.NodeBuilders/DependenciesNodeBuilder.cs
index 107c754e6b..c1f2df9a23 100644
--- a/main/src/addins/MonoDevelop.DotNetCore/MonoDevelop.DotNetCore.NodeBuilders/DependenciesNodeBuilder.cs
+++ b/main/src/addins/MonoDevelop.DotNetCore/MonoDevelop.DotNetCore.NodeBuilders/DependenciesNodeBuilder.cs
@@ -68,21 +68,29 @@ namespace MonoDevelop.DotNetCore.NodeBuilders
 		{
 			var node = (DependenciesNode)dataObject;
 			node.PackageDependencyCache.Refresh ();
+		}
 
+		internal void BuildChildNodes (ITreeBuilder treeBuilder, DependenciesNode node)
+		{
 			var packagesNode = new PackageDependenciesNode (node);
 			if (packagesNode.HasChildNodes ())
-				treeBuilder.AddChild (packagesNode);
+				AddChild (treeBuilder, packagesNode);
 
 			var sdkNode = new SdkDependenciesNode (node);
-			treeBuilder.AddChild (sdkNode);
+			AddChild (treeBuilder, sdkNode);
 
 			var assembliesNode = new AssemblyDependenciesNode (node.Project);
 			if (assembliesNode.HasChildNodes ())
-				treeBuilder.AddChild (assembliesNode);
+				AddChild (treeBuilder, assembliesNode);
 
 			var projectsNode = new ProjectDependenciesNode (node.Project);
 			if (projectsNode.HasChildNodes ())
-				treeBuilder.AddChild (projectsNode);
+				AddChild (treeBuilder, projectsNode);
+		}
+
+		protected virtual void AddChild (ITreeBuilder treeBuilder, object dataObject)
+		{
+			treeBuilder.AddChild (dataObject);
 		}
 	}
 }