add todos

Author: wu-hui

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

@@ -102,7 +102,8 @@ apiDescribe('Numeric Transforms:', persistence => {
     });
   });
 
-  // Skipped due to test flakiness: timeout
+  // TODO(b/295872012): This test is skipped due to a timeout test flakiness
+  // We should investigate if this is an acutal bug.
   // eslint-disable-next-line no-restricted-properties
   it.skip('increment existing integer with integer', async () => {
     await withTestSetup(async () => {
@@ -160,8 +161,9 @@ apiDescribe('Numeric Transforms:', persistence => {
     });
   });
 
-  // Skipped due to test flakiness:
+  // TODO(b/295872012): This test is skipped due to test flakiness:
   // AssertionError: expected 0.122 to be close to 0.111 +/- 0.000001
+  // We should investigate the root cause, it might be an acutal bug.
   // eslint-disable-next-line no-restricted-properties
   it.skip('multiple double increments', async () => {
     await withTestSetup(async () => {

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

@@ -444,7 +444,8 @@ apiDescribe('Queries', persistence => {
     });
   });
 
-  // Skips because the checks on hasPendingWrites are flaky.
+  // TODO(b/295872012): This test is skipped due to the flakiness around the checks of hasPendingWrites.
+  // We should investigate if this is an acutal bug.
   // eslint-disable-next-line no-restricted-properties
   it.skip('can listen for the same query with different options', () => {
     const testDocs = { a: { v: 'a' }, b: { v: 'b' } };