Add assert

Author: schmidt-sebastian

packages/firestore/src/local/indexeddb_index_manager.ts

@@ -857,6 +857,10 @@ export class IndexedDbIndexManager implements IndexManager {
     // that we would split an existing range into multiple ranges that exclude
     // the values from any notIn filter.
     for (const indexRange of indexRanges) {
+      debugAssert(
+        indexRange.lower[0] === indexRange.upper[0],
+        'Index ID must match for index range'
+      );
       const lowerBound = new Uint8Array(indexRange.lower[3]);
       const upperBound = new Uint8Array(indexRange.upper[3]);
 
@@ -873,7 +877,7 @@ export class IndexedDbIndexManager implements IndexManager {
           ranges.push(
             IDBKeyRange.bound(
               this.generateBound(indexRange.lower, lastBound),
-              this.generateBound(indexRange.lower, currentBound),
+              this.generateBound(indexRange.upper, currentBound),
               lastOpen,
               /* upperOpen= */ indexRange.upperOpen
             )

packages/firestore/test/unit/local/index_manager.test.ts

@@ -446,6 +446,18 @@ describe('IndexedDbIndexManager', async () => {
     await verifyResults(q, 'coll/val2', 'coll/val3');
   });
 
+  it('applies startAt filter with notIn', async () => {
+    await setUpSingleValueFilter();
+    const q = queryWithStartAt(
+      queryWithAddedOrderBy(
+        queryWithAddedFilter(query('coll'), filter('count', '!=', 2)),
+        orderBy('count')
+      ),
+      bound([2], true)
+    );
+    await verifyResults(q, 'coll/val3');
+  });
+
   it('applies startAfter filter', async () => {
     await setUpSingleValueFilter();
     const q = queryWithStartAt(