Update encoder/decoder

Author: natario1

Parse/src/main/java/com/parse/ParseFile.java

@@ -240,6 +240,7 @@ protected ParseFile(Parcel source) {
    * Creates a new file instance from a {@link Parcel} using the given {@link ParseParcelDecoder}.
    * The decoder is currently unused, but it might be in the future, plus this is the pattern we
    * are using in parcelable classes.
+   *
    * @param source the parcel
    * @param decoder the decoder
    */

Parse/src/main/java/com/parse/ParseGeoPoint.java

@@ -70,16 +70,26 @@ public ParseGeoPoint(ParseGeoPoint point) {
     this(point.getLatitude(), point.getLongitude());
   }
 
-
   /**
-   * Creates a new point instance from a Parcel {@code source}. This is used when unparceling a
+   * Creates a new point instance from a {@link Parcel} source. This is used when unparceling a
    * ParseGeoPoint. Subclasses that need Parcelable behavior should provide their own
    * {@link android.os.Parcelable.Creator} and override this constructor.
    *
-   * @param source
-   *          The recovered parcel.
+   * @param source The recovered parcel.
    */
   protected ParseGeoPoint(Parcel source) {
+    this(source, ParseParcelDecoder.get());
+  }
+
+  /**
+   * Creates a new point instance from a {@link Parcel} using the given {@link ParseParcelDecoder}.
+   * The decoder is currently unused, but it might be in the future, plus this is the pattern we
+   * are using in parcelable classes.
+   *
+   * @param source the parcel
+   * @param decoder the decoder
+   */
+  ParseGeoPoint(Parcel source, ParseParcelDecoder decoder) {
     setLatitude(source.readDouble());
     setLongitude(source.readDouble());
   }
@@ -292,14 +302,18 @@ public int describeContents() {
 
   @Override
   public void writeToParcel(Parcel dest, int flags) {
+    writeToParcel(dest, ParseParcelEncoder.get());
+  }
+
+  void writeToParcel(Parcel dest, ParseParcelEncoder encoder) {
     dest.writeDouble(latitude);
     dest.writeDouble(longitude);
   }
 
   public final static Creator<ParseGeoPoint> CREATOR = new Creator<ParseGeoPoint>() {
     @Override
     public ParseGeoPoint createFromParcel(Parcel source) {
-      return new ParseGeoPoint(source);
+      return new ParseGeoPoint(source, ParseParcelDecoder.get());
     }
 
     @Override

Parse/src/main/java/com/parse/ParseParcelDecoder.java

@@ -62,6 +62,9 @@ public Object decode(Parcel source) {
       case ParseParcelEncoder.TYPE_FILE:
         return new ParseFile(source, this);
 
+      case ParseParcelEncoder.TYPE_GEOPOINT:
+        return new ParseGeoPoint(source, this);
+
       case ParseParcelEncoder.TYPE_ACL:
         return new ParseACL(source, this);
 

Parse/src/main/java/com/parse/ParseParcelEncoder.java

@@ -41,7 +41,7 @@ private static boolean isValidType(Object value) {
     return ParseEncoder.isValidType(value);
   }
 
-  /* package */ final static String TYPE_OBJECT = "ParseObject";
+  /* package */ final static String TYPE_OBJECT = "Object";
   /* package */ final static String TYPE_POINTER = "Pointer";
   /* package */ final static String TYPE_DATE = "Date";
   /* package */ final static String TYPE_BYTES = "Bytes";
@@ -53,7 +53,8 @@ private static boolean isValidType(Object value) {
   /* package */ final static String TYPE_NULL = "Null";
   /* package */ final static String TYPE_NATIVE = "Native";
   /* package */ final static String TYPE_OP = "Operation";
-  /* package */ final static String TYPE_FILE = "ParseFile";
+  /* package */ final static String TYPE_FILE = "File";
+  /* package */ final static String TYPE_GEOPOINT = "GeoPoint";
 
   public void encode(Object object, Parcel dest) {
     try {
@@ -80,7 +81,8 @@ public void encode(Object object, Parcel dest) {
         ((ParseFile) object).writeToParcel(dest, this);
 
       } else if (object instanceof ParseGeoPoint) {
-        throw new IllegalArgumentException("Not supported yet");
+        dest.writeString(TYPE_GEOPOINT);
+        ((ParseGeoPoint) object).writeToParcel(dest, this);
 
       } else if (object instanceof ParseACL) {
         dest.writeString(TYPE_ACL);

Parse/src/test/java/com/parse/ParseGeoPointTest.java

@@ -18,7 +18,7 @@
 import static org.junit.Assert.*;
 
 @RunWith(RobolectricTestRunner.class)
-@Config(constants = BuildConfig.class, sdk = 23)
+@Config(constants = BuildConfig.class, sdk = TestHelper.ROBOLECTRIC_SDK_VERSION)
 public class ParseGeoPointTest {
 
   @Test

Parse/src/test/java/com/parse/ParseObjectTest.java

@@ -497,7 +497,6 @@ public void testGetLongWithWrongValue() throws Exception {
 
   @Test
   public void testParcelable() throws Exception {
-    // TODO test ParseGeoPoint after merge
     ParseObject object = ParseObject.createWithoutData("Test", "objectId");
     object.isDeleted = true;
     object.put("long", 200L);
@@ -531,6 +530,9 @@ public void testParcelable() throws Exception {
     // File
     ParseFile file = new ParseFile(new ParseFile.State.Builder().url("fileUrl").build());
     object.put("file", file);
+    // GeoPoint
+    ParseGeoPoint point = new ParseGeoPoint(30d, 50d);
+    object.put("point", point);
 
     Parcel parcel = Parcel.obtain();
     object.writeToParcel(parcel, 0);
@@ -557,6 +559,8 @@ public void testParcelable() throws Exception {
     assertEquals(newRel.getKnownObjects().size(), rel.getKnownObjects().size());
     newRel.hasKnownObject(related);
     assertEquals(newObject.getParseFile("file").getUrl(), object.getParseFile("file").getUrl());
+    assertEquals(newObject.getParseGeoPoint("point").getLatitude(),
+            object.getParseGeoPoint("point").getLatitude(), 0);
   }
 
   @Test