From 9f6e6e4701bf91561e68104e7f28a00a9f777728 Mon Sep 17 00:00:00 2001 From: Mike Date: Wed, 10 Jan 2018 20:58:10 +0000 Subject: [PATCH] Add tests to SimilarNodeNameTests to check for null and "" values. --- .../Persistence/Repositories/SimilarNodeNameTests.cs | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/Umbraco.Tests/Persistence/Repositories/SimilarNodeNameTests.cs b/src/Umbraco.Tests/Persistence/Repositories/SimilarNodeNameTests.cs index 275398091b..e0e650fa66 100644 --- a/src/Umbraco.Tests/Persistence/Repositories/SimilarNodeNameTests.cs +++ b/src/Umbraco.Tests/Persistence/Repositories/SimilarNodeNameTests.cs @@ -25,6 +25,7 @@ namespace Umbraco.Tests.Persistence.Repositories [TestCase("Kilo", "Golf (2)", +1)] [TestCase("Kilo (1)", "Golf (2)", +1)] [TestCase("", "", 0)] + [TestCase(null, null, 0)] public void ComparerTest(string name1, string name2, int expected) { var comparer = new SimilarNodeName.Comparer(); @@ -77,7 +78,9 @@ namespace Umbraco.Tests.Persistence.Repositories [TestCase(0, "Alpha", "Alpha (3)")] [TestCase(0, "Kilo (1)", "Kilo (1) (1)")] // though... we might consider "Kilo (2)" [TestCase(6, "Kilo (1)", "Kilo (1)")] // because of the id - public void Test(int nodeId, string nodeName, string expected) + [TestCase(0, "", " (1)")] + [TestCase(0, null, " (1)")] + public void Test(int nodeId, string nodeName, string expected) { var names = new[] {