Comment cleanup

Author: schmidt-sebastian

packages/firestore/src/local/tab_notification_channel.ts

@@ -62,12 +62,12 @@ export interface TabNotificationChannel {
   getAllActiveQueryTargets(): Set<TargetId>;
 
   /**
-   * Starts the TabNotificationChannel,  reads existing instance data for all
+   * Starts the TabNotificationChannel, reads existing instance data for all
    * `knownInstances` and registers listeners for newly added instances.
    */
   start(knownInstances: InstanceKey[]): void;
 
-  /** Shutdown the `TabNotificationChannel` and its listeners. */
+  /** Shuts down the `TabNotificationChannel` and its listeners. */
   shutdown(): void;
 }
 

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

@@ -87,15 +87,13 @@ describe('LocalStorageNotificationChannel', () => {
         });
     });
 
-    it('with one batch', () => {
-      notificationChannel.addPendingMutation(0);
-      assertInstanceState([], 0, 0);
+    it('with empty batch', () => {
+      assertInstanceState([]);
     });
 
-    it('with empty batch', () => {
+    it('with one batch', () => {
       notificationChannel.addPendingMutation(0);
-      notificationChannel.removePendingMutation(0);
-      assertInstanceState([]);
+      assertInstanceState([], 0, 0);
     });
 
     it('with multiple batches', () => {
@@ -123,8 +121,6 @@ describe('LocalStorageNotificationChannel', () => {
     });
 
     it('with empty targets', () => {
-      notificationChannel.addActiveQueryTarget(0);
-      notificationChannel.removeActiveQueryTarget(0);
       assertInstanceState([]);
     });