Fix empty message reject in transaction body

Author: cherylEnkidu

packages/firestore/src/core/transaction_runner.ts

@@ -113,7 +113,7 @@ export class TransactionRunner<T> {
   }
 
   private isRetryableTransactionError(error: Error): boolean {
-    if (error.name === 'FirebaseError') {
+    if (error?.name === 'FirebaseError') {
       // In transactions, the backend will fail outdated reads with FAILED_PRECONDITION and
       // non-matching document versions with ABORTED. These errors should be retried.
       const code = (error as FirestoreError).code;

packages/firestore/test/integration/api/transactions.test.ts

@@ -593,6 +593,19 @@ apiDescribe('Database transactions', persistence => {
     }
   );
 
+  it('runTransaction with empty message reject inside', () => {
+    return withTestDb(persistence, async db => {
+      try {
+        await runTransaction(db, () => {
+          return Promise.reject();
+        });
+        expect.fail('transaction should fail');
+      } catch (err) {
+        expect(err).to.be.undefined;
+      }
+    });
+  });
+
   describe('must return a promise:', () => {
     const noop = (): void => {
       /* -_- */