Refactors changes to reflect pull request feedback

Author: brenmcnamara

src/Mutation.js

@@ -65,20 +65,17 @@ function validateColumn(column: string) {
 }
 
 function validateFields(data) {
-
-  function hasMutlipleGeoPoints() {
-    var geoPointCount = 0;
-    for (var prop in data) {
-      if (data.hasOwnProperty(prop) &&
-          (data[prop] instanceof Parse.GeoPoint)) {
-          ++geoPointCount;
-          if (geoPointCount > 1) {
-            return true;
-          }
-      }
+    // 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.');
+        }
     }
-    return false;
-  };
+  }
 
   if (data.hasOwnProperty('objectId')) {
     warning('Ignoring reserved field: objectId');
@@ -96,9 +93,6 @@ function validateFields(data) {
     warning('Ignoring reserved field: updatedAt');
     delete data.updatedAt;
   }
-  if (hasMutlipleGeoPoints()) {
-    throw Error('There can only be 1 GeoPoint when mutating an object.');
-  }
 }
 
 class Mutation {

src/__tests__/Mutation-test.js

@@ -55,7 +55,7 @@ describe('Mutation Generators', function() {
           g1: new Parse.GeoPoint(),
           g2: new Parse.GeoPoint(),
         });
-      }).toThrow();
+      }).toThrow('There can only be 1 GeoPoint when mutating an object.');
   });
 
   it('generates an Unset Mutation', function() {