Fix Unit Test

Author: schmidt-sebastian

firebase-firestore/src/test/java/com/google/firebase/firestore/local/LocalStoreTestCase.java

@@ -249,22 +249,25 @@ public void testHandlesSetMutationThenAckThenRelease() {
     allocateQuery(query);
 
     writeMutation(setMutation("foo/bar", map("foo", "bar")));
+    notifyLocalViewChanges(viewChanges(2, asList("foo/bar"), emptyList()));
+
     assertChanged(doc("foo/bar", 0, map("foo", "bar"), Document.DocumentState.LOCAL_MUTATIONS));
     assertContains(doc("foo/bar", 0, map("foo", "bar"), Document.DocumentState.LOCAL_MUTATIONS));
 
     acknowledgeMutation(1);
-    notifyLocalViewChanges(viewChanges(2, asList("foo/bar"), emptyList()));
 
     assertChanged(doc("foo/bar", 1, map("foo", "bar"), Document.DocumentState.COMMITTED_MUTATIONS));
-    assertContains(doc("foo/bar", 1, map("foo", "bar"), Document.DocumentState.COMMITTED_MUTATIONS));
+    assertContains(
+        doc("foo/bar", 1, map("foo", "bar"), Document.DocumentState.COMMITTED_MUTATIONS));
 
     releaseQuery(query);
 
     // It has been acknowledged, and should no longer be retained as there is no target and mutation
     if (garbageCollectorIsEager()) {
       assertNotContains("foo/bar");
     } else {
-      assertContains(doc("foo/bar", 0, map("foo", "bar"), Document.DocumentState.COMMITTED_MUTATIONS));
+      assertContains(
+          doc("foo/bar", 1, map("foo", "bar"), Document.DocumentState.COMMITTED_MUTATIONS));
     }
   }
 

firebase-firestore/src/test/resources/json/write_spec_test.json

@@ -5115,7 +5115,7 @@
     "describeName": "Writes:",
     "itName": "Held writes are released when there are no queries left.",
     "tags": [
-      "eager-gc", "exclusive"
+      "eager-gc"
     ],
     "comment": "This test expects a new target id for a new listen, but without eager gc, the same target id is reused",
     "config": {