Do not mark the empty pushs as failed (#4497)

Author: flovilmart

spec/PushController.spec.js

@@ -951,7 +951,7 @@ describe('PushController', () => {
     });
   });
 
-  it('should mark the _PushStatus as failed when audience has no deviceToken', (done) => {
+  it('should not mark the _PushStatus as failed when audience has no deviceToken', (done) => {
     var auth = {
       isMaster: true
     }
@@ -999,16 +999,14 @@ describe('PushController', () => {
       var config = Config.get(Parse.applicationId);
       return Parse.Object.saveAll(installations).then(() => {
         return pushController.sendPush(payload, {}, config, auth)
-          .then(() => { done.fail('should not success') })
-          .catch(() => {})
       }).then(() => new Promise(resolve => setTimeout(resolve, 100)));
     }).then(() => {
       const query = new Parse.Query('_PushStatus');
       return query.find({useMasterKey: true}).then((results) => {
         expect(results.length).toBe(1);
         const pushStatus = results[0];
         expect(pushStatus.get('numSent')).toBe(0);
-        expect(pushStatus.get('status')).toBe('failed');
+        expect(pushStatus.get('status')).toBe('succeeded');
         done();
       });
     }).catch((err) => {

src/Push/PushQueue.js

@@ -38,7 +38,7 @@ export class PushQueue {
         {limit: 0, count: true});
     }).then(({results, count}) => {
       if (!results || count == 0) {
-        return Promise.reject({error: 'PushController: no results in query'})
+        return pushStatus.complete();
       }
       pushStatus.setRunning(Math.ceil(count / limit));
       let skip = 0;