Skip to content

Commit ac62b94

Browse files
authored
Merge pull request #16 from ufcpp/fix-update
fix AddDuplicateBehavior.Update
2 parents ce7096d + c03e731 commit ac62b94

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

KdTreeLib/KdTree.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ public bool Add(TKey[] point, TValue value)
7878

7979
case AddDuplicateBehavior.Update:
8080
parent.Value = value;
81-
break;
81+
return true;
8282

8383
default:
8484
// Should never happen

KdTreeTestsLib/KdTreeTests.cs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -117,11 +117,15 @@ public void TestAddDuplicateInUpdateMode()
117117

118118
var newValue = "I love chicken, I love liver, Meow Mix Meow Mix please deliver";
119119

120+
var olcCount = tree.Count();
121+
120122
tree.Add(testNodes[0].Point, newValue);
121123

122124
var actualValue = tree.FindValueAt(testNodes[0].Point);
125+
var newCount = tree.Count();
123126

124127
Assert.AreEqual(newValue, actualValue);
128+
Assert.AreEqual(olcCount, newCount);
125129
}
126130

127131
[TestMethod]

0 commit comments

Comments
 (0)