added reproduction for akkadotnet#4199

Aaronontheweb · Feb 11, 2020 · a78844d · a78844d
1 parent fc11431
commit a78844d
Showing 1 changed file with 19 additions and 1 deletion.
diff --git a/src/contrib/cluster/Akka.DistributedData.Tests/PNCounterDictionarySpec.cs b/src/contrib/cluster/Akka.DistributedData.Tests/PNCounterDictionarySpec.cs
@@ -153,7 +153,7 @@ public void Bugfix_4198_PNCounterMapDeltas_must_merge_other_PNCounterMaps()
                 dataEnvelope = dataEnvelope.WithData(withDelta.Zero.MergeDelta(withDelta));
             }
 
-            // Bug: this is now an ORDictionary<string, PNCounter> under #4198
+            // Bug: this is was an ORDictionary<string, PNCounter> under #4198
             var storedData = dataEnvelope.Data;
 
             // simulate merging an update
@@ -168,5 +168,23 @@ public void Bugfix_4198_PNCounterMapDeltas_must_merge_other_PNCounterMaps()
 
             m3.Entries.ShouldBeEquivalentTo(expected);
         }
+
+        /// <summary>
+        /// Bug reproduction: https://github.com/akkadotnet/akka.net/issues/4199
+        /// </summary>
+        [Fact]
+        public void Bugfix_4199_PNCounterMaps_must_support_pruning()
+        {
+            var m1 = PNCounterDictionary<string>.Empty
+                .Increment(_node1, "a", 1)
+                .Increment(_node1, "b", 3)
+                .Increment(_node1, "c", 2)
+                .Increment(_node2, "a", 2);
+
+            m1.Entries["a"].ShouldBe(3);
+
+            var p1 = m1.Prune(_node2, _node1);
+            p1.Entries["a"].ShouldBe(m1.Entries["a"]); // values should be merged after pruning
+        }
     }
 }