Fixes #4423 - Getter on mutations size (#4424)

* Fixes #4423 Getter over mutations size Allow knowing the size of the list. For exemple, the batch size is limited to 500 mutations * Fixes #4423 - Getter on mutations size Allow knowing the size of the list. For exemple, the batch size is limited to 500 mutations Fixes #4423 * Correction format code
googleapis · Feb 8, 2019 · 64f3b2b · 64f3b2b
1 parent bb2bb7c
commit 64f3b2b
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/...lients/google-cloud-firestore/src/main/java/com/google/cloud/firestore/UpdateBuilder.java b/...lients/google-cloud-firestore/src/main/java/com/google/cloud/firestore/UpdateBuilder.java
@@ -664,4 +664,9 @@ public List<WriteResult> apply(CommitResponse commitResponse) {
   boolean isEmpty() {
     return mutations.isEmpty();
   }
+
+  /** Get the number of mutations. */
+  public int getMutationsSize() {
+    return mutations.size();
+  }
 }
diff --git a/...ients/google-cloud-firestore/src/test/java/com/google/cloud/firestore/WriteBatchTest.java b/...ients/google-cloud-firestore/src/test/java/com/google/cloud/firestore/WriteBatchTest.java
@@ -91,6 +91,8 @@ public void updateDocument() throws Exception {
     batch.update(documentReference, updateTime, "foo", "bar");
     batch.update(documentReference, LocalFirestoreHelper.SINGLE_FIELD_MAP, updateTime);
 
+    assertEquals(4, batch.getMutationsSize());
+
     List<WriteResult> writeResults = batch.commit().get();
     List<Write> writes = new ArrayList<>();
 
@@ -126,6 +128,8 @@ public void setDocument() throws Exception {
     writes.add(set(LocalFirestoreHelper.SINGLE_FIELD_PROTO, Arrays.asList("foo")));
     writes.add(set(LocalFirestoreHelper.SINGLE_FIELD_PROTO, Arrays.asList("foo")));
 
+    assertEquals(4, batch.getMutationsSize());
+
     List<WriteResult> writeResults = batch.commit().get();
     for (int i = 0; i < writeResults.size(); ++i) {
       assertEquals(Timestamp.ofTimeSecondsAndNanos(i, i), writeResults.get(i).getUpdateTime());
@@ -144,6 +148,8 @@ public void omitWriteResultForDocumentTransforms() throws Exception {
 
     batch.set(documentReference, map("time", FieldValue.serverTimestamp()));
 
+    assertEquals(1, batch.getMutationsSize());
+
     List<WriteResult> writeResults = batch.commit().get();
     assertEquals(1, writeResults.size());
   }
@@ -159,6 +165,8 @@ public void createDocument() throws Exception {
         .create(documentReference, LocalFirestoreHelper.SINGLE_FIELD_MAP)
         .create(documentReference, LocalFirestoreHelper.SINGLE_FIELD_OBJECT);
 
+    assertEquals(2, batch.getMutationsSize());
+
     List<WriteResult> writeResults = batch.commit().get();
     List<Write> writes = new ArrayList<>();
 
@@ -188,6 +196,8 @@ public void deleteDocument() throws Exception {
     precondition.getUpdateTimeBuilder().setSeconds(1).setNanos(2);
     writes.add(delete(precondition.build()));
 
+    assertEquals(2, batch.getMutationsSize());
+
     List<WriteResult> writeResults = batch.commit().get();
 
     for (int i = 0; i < writeResults.size(); ++i) {