input symbol nodes should have default names

DynamoDS · May 6, 2024 · 6564bd0 · 6564bd0
1 parent 9257a46
commit 6564bd0
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/src/DynamoCore/Graph/Nodes/CustomNodes/Function.cs b/src/DynamoCore/Graph/Nodes/CustomNodes/Function.cs
@@ -388,8 +388,8 @@ public Symbol()
             RegisterAllPorts();
 
             ArgumentLacing = LacingStrategy.Disabled;
-
-            InputSymbol = String.Empty;
+            //TODO localize
+            InputSymbol = new TypedParameter($"DefaultInputName", "var",-1,null).ToCommentNameString();
 
             ElementResolver = new ElementResolver();
         }
@@ -463,7 +463,7 @@ public string InputSymbol
                 }
 
                 OnNodeModified();
-                RaisePropertyChanged("InputSymbol");
+                RaisePropertyChanged(nameof(InputSymbol));
             }
         }
 

diff --git a/test/DynamoCoreTests/CustomNodes.cs b/test/DynamoCoreTests/CustomNodes.cs
@@ -1423,5 +1423,11 @@ public void LooseCustomNodeShouldNotHavePackageInfoOrPackageMember()
             Assert.IsFalse(nodeInfo.Value.IsPackageMember);
             Assert.IsNull(nodeInfo.Value.PackageInfo);
         }
+        [Test]
+        public void InputNodeShouldHaveNameByDefault()
+        {
+            var input = new Symbol();
+            Assert.NotNull(input.Parameter.Name);
+        }
     }
 }