Fixes issue #639 (#640)

* Adds failing test for #639

* Fixes #639

Author: flovilmart

src/LiveQuerySubscription.js

@@ -106,7 +106,6 @@ class Subscription extends EventEmitter {
     return CoreManager.getLiveQueryController().getDefaultLiveQueryClient().then((liveQueryClient) => {
       liveQueryClient.unsubscribe(this);
       this.emit('close');
-      this.resolve();
     });
   }
 }

src/__tests__/ParseLiveQuery-test.js

@@ -20,6 +20,7 @@ jest.dontMock('../promiseUtils');
 require('../ParseLiveQuery');
 const CoreManager = require('../CoreManager');
 const ParseQuery = require('../ParseQuery').default;
+const LiveQuerySubscription = require('../LiveQuerySubscription').default;
 
 describe('ParseLiveQuery', () => {
   beforeEach(() => {
@@ -182,4 +183,11 @@ describe('ParseLiveQuery', () => {
     });
 
   });
+
+  it('should not throw on usubscribe', (done) => {
+    const query = new ParseQuery("ObjectType");
+    query.equalTo("test", "value");
+    const subscription = new LiveQuerySubscription('0', query, 'token');
+    subscription.unsubscribe().then(done).catch(done.fail);
+  });
 });