Skip collections that has no documents

Author: ilearnio

src/firestore-document.js

@@ -175,7 +175,12 @@ MockFirestoreDocument.prototype.getCollections = function () {
   return new Promise(function (resolve, reject) {
     self._defer('getCollections', _.toArray(arguments), function () {
       if (err === null) {
-        resolve(_.toArray(this.children));
+        var collections = _.toArray(this.children);
+        // Filter out empty collections
+        collections = _.filter(collections, function (collection) {
+          return !_.isEmpty(collection.data);
+        });
+        resolve(collections);
       } else {
         reject(err);
       }

test/unit/firestore-document.js

@@ -391,14 +391,29 @@ describe('MockFirestoreDocument', function () {
     });
 
     context('when not present', function () {
-      it('returns deeply nested collections of document', function (done) {
+      it('returns empty list of collections', function (done) {
         db.doc('not-existing').getCollections().then(function (colRefs) {
           expect(colRefs).to.be.an('array');
           expect(colRefs).to.have.length(0);
           done();
         });
         db.flush();
       });
+
+      it('skips collections that has no documents', function (done) {
+        db.doc('doc/subcol/subcol-doc').delete();
+        db.doc('doc/subcol2/subcol-doc').delete();
+        db.doc('doc/subcol/subcol-doc/deep-col/deep-doc').delete();
+        db.doc('doc/subcol/subcol-doc/deep-col2/deep-doc').delete();
+        db.flush();
+
+        db.doc('doc').getCollections().then(function (colRefs) {
+          expect(colRefs).to.be.an('array');
+          expect(colRefs).to.have.length(0);
+          done();
+        });
+        db.flush();
+      });
     });
   });
 });