Correct the return value of the forEach function

Author: Byeol

spec/providers/database.spec.ts

@@ -208,7 +208,7 @@ describe('DeltaSnapshot', () => {
     });
   });
 
-  describe('#forEach(childAction: Function)', () => {
+  describe('#forEach(action: (a: DeltaSnapshot) => boolean): boolean', () => {
     it('should iterate through child snapshots', () => {
       populate({ a: 'b' }, { c: 'd' });
       let out = '';
@@ -229,6 +229,32 @@ describe('DeltaSnapshot', () => {
       subject.forEach(counter);
       expect(count).to.eq(0);
     });
+
+    it('should cancel further enumeration if callback returns true', () => {
+      populate(null, { a: 'b', c: 'd', e: 'f', g: 'h' });
+      let out = '';
+      const ret = subject.forEach(snap => {
+        if (snap.val() === 'f') {
+          return true;
+        }
+        out += snap.val();
+      });
+      expect(out).to.equal('bd');
+      expect(ret).to.equal(true);
+    });
+
+    it('should not cancel further enumeration if callback does not return true', () => {
+      populate(null, { a: 'b', c: 'd', e: 'f', g: 'h' });
+      let out = '';
+      const ret = subject.forEach(snap => {
+        if (snap.val() === 'a') {
+          return true;
+        }
+        out += snap.val();
+      });
+      expect(out).to.equal('bdfh');
+      expect(ret).to.equal(false);
+    });
   });
 
   describe('#numChildren()', () => {

src/providers/database.ts

@@ -206,11 +206,10 @@ export class DeltaSnapshot implements firebase.database.DataSnapshot {
     return valAt(this._delta, this._childPath) !== undefined;
   }
 
-  // TODO(inlined) what is this boolean for?
-  forEach(action: (a: DeltaSnapshot) => void): boolean {
+  forEach(action: (a: DeltaSnapshot) => boolean): boolean {
     let val = this.val();
     if (_.isPlainObject(val)) {
-      _.keys(val).forEach(key => action(this.child(key)));
+      return _.keys(val).some(key => action(this.child(key)));
     }
     return false;
   }