Fix bundle query resume issue

Author: wu-hui

firebase-firestore/src/main/java/com/google/firebase/firestore/bundle/BundleSerializer.java

@@ -22,7 +22,6 @@
 import com.google.firebase.firestore.core.Filter;
 import com.google.firebase.firestore.core.OrderBy;
 import com.google.firebase.firestore.core.Query;
-import com.google.firebase.firestore.core.Target;
 import com.google.firebase.firestore.model.DocumentKey;
 import com.google.firebase.firestore.model.FieldPath;
 import com.google.firebase.firestore.model.MutableDocument;
@@ -154,7 +153,9 @@ private BundledQuery decodeBundledQuery(JSONObject bundledQuery) throws JSONExce
     Query.LimitType limitType = decodeLimitType(bundledQuery);
 
     return new BundledQuery(
-        new Target(parent, collectionGroup, filters, orderBys, limit, startAt, endAt), limitType);
+        new Query(parent, collectionGroup, filters, orderBys, limit, limitType, startAt, endAt)
+            .toTarget(),
+        limitType);
   }
 
   private int decodeLimit(JSONObject structuredQuery) {

firebase-firestore/src/test/java/com/google/firebase/firestore/bundle/BundleReaderTest.java

@@ -24,7 +24,6 @@
 import static org.junit.Assert.assertThrows;
 
 import com.google.firebase.firestore.core.Query;
-import com.google.firebase.firestore.core.Target;
 import com.google.firebase.firestore.model.DatabaseId;
 import com.google.firebase.firestore.model.ObjectValue;
 import com.google.firebase.firestore.model.ResourcePath;
@@ -57,28 +56,32 @@ public class BundleReaderTest {
       new NamedQuery(
           "limitQuery",
           new BundledQuery(
-              new Target(
-                  ResourcePath.fromString("foo"),
-                  null,
-                  Collections.emptyList(),
-                  Collections.singletonList(orderBy("sort")),
-                  1,
-                  null,
-                  null),
+              new Query(
+                      ResourcePath.fromString("foo"),
+                      null,
+                      Collections.emptyList(),
+                      Collections.singletonList(orderBy("sort")),
+                      1,
+                      Query.LimitType.LIMIT_TO_FIRST,
+                      null,
+                      null)
+                  .toTarget(),
               Query.LimitType.LIMIT_TO_FIRST),
           version(1590011379000001L));
   public static final NamedQuery LIMIT_TO_LAST_QUERY =
       new NamedQuery(
           "limitToLastQuery",
           new BundledQuery(
-              new Target(
-                  ResourcePath.fromString("foo"),
-                  null,
-                  Collections.emptyList(),
-                  Collections.singletonList(orderBy("sort")),
-                  1,
-                  null,
-                  null),
+              new Query(
+                      ResourcePath.fromString("foo"),
+                      null,
+                      Collections.emptyList(),
+                      Collections.singletonList(orderBy("sort")),
+                      1,
+                      Query.LimitType.LIMIT_TO_LAST,
+                      null,
+                      null)
+                  .toTarget(),
               Query.LimitType.LIMIT_TO_LAST),
           version(1590011379000002L));
   public static final BundledDocumentMetadata DELETED_DOC_METADATA =

firebase-firestore/src/test/java/com/google/firebase/firestore/bundle/BundleSerializerTest.java

@@ -51,8 +51,10 @@
 @RunWith(RobolectricTestRunner.class)
 @Config(manifest = Config.NONE)
 public class BundleSerializerTest {
-  // Note: This tests uses single-quoted JSON strings, which are accepted by org.json.JSONObject.
-  // While they are invalid JSON, they allow us to no use non-escaped quotes in the test file.
+  // Note: This tests uses single-quoted JSON strings, which are accepted by
+  // org.json.JSONObject.
+  // While they are invalid JSON, they allow us to no use non-escaped quotes in
+  // the test file.
 
   private static String TEST_PROJECT = "projects/project/databases/(default)/documents";
   private static String TEST_DOCUMENT = TEST_PROJECT + "/coll/doc";
@@ -672,6 +674,32 @@ public void testDecodesBundledDocumentMetadata() throws JSONException {
     assertEquals(expectedMetadata, actualMetadata);
   }
 
+  @Test
+  public void testDecodesTargetWithoutImplicitOrderByOnName() throws JSONException {
+    String json =
+        "{\"name\":\"myNamedQuery\",\"bundledQuery\":{\"parent\":\"projects/project/databases"
+            + "/(default)/documents\",\"structuredQuery\":{\"from\":[{\"collectionId\":\"foo\"}],"
+            + "\"limit\":{\"value\":10}},\"limitType\":\"FIRST\"},"
+            + "\"readTime\":{\"seconds\":\"1679674432\",\"nanos\":579934000}}";
+    NamedQuery query = serializer.decodeNamedQuery(new JSONObject(json));
+    assertEquals(
+        TestUtil.query("foo").limitToFirst(10).toTarget(), query.getBundledQuery().getTarget());
+    assertEquals(Query.LimitType.LIMIT_TO_FIRST, query.getBundledQuery().getLimitType());
+  }
+
+  @Test
+  public void testDecodesLimitToLastTargetWithoutImplicitOrderByOnName() throws JSONException {
+    String json =
+        "{\"name\":\"myNamedQuery\",\"bundledQuery\":{\"parent\":\"projects/project/databases"
+            + "/(default)/documents\",\"structuredQuery\":{\"from\":[{\"collectionId\":\"foo\"}],"
+            + "\"limit\":{\"value\":10}},\"limitType\":\"LAST\"},"
+            + "\"readTime\":{\"seconds\":\"1679674432\",\"nanos\":579934000}}";
+    NamedQuery query = serializer.decodeNamedQuery(new JSONObject(json));
+    assertEquals(
+        TestUtil.query("foo").limitToLast(10).toTarget(), query.getBundledQuery().getTarget());
+    assertEquals(Query.LimitType.LIMIT_TO_LAST, query.getBundledQuery().getLimitType());
+  }
+
   private void assertDecodesValue(String json, Value proto) throws JSONException {
     String documentJson =
         "{\n"
@@ -719,15 +747,7 @@ private void assertDecodesNamedQuery(String json, Query query) throws JSONExcept
             + "}";
     NamedQuery actualNamedQuery = serializer.decodeNamedQuery(new JSONObject(queryJson));
 
-    Target target =
-        new Target(
-            query.getPath(),
-            query.getCollectionGroup(),
-            query.getFilters(),
-            query.getExplicitOrderBy(),
-            query.getLimit(),
-            query.getStartAt(),
-            query.getEndAt());
+    Target target = query.toTarget();
     BundledQuery bundledQuery =
         new BundledQuery(
             target,