resolve comments

Author: milaGGL

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

@@ -405,7 +405,7 @@ public void testEncodesTargetData() {
   }
 
   @Test
-  public void localSerializerDropsExpectedCountInTargetData() {
+  public void localSerializerShouldDropExpectedCountInTargetData() {
     Query query = TestUtil.query("room");
     int targetId = 42;
     long sequenceNumber = 10;

firebase-firestore/src/test/java/com/google/firebase/firestore/remote/RemoteEventTest.java

@@ -459,7 +459,7 @@ public void testExistenceFilterMismatchClearsTarget() {
   }
 
   @Test
-  public void existenceFilterMismatchWithBloomFilterSuccess() {
+  public void existenceFilterMismatchWithSuccessfulBloomFilterApplication() {
     Map<Integer, TargetData> targetMap = activeQueries(1, 2);
 
     MutableDocument doc1 = doc("docs/1", 1, map("value", 1));
@@ -551,10 +551,10 @@ public void existenceFilterMismatchWithBloomFilterFalsePositiveResult() {
     TargetChange mapping2 = targetChange(resumeToken, false, null, null, null);
     assertEquals(mapping2, event.getTargetChanges().get(2));
 
-    // This BloomFilter will return true on both MightContain(doc1) and MightContain(doc2).
+    // With this BloomFilter, mightContain() will return true for all documents.
     BitSequence.Builder bitSequence = BitSequence.newBuilder();
     bitSequence.setPadding(7);
-    bitSequence.setBitmap(ByteString.copyFrom(new byte[] {0x42, (byte) 0xFE}));
+    bitSequence.setBitmap(ByteString.copyFrom(new byte[] {(byte) 0xFF, (byte) 0xFF}));
     com.google.firestore.v1.BloomFilter.Builder bloomFilter = BloomFilter.newBuilder();
     bloomFilter.setBits(bitSequence);
     bloomFilter.setHashCount(33);

firebase-firestore/src/test/java/com/google/firebase/firestore/remote/RemoteSerializerTest.java

@@ -1221,7 +1221,7 @@ public void encodesExpectedCountWhenReadTimeIsPresent() {
   }
 
   @Test
-  public void ignoresExpectedCountWithoutResumeTokenOrReadTime() {
+  public void shouldIgnoreExpectedCountWithoutResumeTokenOrReadTime() {
     Query q = Query.atPath(ResourcePath.fromString("docs"));
     TargetData targetData =
         new TargetData(q.toTarget(), 1, 2, QueryPurpose.LISTEN).withExpectedCount(42);