discourage use of getData by making "private"

Author: soumak77

src/firestore-document.js

@@ -50,7 +50,7 @@ MockFirestoreDocument.prototype.getFlushQueue = function () {
   return this.queue.getEvents();
 };
 
-MockFirestoreDocument.prototype.getData = function () {
+MockFirestoreDocument.prototype._getData = function () {
   return _.cloneDeep(this.data);
 };
 
@@ -79,7 +79,7 @@ MockFirestoreDocument.prototype.get = function () {
   return new Promise(function (resolve, reject) {
     self._defer('get', _.toArray(arguments), function () {
       if (err === null) {
-        resolve(new DocumentSnapshot(self.id, self.ref, self.getData()));
+        resolve(new DocumentSnapshot(self.id, self.ref, self._getData()));
       } else {
         reject(err);
       }
@@ -118,7 +118,7 @@ MockFirestoreDocument.prototype._update = function (changes, opts, callback) {
   return new Promise(function (resolve, reject) {
     self._defer('update', _.toArray(arguments), function () {
       if (!err) {
-        var base = self.getData();
+        var base = self._getData();
         var data;
         if (_opts.setMerge) {
           data = _.merge(_.isObject(base) ? base : {}, changes);

src/firestore-query.js

@@ -53,7 +53,7 @@ MockFirestoreQuery.prototype._setData = function (data) {
   this.data = utils.cleanFirestoreData(_.cloneDeep(data) || null);
 };
 
-MockFirestoreQuery.prototype.getData = function () {
+MockFirestoreQuery.prototype._getData = function () {
   return _.cloneDeep(this.data);
 };
 
@@ -138,14 +138,14 @@ MockFirestoreQuery.prototype.where = function (property, operator, value) {
 };
 
 MockFirestoreQuery.prototype.orderBy = function (property, direction) {
-  var query = new MockFirestoreQuery(this.path, this.getData(), this.parent, this.id);
+  var query = new MockFirestoreQuery(this.path, this._getData(), this.parent, this.id);
   query.orderedProperties.push(property);
   query.orderedDirections.push(direction || 'asc');
   return query;
 };
 
 MockFirestoreQuery.prototype.limit = function (limit) {
-  var query = new MockFirestoreQuery(this.path, this.getData(), this.parent, this.id);
+  var query = new MockFirestoreQuery(this.path, this._getData(), this.parent, this.id);
   query.limited = limit;
   return query;
 };