adds a unit test showing that the SimilarNodeName is broken.

src/Umbraco.Tests/Persistence/Repositories/SimilarNodeNameTests.cs

@@ -36,6 +36,8 @@ namespace Umbraco.Tests.Persistence.Repositories
                 Assert.IsTrue(result > 0, "Expected >0 but was " + result);
         }
 
+        
+
         [Test]
         public void OrderByTest()
         {
@@ -96,5 +98,21 @@ namespace Umbraco.Tests.Persistence.Repositories
 
             Assert.AreEqual(expected, SimilarNodeName.GetUniqueName(names, nodeId, nodeName));
         }
+
+        [Test]
+        public void TestMany()
+        {
+            var names = new[]
+            {
+                new SimilarNodeName { Id = 1, Name = "Alpha (2)" },
+                new SimilarNodeName { Id = 2, Name = "Test" },
+                new SimilarNodeName { Id = 3, Name = "Test (1)" },
+                new SimilarNodeName { Id = 4, Name = "Test (2)" },
+                new SimilarNodeName { Id = 22, Name = "Test (1) (1)" },
+            };
+
+            //fixme - this will yield "Test (2)" which is already in use
+            Assert.AreEqual("Test (3)", SimilarNodeName.GetUniqueName(names, 0, "Test"));
+        }
     }
 }