Merge branch 'feat/partial-length-unit-tests' of https://github.com/A…

…be27342/FluidFramework into feat/partial-length-unit-tests
microsoft · Aug 16, 2022 · 9081e16 · 9081e16
2 parents d7fad08 + c4af062
commit 9081e16
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/packages/dds/merge-tree/src/partialLengths.ts b/packages/dds/merge-tree/src/partialLengths.ts
@@ -870,8 +870,9 @@ function verifyPartialLengths(
             // Only the flat partialLengths can have overlapRemoveClients, the per client view shouldn't
             assert(!clientPartials, 0x058 /* "Both overlapRemoveClients and clientPartials are set!" */);
 
-            // Each overlap client count as one
-            count += partialLength.overlapRemoveClients.size();
+            // Each overlap client count as one, but the first remove to sequence was already counted.
+            // (this aligns with the logic to omit the removing client in `addClientSeqNumberFromPartial`)
+            count += partialLength.overlapRemoveClients.size() - 1;
         }
     }
     return count;