fix: use rejected Promise for terminate() (#845)

dev/src/pool.ts

@@ -184,7 +184,7 @@ export class ClientPool<T> {
    */
   run<V>(requestTag: string, op: (client: T) => Promise<V>): Promise<V> {
     if (this.terminated) {
-      throw new Error('The client has already been terminated');
+      return Promise.reject('The client has already been terminated');
     }
     const client = this.acquire(requestTag);
 

dev/system-test/firestore.ts

@@ -130,19 +130,14 @@ describe('Firestore class', () => {
 
   it('cannot make calls after the client has been terminated', () => {
     const ref1 = randomCol.doc('doc1');
-    ref1.onSnapshot(snapshot => {
-      return Promise.reject('onSnapshot() should be called');
-    });
     return firestore
       .terminate()
       .then(() => {
         return ref1.set({foo: 100});
       })
-      .then(() => {
-        Promise.reject('set() should have failed');
-      })
+      .then(() => Promise.reject('set() should have failed'))
       .catch(err => {
-        expect(err.message).to.equal('The client has already been terminated');
+        expect(err).to.equal('The client has already been terminated');
       });
   });
 });