@dereekb/firebase 13.10.1 → 13.10.3

package/test/index.cjs.js

@@ -419,8 +419,7 @@ var bucketTestNameKey = 0;
     var defaultBucket = !driver.getDefaultBucket && useTestDefaultBucket !== false || useTestDefaultBucket === true ? util.cachedGetter(function() {
         var time = new Date().getTime();
         var random = Math.ceil(Math.random() * 999999) % 999999;
-        var testBucketName = "test-bucket-".concat(time, "-").concat(random, "-").concat(bucketTestNameKey += 1);
-        return testBucketName;
+        return "test-bucket-".concat(time, "-").concat(random, "-").concat(bucketTestNameKey += 1);
     }) : driver.getDefaultBucket;
     var injectedDriver = _object_spread_props$1(_object_spread$2({}, driver), {
         getDefaultBucket: defaultBucket
@@ -1446,10 +1445,12 @@ function _is_native_reflect_construct$4() {
 }(firebase.AbstractFirestoreDocument);
 /**
  * Firestore collection path name.
- */ var mockItemUserCollectionName = 'mockItemUser';
+ */ // eslint-disable-next-line @typescript-eslint/naming-convention -- camelCase chosen to match neighboring mock exports in this test fixture
+var mockItemUserCollectionName = 'mockItemUser';
 /**
  * Default document identifier used for MockItemUser in tests.
- */ var mockItemUserIdentifier = '0';
+ */ // eslint-disable-next-line @typescript-eslint/naming-convention -- camelCase chosen to match neighboring mock exports in this test fixture
+var mockItemUserIdentifier = '0';
 /**
  * Used to build a FirestoreDataConverter. Fields are configured via configuration. See the SnapshotConverterFunctions for more info.
  */ var mockItemUserConverter = firebase.snapshotConverterFunctions({
@@ -1723,7 +1724,7 @@ function _class_call_check$4(instance, Constructor) {
 /**
  * Model service factory for {@link MockItem}. Returns configurable roles from `context.rolesToReturn`, defaulting to `{ read: true }`.
  */ var mockItemFirebaseModelServiceFactory = firebase.firebaseModelServiceFactory({
-    roleMapForModel: function roleMapForModel(output, context, model) {
+    roleMapForModel: function roleMapForModel(output, context, _model) {
         var _context_rolesToReturn;
         var roles = (_context_rolesToReturn = context.rolesToReturn) !== null && _context_rolesToReturn !== void 0 ? _context_rolesToReturn : {
             read: true
@@ -1737,7 +1738,7 @@ function _class_call_check$4(instance, Constructor) {
 /**
  * Model service factory for {@link MockItemPrivate}. Returns configurable roles from `context.rolesToReturn`, defaulting to `{ read: true }`.
  */ var mockItemPrivateFirebaseModelServiceFactory = firebase.firebaseModelServiceFactory({
-    roleMapForModel: function roleMapForModel(output, context, model) {
+    roleMapForModel: function roleMapForModel(output, context, _model) {
         var _context_rolesToReturn;
         var roles = (_context_rolesToReturn = context.rolesToReturn) !== null && _context_rolesToReturn !== void 0 ? _context_rolesToReturn : {
             read: true
@@ -1768,7 +1769,7 @@ function _class_call_check$4(instance, Constructor) {
 /**
  * Model service factory for {@link MockItemSubItem}. Returns configurable roles from `context.rolesToReturn`, defaulting to `{ read: true }`.
  */ var mockItemSubItemFirebaseModelServiceFactory = firebase.firebaseModelServiceFactory({
-    roleMapForModel: function roleMapForModel(output, context, model) {
+    roleMapForModel: function roleMapForModel(output, context, _model) {
         var _context_rolesToReturn;
         var roles = (_context_rolesToReturn = context.rolesToReturn) !== null && _context_rolesToReturn !== void 0 ? _context_rolesToReturn : {
             read: true
@@ -1782,7 +1783,7 @@ function _class_call_check$4(instance, Constructor) {
 /**
  * Model service factory for {@link MockItemSubItemDeep}. Returns configurable roles from `context.rolesToReturn`, defaulting to `{ read: true }`.
  */ var mockItemSubItemDeepFirebaseModelServiceFactory = firebase.firebaseModelServiceFactory({
-    roleMapForModel: function roleMapForModel(output, context, model) {
+    roleMapForModel: function roleMapForModel(output, context, _model) {
         var _context_rolesToReturn;
         var roles = (_context_rolesToReturn = context.rolesToReturn) !== null && _context_rolesToReturn !== void 0 ? _context_rolesToReturn : {
             read: true
@@ -1796,7 +1797,7 @@ function _class_call_check$4(instance, Constructor) {
 /**
  * Model service factory for {@link SystemState}. Only grants access to system admins via {@link grantFullAccessIfAdmin}.
  */ var mockItemSystemStateFirebaseModelServiceFactory = firebase.firebaseModelServiceFactory({
-    roleMapForModel: function roleMapForModel(output, context, model) {
+    roleMapForModel: function roleMapForModel(output, context, _model) {
         return firebase.grantFullAccessIfAdmin(context); // only sys-admin allowed
     },
     getFirestoreCollection: function getFirestoreCollection(c) {
@@ -2018,7 +2019,7 @@ function _is_native_reflect_construct$3() {
  *   it('should work', () => { ... });
  * }));
  * ```
- */ function testWithMockItemCollectionFixture(config) {
+ */ function testWithMockItemCollectionFixture(_config) {
     return test.instanceWrapTestContextFactory({
         wrapFixture: function wrapFixture(fixture) {
             return new MockItemCollectionFixture(fixture);
@@ -2026,7 +2027,9 @@ function _is_native_reflect_construct$3() {
         makeInstance: function makeInstance(wrap) {
             return new MockItemCollectionFixtureInstance(wrap);
         },
-        teardownInstance: function teardownInstance(instance) {}
+        teardownInstance: function teardownInstance(_instance) {
+            return undefined;
+        }
     });
 }
 
@@ -2182,7 +2185,7 @@ function _is_native_reflect_construct$2() {
  *   it('should upload', () => { ... });
  * }));
  * ```
- */ function testWithMockItemStorageFixture(config) {
+ */ function testWithMockItemStorageFixture(_config) {
     return test.instanceWrapTestContextFactory({
         wrapFixture: function wrapFixture(fixture) {
             return new MockItemStorageFixture(fixture);
@@ -2190,7 +2193,9 @@ function _is_native_reflect_construct$2() {
         makeInstance: function makeInstance(wrap) {
             return new MockItemStorageFixtureInstance(wrap);
         },
-        teardownInstance: function teardownInstance(instance) {}
+        teardownInstance: function teardownInstance(_instance) {
+            return undefined;
+        }
     });
 }
 
@@ -2554,7 +2559,6 @@ function _ts_generator$4(thisArg, body) {
             var itemDocument;
             beforeEach(function() {
                 itemDocument = items[0];
-                itemDocument.accessor;
             });
             describe('accessors', function() {
                 describeFirestoreDocumentAccessorTests(function() {
@@ -2755,24 +2759,19 @@ function _ts_generator$4(thisArg, body) {
                                                         return _ts_generator$4(this, function(_state) {
                                                             switch(_state.label){
                                                                 case 0:
-                                                                    return [
-                                                                        4,
-                                                                        f.instance.mockItemCollection.documentAccessorForTransaction(transaction).loadDocumentForId(itemDocument.id)
-                                                                    ];
-                                                                case 1:
-                                                                    itemDocumentInTransaction = _state.sent();
+                                                                    itemDocumentInTransaction = f.instance.mockItemCollection.documentAccessorForTransaction(transaction).loadDocumentForId(itemDocument.id);
                                                                     return [
                                                                         4,
                                                                         itemDocumentInTransaction.snapshotData()
                                                                     ];
-                                                                case 2:
+                                                                case 1:
                                                                     data = _state.sent();
                                                                     expect(data === null || data === void 0 ? void 0 : data.number).toBe(undefined);
                                                                     return [
                                                                         4,
                                                                         itemDocumentInTransaction.increment(update)
                                                                     ];
-                                                                case 3:
+                                                                case 2:
                                                                     _state.sent();
                                                                     return [
                                                                         2
@@ -2810,29 +2809,24 @@ function _ts_generator$4(thisArg, body) {
                                                 number: 3
                                             };
                                             writeBatch = f.parent.firestoreContext.batch();
-                                            return [
-                                                4,
-                                                f.instance.mockItemCollection.documentAccessorForWriteBatch(writeBatch).loadDocumentForId(itemDocument.id)
-                                            ];
-                                        case 1:
-                                            itemDocumentForWriteBatch = _state.sent();
+                                            itemDocumentForWriteBatch = f.instance.mockItemCollection.documentAccessorForWriteBatch(writeBatch).loadDocumentForId(itemDocument.id);
                                             return [
                                                 4,
                                                 itemDocumentForWriteBatch.increment(update)
                                             ];
-                                        case 2:
+                                        case 1:
                                             _state.sent();
                                             return [
                                                 4,
                                                 writeBatch.commit()
                                             ];
-                                        case 3:
+                                        case 2:
                                             _state.sent();
                                             return [
                                                 4,
                                                 itemDocument.snapshotData()
                                             ];
-                                        case 4:
+                                        case 3:
                                             result = _state.sent();
                                             expect(result === null || result === void 0 ? void 0 : result.number).toBe(update.number);
                                             return [
@@ -2954,7 +2948,6 @@ function _ts_generator$4(thisArg, body) {
                         testUserId = 'userid' + Math.ceil(Math.random() * 100000);
                         mockItemUserFirestoreCollection = f.instance.collections.mockItemUserCollectionFactory(itemDocument);
                         itemUserDataDocument = mockItemUserFirestoreCollection.documentAccessor().loadDocumentForId(testUserId);
-                        itemUserDataDocument.accessor;
                     });
                     describe('create()', function() {
                         describe('mockItemUserAccessorFactory usage', function() {
@@ -3398,10 +3391,10 @@ function _ts_generator$4(thisArg, body) {
                                     hasDataFromUpdate: function hasDataFromUpdate(data) {
                                         return data.comments === TEST_COMMENTS;
                                     },
-                                    loadDocumentForTransaction: function loadDocumentForTransaction(transaction, ref) {
+                                    loadDocumentForTransaction: function loadDocumentForTransaction(transaction) {
                                         return mockItemPrivateFirestoreCollection.loadDocumentForTransaction(transaction);
                                     },
-                                    loadDocumentForWriteBatch: function loadDocumentForWriteBatch(writeBatch, ref) {
+                                    loadDocumentForWriteBatch: function loadDocumentForWriteBatch(writeBatch) {
                                         return mockItemPrivateFirestoreCollection.loadDocumentForWriteBatch(writeBatch);
                                     }
                                 };
@@ -3535,29 +3528,104 @@ function _ts_generator$4(thisArg, body) {
                     })();
                 });
                 test.itShouldFail('if the path is invalid (points to collection)', function() {
-                    test.expectFail(function() {
-                        mockItemFirestoreDocumentAccessor.loadDocumentForKey('path');
-                    });
+                    return _async_to_generator$4(function() {
+                        return _ts_generator$4(this, function(_state) {
+                            switch(_state.label){
+                                case 0:
+                                    return [
+                                        4,
+                                        test.expectFail(function() {
+                                            mockItemFirestoreDocumentAccessor.loadDocumentForKey('path');
+                                        })
+                                    ];
+                                case 1:
+                                    _state.sent();
+                                    return [
+                                        2
+                                    ];
+                            }
+                        });
+                    })();
                 });
                 test.itShouldFail('if the path points to a different type/collection', function() {
-                    test.expectFail(function() {
-                        mockItemFirestoreDocumentAccessor.loadDocumentForKey('path/id');
-                    });
+                    return _async_to_generator$4(function() {
+                        return _ts_generator$4(this, function(_state) {
+                            switch(_state.label){
+                                case 0:
+                                    return [
+                                        4,
+                                        test.expectFail(function() {
+                                            mockItemFirestoreDocumentAccessor.loadDocumentForKey('path/id');
+                                        })
+                                    ];
+                                case 1:
+                                    _state.sent();
+                                    return [
+                                        2
+                                    ];
+                            }
+                        });
+                    })();
                 });
                 test.itShouldFail('if the path is empty.', function() {
-                    test.expectFail(function() {
-                        mockItemFirestoreDocumentAccessor.loadDocumentForKey('');
-                    });
+                    return _async_to_generator$4(function() {
+                        return _ts_generator$4(this, function(_state) {
+                            switch(_state.label){
+                                case 0:
+                                    return [
+                                        4,
+                                        test.expectFail(function() {
+                                            mockItemFirestoreDocumentAccessor.loadDocumentForKey('');
+                                        })
+                                    ];
+                                case 1:
+                                    _state.sent();
+                                    return [
+                                        2
+                                    ];
+                            }
+                        });
+                    })();
                 });
                 test.itShouldFail('if the path is undefined.', function() {
-                    test.expectFail(function() {
-                        mockItemFirestoreDocumentAccessor.loadDocumentForKey(undefined);
-                    });
+                    return _async_to_generator$4(function() {
+                        return _ts_generator$4(this, function(_state) {
+                            switch(_state.label){
+                                case 0:
+                                    return [
+                                        4,
+                                        test.expectFail(function() {
+                                            mockItemFirestoreDocumentAccessor.loadDocumentForKey(undefined);
+                                        })
+                                    ];
+                                case 1:
+                                    _state.sent();
+                                    return [
+                                        2
+                                    ];
+                            }
+                        });
+                    })();
                 });
                 test.itShouldFail('if the path is null.', function() {
-                    test.expectFail(function() {
-                        mockItemFirestoreDocumentAccessor.loadDocumentForKey(null);
-                    });
+                    return _async_to_generator$4(function() {
+                        return _ts_generator$4(this, function(_state) {
+                            switch(_state.label){
+                                case 0:
+                                    return [
+                                        4,
+                                        test.expectFail(function() {
+                                            mockItemFirestoreDocumentAccessor.loadDocumentForKey(null);
+                                        })
+                                    ];
+                                case 1:
+                                    _state.sent();
+                                    return [
+                                        2
+                                    ];
+                            }
+                        });
+                    })();
                 });
             });
             describe('loadDocumentForId()', function() {
@@ -3566,14 +3634,44 @@ function _ts_generator$4(thisArg, body) {
                     expect(document).toBeDefined();
                 });
                 test.itShouldFail('if the id is empty.', function() {
-                    test.expectFail(function() {
-                        mockItemFirestoreDocumentAccessor.loadDocumentForId('');
-                    });
+                    return _async_to_generator$4(function() {
+                        return _ts_generator$4(this, function(_state) {
+                            switch(_state.label){
+                                case 0:
+                                    return [
+                                        4,
+                                        test.expectFail(function() {
+                                            mockItemFirestoreDocumentAccessor.loadDocumentForId('');
+                                        })
+                                    ];
+                                case 1:
+                                    _state.sent();
+                                    return [
+                                        2
+                                    ];
+                            }
+                        });
+                    })();
                 });
                 test.itShouldFail('if the id is undefined.', function() {
-                    test.expectFail(function() {
-                        mockItemFirestoreDocumentAccessor.loadDocumentForId(undefined);
-                    });
+                    return _async_to_generator$4(function() {
+                        return _ts_generator$4(this, function(_state) {
+                            switch(_state.label){
+                                case 0:
+                                    return [
+                                        4,
+                                        test.expectFail(function() {
+                                            mockItemFirestoreDocumentAccessor.loadDocumentForId(undefined);
+                                        })
+                                    ];
+                                case 1:
+                                    _state.sent();
+                                    return [
+                                        2
+                                    ];
+                            }
+                        });
+                    })();
                 });
             });
         });
@@ -3991,17 +4089,12 @@ function _ts_generator$4(thisArg, body) {
                                                 return _ts_generator$4(this, function(_state) {
                                                     switch(_state.label){
                                                         case 0:
-                                                            return [
-                                                                4,
-                                                                c.loadDocumentForTransaction(transaction, firestoreDocument.documentRef)
-                                                            ];
-                                                        case 1:
-                                                            transactionDocument = _state.sent();
+                                                            transactionDocument = c.loadDocumentForTransaction(transaction, firestoreDocument.documentRef);
                                                             return [
                                                                 4,
                                                                 transactionDocument.snapshotData()
                                                             ];
-                                                        case 2:
+                                                        case 1:
                                                             currentData = _state.sent();
                                                             expect(currentData).toBeDefined();
                                                             data = c.dataForUpdate();
@@ -4009,14 +4102,10 @@ function _ts_generator$4(thisArg, body) {
                                                                 4,
                                                                 transactionDocument.update(data)
                                                             ];
-                                                        case 3:
+                                                        case 2:
                                                             _state.sent();
-                                                            return [
-                                                                4,
-                                                                c.loadDocumentForTransaction(transaction, firestoreDocument.documentRef)
-                                                            ];
-                                                        case 4:
-                                                            secondLoading = _state.sent();
+                                                            // stream$ and data$ do not call stream() until called directly.
+                                                            secondLoading = c.loadDocumentForTransaction(transaction, firestoreDocument.documentRef);
                                                             expect(secondLoading).toBeDefined();
                                                             return [
                                                                 2
@@ -4049,17 +4138,12 @@ function _ts_generator$4(thisArg, body) {
                                                     return _ts_generator$4(this, function(_state) {
                                                         switch(_state.label){
                                                             case 0:
-                                                                return [
-                                                                    4,
-                                                                    c.loadDocumentForTransaction(transaction, firestoreDocument.documentRef)
-                                                                ];
-                                                            case 1:
-                                                                transactionDocument = _state.sent();
+                                                                transactionDocument = c.loadDocumentForTransaction(transaction, firestoreDocument.documentRef);
                                                                 return [
                                                                     4,
                                                                     transactionDocument.snapshotData()
                                                                 ];
-                                                            case 2:
+                                                            case 1:
                                                                 currentData = _state.sent();
                                                                 expect(currentData).toBeDefined();
                                                                 data = c.dataForUpdate();
@@ -4067,14 +4151,14 @@ function _ts_generator$4(thisArg, body) {
                                                                     4,
                                                                     transactionDocument.update(data)
                                                                 ];
-                                                            case 3:
+                                                            case 2:
                                                                 _state.sent();
                                                                 // read the stream using a promise so the error is captured
                                                                 return [
                                                                     4,
                                                                     rxjs$1.firstValueFrom(c.loadDocumentForTransaction(transaction, firestoreDocument.documentRef).stream$)
                                                                 ];
-                                                            case 4:
+                                                            case 3:
                                                                 _state.sent();
                                                                 return [
                                                                     2
@@ -4110,17 +4194,12 @@ function _ts_generator$4(thisArg, body) {
                                                 return _ts_generator$4(this, function(_state) {
                                                     switch(_state.label){
                                                         case 0:
-                                                            return [
-                                                                4,
-                                                                c.loadDocumentForTransaction(transaction, firestoreDocument.documentRef)
-                                                            ];
-                                                        case 1:
-                                                            transactionDocument = _state.sent();
+                                                            transactionDocument = c.loadDocumentForTransaction(transaction, firestoreDocument.documentRef);
                                                             return [
                                                                 4,
                                                                 transactionDocument.snapshotData()
                                                             ];
-                                                        case 2:
+                                                        case 1:
                                                             currentData = _state.sent();
                                                             expect(currentData).toBeDefined();
                                                             data = c.dataForUpdate();
@@ -4128,7 +4207,7 @@ function _ts_generator$4(thisArg, body) {
                                                                 4,
                                                                 transactionDocument.update(data)
                                                             ];
-                                                        case 3:
+                                                        case 2:
                                                             _state.sent();
                                                             return [
                                                                 2
@@ -4169,17 +4248,12 @@ function _ts_generator$4(thisArg, body) {
                                                     return _ts_generator$4(this, function(_state) {
                                                         switch(_state.label){
                                                             case 0:
-                                                                return [
-                                                                    4,
-                                                                    c.loadDocumentForTransaction(transaction, firestoreDocument.documentRef)
-                                                                ];
-                                                            case 1:
-                                                                transactionDocument = _state.sent();
+                                                                transactionDocument = c.loadDocumentForTransaction(transaction, firestoreDocument.documentRef);
                                                                 return [
                                                                     4,
                                                                     transactionDocument.snapshotData()
                                                                 ];
-                                                            case 2:
+                                                            case 1:
                                                                 currentData = _state.sent();
                                                                 expect(currentData).toBeDefined();
                                                                 firstData = c.dataForFirstOfTwoUpdates();
@@ -4187,14 +4261,14 @@ function _ts_generator$4(thisArg, body) {
                                                                     4,
                                                                     transactionDocument.update(firstData)
                                                                 ];
-                                                            case 3:
+                                                            case 2:
                                                                 _state.sent();
                                                                 data = c.dataForUpdate();
                                                                 return [
                                                                     4,
                                                                     transactionDocument.update(data)
                                                                 ];
-                                                            case 4:
+                                                            case 3:
                                                                 _state.sent();
                                                                 return [
                                                                     2
@@ -4235,30 +4309,25 @@ function _ts_generator$4(thisArg, body) {
                             switch(_state.label){
                                 case 0:
                                     batch = c.context.batch();
-                                    return [
-                                        4,
-                                        c.loadDocumentForWriteBatch(batch, firestoreDocument.documentRef)
-                                    ];
-                                case 1:
-                                    batchDocument = _state.sent();
+                                    batchDocument = c.loadDocumentForWriteBatch(batch, firestoreDocument.documentRef);
                                     data = c.dataForUpdate();
                                     return [
                                         4,
                                         batchDocument.update(data)
                                     ];
-                                case 2:
+                                case 1:
                                     _state.sent();
                                     return [
                                         4,
                                         batch.commit()
                                     ];
-                                case 3:
+                                case 2:
                                     _state.sent();
                                     return [
                                         4,
                                         firestoreDocument.snapshot()
                                     ];
-                                case 4:
+                                case 3:
                                     snapshot = _state.sent();
                                     expect(c.hasDataFromUpdate(snapshot.data())).toBe(true);
                                     return [
@@ -4274,24 +4343,8 @@ function _ts_generator$4(thisArg, body) {
     describe('accessor', function() {
         describe('stream()', function() {
             it('should return a snapshot stream', function() {
-                return _async_to_generator$4(function() {
-                    var result;
-                    return _ts_generator$4(this, function(_state) {
-                        switch(_state.label){
-                            case 0:
-                                return [
-                                    4,
-                                    accessor.stream()
-                                ];
-                            case 1:
-                                result = _state.sent();
-                                expect(result).toBeDefined();
-                                return [
-                                    2
-                                ];
-                        }
-                    });
-                })();
+                var result = accessor.stream();
+                expect(result).toBeDefined();
             });
             it('should emit values on updates from the observable.', test.callbackTest(function(done) {
                 var count = 0;
@@ -4305,7 +4358,7 @@ function _ts_generator$4(thisArg, body) {
                     }
                 });
                 setTimeout(function() {
-                    accessor.update(c.dataForUpdate());
+                    void accessor.update(c.dataForUpdate());
                 }, 100);
             }));
             describe('in transition context', function() {
@@ -5100,10 +5153,9 @@ function _ts_generator$3(thisArg, body) {
                                 factory = util.idBatchFactory({
                                     verifier: mockItemIdBatchVerifier(f.instance.mockItemCollection),
                                     factory: function factory(count) {
-                                        var ids = [
+                                        return [
                                             random()
                                         ].concat(_to_consumable_array$1(idFactory(count)));
-                                        return ids;
                                     }
                                 });
                                 idsToMake = 30;
@@ -5535,7 +5587,6 @@ function _ts_generator$3(thisArg, body) {
                                 return _ts_generator$3(this, function(_state) {
                                     switch(_state.label){
                                         case 0:
-                                            f.instance.mockItemUserCollectionGroup.documentAccessor();
                                             mockUserItemsVisited = new Set();
                                             batchSize = 2;
                                             return [
@@ -5761,7 +5812,7 @@ function _ts_generator$3(thisArg, body) {
                                                         batchSize: batchSize,
                                                         limitPerCheckpoint: 1,
                                                         maxParallelCheckpoints: maxParallelCheckpoints,
-                                                        iterateSnapshotBatch: function iterateSnapshotBatch(x, batchIndex) {
+                                                        iterateSnapshotBatch: function iterateSnapshotBatch(_x, _batchIndex) {
                                                             return _async_to_generator$3(function() {
                                                                 return _ts_generator$3(this, function(_state) {
                                                                     switch(_state.label){
@@ -5992,12 +6043,10 @@ function _ts_generator$3(thisArg, body) {
                 var queryDeepSubItems;
                 beforeEach(function() {
                     return _async_to_generator$3(function() {
-                        var results;
                         return _ts_generator$3(this, function(_state) {
                             switch(_state.label){
                                 case 0:
                                     queryDeepSubItems = f.instance.mockItemSubItemDeepCollectionGroup.query;
-                                    allSubItems[0];
                                     return [
                                         4,
                                         Promise.all(allSubItems.map(function(parent) {
@@ -6012,8 +6061,7 @@ function _ts_generator$3(thisArg, body) {
                                         }))
                                     ];
                                 case 1:
-                                    results = _state.sent();
-                                    results.flat();
+                                    _state.sent();
                                     return [
                                         2
                                     ];
@@ -6164,7 +6212,7 @@ function _ts_generator$3(thisArg, body) {
                                     tryComplete();
                                 });
                                 // add one item
-                                firebase.makeDocuments(f.instance.mockItemSubItemCollection(parentA).documentAccessor(), {
+                                void firebase.makeDocuments(f.instance.mockItemSubItemCollection(parentA).documentAccessor(), {
                                     count: itemsToAdd,
                                     init: function init(i) {
                                         return {
@@ -6192,7 +6240,7 @@ function _ts_generator$3(thisArg, body) {
                                     expect(results.docs.length).toBe(allSubItems.length - itemsToRemove);
                                     tryComplete();
                                 });
-                                allSubItems[0].accessor.exists().then(function(exists) {
+                                void allSubItems[0].accessor.exists().then(function(exists) {
                                     expect(exists).toBe(true);
                                     // remove one item
                                     return allSubItems[0].accessor.delete().then(function() {
@@ -6412,7 +6460,7 @@ function _ts_generator$3(thisArg, body) {
                         tryComplete();
                     });
                     // add one item
-                    util.waitForMs(10).then(function() {
+                    void util.waitForMs(10).then(function() {
                         return firebase.makeDocuments(f.instance.mockItemCollection.documentAccessor(), {
                             count: itemsToAdd,
                             init: function init(i) {
@@ -6441,7 +6489,7 @@ function _ts_generator$3(thisArg, body) {
                         expect(documents.length).toBe(items.length - itemsToRemove);
                         tryComplete();
                     });
-                    util.waitForMs(10).then(function() {
+                    void util.waitForMs(10).then(function() {
                         return items[0].exists().then(function(exists) {
                             expect(exists).toBe(true);
                             // remove one item
@@ -6488,7 +6536,7 @@ function _ts_generator$3(thisArg, body) {
                         tryComplete();
                     });
                     // add one item
-                    util.waitForMs(10).then(function() {
+                    void util.waitForMs(10).then(function() {
                         return firebase.makeDocuments(f.instance.mockItemCollection.documentAccessor(), {
                             count: itemsToAdd,
                             init: function init(i) {
@@ -6517,7 +6565,7 @@ function _ts_generator$3(thisArg, body) {
                         expect(documents.length).toBe(items.length - itemsToRemove);
                         tryComplete();
                     });
-                    util.waitForMs(10).then(function() {
+                    void util.waitForMs(10).then(function() {
                         return items[0].exists().then(function(exists) {
                             expect(exists).toBe(true);
                             // remove one item
@@ -6567,7 +6615,7 @@ function _ts_generator$3(thisArg, body) {
                         tryComplete();
                     });
                     // add one item
-                    util.waitForMs(10).then(function() {
+                    void util.waitForMs(10).then(function() {
                         return firebase.makeDocuments(f.instance.mockItemCollection.documentAccessor(), {
                             count: itemsToAdd,
                             init: function init(i) {
@@ -6596,7 +6644,7 @@ function _ts_generator$3(thisArg, body) {
                         expect(results.docs.length).toBe(items.length - itemsToRemove);
                         tryComplete();
                     });
-                    util.waitForMs(10).then(function() {
+                    void util.waitForMs(10).then(function() {
                         return items[0].accessor.exists().then(function(exists) {
                             expect(exists).toBe(true);
                             // remove one item
@@ -8489,7 +8537,7 @@ function _ts_generator$2(thisArg, body) {
                 });
                 it('should preserve ordering of the input array', function() {
                     return _async_to_generator$2(function() {
-                        var snapshots, i;
+                        var snapshots, _iteratorNormalCompletion, _didIteratorError, _iteratorError, _iterator, _step, _step_value, i, item;
                         return _ts_generator$2(this, function(_state) {
                             switch(_state.label){
                                 case 0:
@@ -8499,8 +8547,25 @@ function _ts_generator$2(thisArg, body) {
                                     ];
                                 case 1:
                                     snapshots = _state.sent();
-                                    for(i = 0; i < items.length; i++){
-                                        expect(snapshots[i].id).toBe(items[i].id);
+                                    _iteratorNormalCompletion = true, _didIteratorError = false, _iteratorError = undefined;
+                                    try {
+                                        for(_iterator = items.entries()[Symbol.iterator](); !(_iteratorNormalCompletion = (_step = _iterator.next()).done); _iteratorNormalCompletion = true){
+                                            _step_value = _sliced_to_array(_step.value, 2), i = _step_value[0], item = _step_value[1];
+                                            expect(snapshots[i].id).toBe(item.id);
+                                        }
+                                    } catch (err) {
+                                        _didIteratorError = true;
+                                        _iteratorError = err;
+                                    } finally{
+                                        try {
+                                            if (!_iteratorNormalCompletion && _iterator.return != null) {
+                                                _iterator.return();
+                                            }
+                                        } finally{
+                                            if (_didIteratorError) {
+                                                throw _iteratorError;
+                                            }
+                                        }
                                     }
                                     return [
                                         2
@@ -10040,9 +10105,9 @@ function _ts_generator$1(thisArg, body) {
                         });
                         var accumulator = firebase.firebaseQuerySnapshotAccumulator(iteration);
                         // First page should return all items but not yet know it's the end
-                        iteration.nextPage().then(function() {
+                        void iteration.nextPage().then(function() {
                             // Load the next page which should discover end via empty snapshot
-                            iteration.nextPage().then(function() {
+                            void iteration.nextPage().then(function() {
                                 sub.subscription = iteration.latestState$.pipe(rxjs$1.filter(function(x) {
                                     return rxjs.isLoadingStateFinishedLoading(x);
                                 }), rxjs$1.first()).subscribe(function(state) {
@@ -10067,7 +10132,7 @@ function _ts_generator$1(thisArg, body) {
                         });
                         var accumulator = firebase.firebaseQuerySnapshotAccumulator(iteration);
                         // Load pages until the iterator reaches the end
-                        rxjs.iteratorNextPageUntilPage(iteration, expectedPages + 1).then(function(page) {
+                        void rxjs.iteratorNextPageUntilPage(iteration, expectedPages + 1).then(function(_page) {
                             sub.subscription = iteration.latestState$.pipe(rxjs$1.filter(function(x) {
                                 return rxjs.isLoadingStateFinishedLoading(x);
                             }), rxjs$1.first()).subscribe(function(state) {
@@ -10151,9 +10216,9 @@ function _ts_generator$1(thisArg, body) {
             });
             describe('nextPage()', function() {
                 it('should load the next page and return when the page has finished loading.', test.callbackTest(function(done) {
-                    iteration.nextPage().then(function() {
+                    void iteration.nextPage().then(function() {
                         var nextPageResult = rxjs$1.from(iteration.nextPage());
-                        sub.subscription = nextPageResult.pipe(rxjs$1.switchMap(function(x) {
+                        sub.subscription = nextPageResult.pipe(rxjs$1.switchMap(function(_x) {
                             return iteration.currentState$;
                         })).subscribe(function(latestState) {
                             var page = latestState.page;
@@ -10185,7 +10250,7 @@ function _ts_generator$1(thisArg, body) {
                         it("should aggregate the array of results into a single array.", test.callbackTest(function(done) {
                             var pagesToLoad = 2;
                             // load up to page 2
-                            rxjs.iteratorNextPageUntilPage(iteration, pagesToLoad).then(function(page) {
+                            void rxjs.iteratorNextPageUntilPage(iteration, pagesToLoad).then(function(page) {
                                 expect(page).toBe(pagesToLoad - 1);
                                 var obs = rxjs.flattenAccumulatorResultItemArray(accumulator);
                                 accumulatorSub.subscription = obs.pipe(rxjs$1.first()).subscribe(function(values) {
@@ -10225,7 +10290,7 @@ function _ts_generator$1(thisArg, body) {
                         it("should aggregate the array of results into a single array.", test.callbackTest(function(done) {
                             var pagesToLoad = 2;
                             // load up to page 2
-                            rxjs.iteratorNextPageUntilPage(iteration, pagesToLoad).then(function(page) {
+                            void rxjs.iteratorNextPageUntilPage(iteration, pagesToLoad).then(function(page) {
                                 expect(page).toBe(pagesToLoad - 1);
                                 var obs = rxjs.flattenAccumulatorResultItemArray(itemAccumulator);
                                 accumulatorSub.subscription = obs.pipe(rxjs$1.first()).subscribe(function(values) {
@@ -10242,7 +10307,7 @@ function _ts_generator$1(thisArg, body) {
                         it("should aggregate the array of results into a single array of the items.", test.callbackTest(function(done) {
                             var pagesToLoad = 2;
                             // load up to page 2
-                            rxjs.iteratorNextPageUntilPage(iteration, pagesToLoad).then(function(page) {
+                            void rxjs.iteratorNextPageUntilPage(iteration, pagesToLoad).then(function(page) {
                                 expect(page).toBe(pagesToLoad - 1);
                                 var obs = rxjs.flattenAccumulatorResultItemArray(itemAccumulator);
                                 accumulatorSub.subscription = obs.pipe(rxjs$1.first()).subscribe(function(values) {
@@ -12116,17 +12181,12 @@ function _ts_generator(thisArg, body) {
                                     return _ts_generator(this, function(_state) {
                                         switch(_state.label){
                                             case 0:
-                                                return [
-                                                    4,
-                                                    f.storageContext.folder('/')
-                                                ];
-                                            case 1:
-                                                rootFolder = _state.sent();
+                                                rootFolder = f.storageContext.folder('/');
                                                 return [
                                                     4,
                                                     rootFolder.list()
                                                 ];
-                                            case 2:
+                                            case 1:
                                                 result = _state.sent();
                                                 expect(result).toBeDefined();
                                                 files = result.files();
@@ -12187,19 +12247,14 @@ function _ts_generator(thisArg, body) {
                                     return _ts_generator(this, function(_state) {
                                         switch(_state.label){
                                             case 0:
-                                                return [
-                                                    4,
-                                                    f.storageContext.folder('/')
-                                                ];
-                                            case 1:
-                                                rootFolder = _state.sent();
+                                                rootFolder = f.storageContext.folder('/');
                                                 return [
                                                     4,
                                                     rootFolder.list({
                                                         includeNestedResults: true
                                                     })
                                                 ];
-                                            case 2:
+                                            case 1:
                                                 result = _state.sent();
                                                 expect(result).toBeDefined();
                                                 files = result.files();
@@ -12220,12 +12275,7 @@ function _ts_generator(thisArg, body) {
                                         return _ts_generator(this, function(_state) {
                                             switch(_state.label){
                                                 case 0:
-                                                    return [
-                                                        4,
-                                                        f.storageContext.folder('/')
-                                                    ];
-                                                case 1:
-                                                    rootFolder = _state.sent();
+                                                    rootFolder = f.storageContext.folder('/');
                                                     limit = 2;
                                                     return [
                                                         4,
@@ -12234,12 +12284,12 @@ function _ts_generator(thisArg, body) {
                                                             maxResults: limit
                                                         })
                                                     ];
-                                                case 2:
+                                                case 1:
                                                     result = _state.sent();
                                                     expect(result).toBeDefined();
                                                     if (!(f.storageContext.drivers.storageAccessorDriver.type === 'server')) return [
                                                         3,
-                                                        4
+                                                        3
                                                     ];
                                                     // Currently only the server can properly limit the number of results returned.
                                                     // The client-side will limit the results somewhat, but if folders are returned then it will return the results of those folders as well.
@@ -12249,12 +12299,12 @@ function _ts_generator(thisArg, body) {
                                                         4,
                                                         result.next()
                                                     ];
-                                                case 3:
+                                                case 2:
                                                     nextPage = _state.sent();
                                                     nextPageFiles = nextPage.files();
                                                     expect(nextPageFiles.length).toBe(limit);
-                                                    _state.label = 4;
-                                                case 4:
+                                                    _state.label = 3;
+                                                case 3:
                                                     folders = result.folders();
                                                     expect(folders.length).toBe(0);
                                                     return [
@@ -12309,17 +12359,12 @@ function _ts_generator(thisArg, body) {
                             return _ts_generator(this, function(_state) {
                                 switch(_state.label){
                                     case 0:
-                                        return [
-                                            4,
-                                            f.storageContext.folder('/')
-                                        ];
-                                    case 1:
-                                        rootFolder = _state.sent();
+                                        rootFolder = f.storageContext.folder('/');
                                         return [
                                             4,
                                             rootFolder.list()
                                         ];
-                                    case 2:
+                                    case 1:
                                         result = _state.sent();
                                         expect(result).toBeDefined();
                                         folders = result.folders();
@@ -12331,7 +12376,7 @@ function _ts_generator(thisArg, body) {
                                             4,
                                             folder.exists()
                                         ];
-                                    case 3:
+                                    case 2:
                                         exists = _state.sent();
                                         expect(exists).toBe(true);
                                         return [
@@ -12350,19 +12395,14 @@ function _ts_generator(thisArg, body) {
                                 switch(_state.label){
                                     case 0:
                                         maxResults = 1;
-                                        return [
-                                            4,
-                                            f.storageContext.folder(existsFolderPath)
-                                        ];
-                                    case 1:
-                                        rootFolder = _state.sent();
+                                        rootFolder = f.storageContext.folder(existsFolderPath);
                                         return [
                                             4,
                                             rootFolder.list({
                                                 maxResults: maxResults
                                             })
                                         ];
-                                    case 2:
+                                    case 1:
                                         result = _state.sent();
                                         expect(result).toBeDefined();
                                         files = result.files();
@@ -12371,7 +12411,7 @@ function _ts_generator(thisArg, body) {
                                             4,
                                             result.next()
                                         ];
-                                    case 3:
+                                    case 2:
                                         next = _state.sent();
                                         expect(next).toBeDefined();
                                         nextFiles = next.files();
@@ -12391,17 +12431,12 @@ function _ts_generator(thisArg, body) {
                             return _ts_generator(this, function(_state) {
                                 switch(_state.label){
                                     case 0:
-                                        return [
-                                            4,
-                                            f.storageContext.folder(existsFolderPath)
-                                        ];
-                                    case 1:
-                                        rootFolder = _state.sent();
+                                        rootFolder = f.storageContext.folder(existsFolderPath);
                                         return [
                                             4,
                                             rootFolder.list({})
                                         ];
-                                    case 2:
+                                    case 1:
                                         result = _state.sent();
                                         expect(result.hasNext).toBe(false);
                                         return [
@@ -12410,7 +12445,7 @@ function _ts_generator(thisArg, body) {
                                                 return result.next();
                                             })
                                         ];
-                                    case 3:
+                                    case 2:
                                         _state.sent();
                                         return [
                                             2
@@ -12428,19 +12463,14 @@ function _ts_generator(thisArg, body) {
                                 switch(_state.label){
                                     case 0:
                                         maxResults = 1;
-                                        return [
-                                            4,
-                                            f.storageContext.folder(existsFolderPath)
-                                        ];
-                                    case 1:
-                                        rootFolder = _state.sent();
+                                        rootFolder = f.storageContext.folder(existsFolderPath);
                                         return [
                                             4,
                                             rootFolder.list({
                                                 maxResults: maxResults
                                             })
                                         ];
-                                    case 2:
+                                    case 1:
                                         result = _state.sent();
                                         expect(result).toBeDefined();
                                         files = result.files();
@@ -12465,19 +12495,14 @@ function _ts_generator(thisArg, body) {
                                 switch(_state.label){
                                     case 0:
                                         maxResults = 1;
-                                        return [
-                                            4,
-                                            f.storageContext.folder('/')
-                                        ];
-                                    case 1:
-                                        rootFolder = _state.sent();
+                                        rootFolder = f.storageContext.folder('/');
                                         return [
                                             4,
                                             rootFolder.list({
                                                 maxResults: maxResults
                                             })
                                         ];
-                                    case 2:
+                                    case 1:
                                         result = _state.sent();
                                         expect(result).toBeDefined();
                                         files = result.files();