Merge pull request #89 from brenmcnamara/master

Checks for multiple GeoPoints when mutating object (Issue 85)

Author: andrewimm

src/Mutation.js

@@ -65,6 +65,18 @@ function validateColumn(column: string) {
 }
 
 function validateFields(data) {
+    // Check for multiple GeoPoints.
+  var geoPointCount = 0;
+  for (var prop in data) {
+    if (data.hasOwnProperty(prop) &&
+        (data[prop] instanceof Parse.GeoPoint)) {
+        ++geoPointCount;
+        if (geoPointCount > 1) {
+          throw Error('There can only be 1 GeoPoint when mutating an object.');
+        }
+    }
+  }
+
   if (data.hasOwnProperty('objectId')) {
     warning('Ignoring reserved field: objectId');
     delete data.objectId;

src/__tests__/Mutation-test.js

@@ -48,6 +48,16 @@ describe('Mutation Generators', function() {
     expect(result instanceof Parse.Promise).toBe(true);
   });
 
+  it("fails to generate a Set Mutation with multiple GeoPoints", function () {
+      expect(function () {
+        var id = new Id('Klass', '01');
+        Mutation.Set(id, {
+          g1: new Parse.GeoPoint(),
+          g2: new Parse.GeoPoint(),
+        });
+      }).toThrow('There can only be 1 GeoPoint when mutating an object.');
+  });
+
   it('generates an Unset Mutation', function() {
     var id = new Id('Klass', 'O1');
     var m = Mutation.Unset(id, 'no_more');