Merge 50aaaf4 into a192efb

Author: dconeybe

firestore/integration_test_internal/src/bundle_test.cc

@@ -142,6 +142,9 @@ class BundleTest : public FirestoreIntegrationTest {
 };
 
 TEST_F(BundleTest, CanLoadBundlesWithoutProgressUpdates) {
+  GTEST_SKIP() << "Bundle tests fails often on Android and iOS in GitHub "
+                  "Actions and needs to be investigated (b/233751585)";
+
   Firestore* db = TestFirestore();
   auto bundle = CreateTestBundle(db);
 
@@ -152,6 +155,9 @@ TEST_F(BundleTest, CanLoadBundlesWithoutProgressUpdates) {
 }
 
 TEST_F(BundleTest, CanLoadBundlesWithProgressUpdates) {
+  GTEST_SKIP() << "Bundle tests fails often on Android and iOS in GitHub "
+                  "Actions and needs to be investigated (b/233751585)";
+
   Firestore* db = TestFirestore();
   auto bundle = CreateTestBundle(db);
 
@@ -180,6 +186,9 @@ TEST_F(BundleTest, CanLoadBundlesWithProgressUpdates) {
 }
 
 TEST_F(BundleTest, CanDeleteFirestoreFromProgressUpdate) {
+  GTEST_SKIP() << "Bundle tests fails often on Android and iOS in GitHub "
+                  "Actions and needs to be investigated (b/233751585)";
+
   Firestore* db = TestFirestore();
   auto bundle = CreateTestBundle(db);
 
@@ -216,6 +225,9 @@ TEST_F(BundleTest, CanDeleteFirestoreFromProgressUpdate) {
 }
 
 TEST_F(BundleTest, LoadBundlesForASecondTimeSkips) {
+  GTEST_SKIP() << "Bundle tests fails often on Android and iOS in GitHub "
+                  "Actions and needs to be investigated (b/233751585)";
+
   // TODO(wuandy): This test fails on Windows CI, but
   // local run is fine. We need to figure out why and re-enable it.
   SKIP_TEST_ON_WINDOWS;
@@ -244,6 +256,9 @@ TEST_F(BundleTest, LoadBundlesForASecondTimeSkips) {
 }
 
 TEST_F(BundleTest, LoadInvalidBundlesShouldFail) {
+  GTEST_SKIP() << "Bundle tests fails often on Android and iOS in GitHub "
+                  "Actions and needs to be investigated (b/233751585)";
+
   // TODO(wuandy): This test fails on Windows CI, but
   // local run is fine. We need to figure out why and re-enable it.
   SKIP_TEST_ON_WINDOWS;
@@ -273,6 +288,9 @@ TEST_F(BundleTest, LoadInvalidBundlesShouldFail) {
 }
 
 TEST_F(BundleTest, LoadBundleWithDocumentsAlreadyPulledFromBackend) {
+  GTEST_SKIP() << "Bundle tests fails often on Android and iOS in GitHub "
+                  "Actions and needs to be investigated (b/233751585)";
+
   // TODO(wuandy, b/189477267): This test fails on Windows CI, but
   // local run is fine. We need to figure out why and re-enable it.
   SKIP_TEST_ON_WINDOWS;
@@ -319,6 +337,9 @@ TEST_F(BundleTest, LoadBundleWithDocumentsAlreadyPulledFromBackend) {
 }
 
 TEST_F(BundleTest, LoadedDocumentsShouldNotBeGarbageCollectedRightAway) {
+  GTEST_SKIP() << "Bundle tests fails often on Android and iOS in GitHub "
+                  "Actions and needs to be investigated (b/233751585)";
+
   // TODO(wuandy, b/189477267): This test fails on Windows CI, but
   // local run is fine. We need to figure out why and re-enable it.
   SKIP_TEST_ON_WINDOWS;
@@ -343,6 +364,9 @@ TEST_F(BundleTest, LoadedDocumentsShouldNotBeGarbageCollectedRightAway) {
 }
 
 TEST_F(BundleTest, LoadDocumentsFromOtherProjectsShouldFail) {
+  GTEST_SKIP() << "Bundle tests fails often on Android and iOS in GitHub "
+                  "Actions and needs to be investigated (b/233751585)";
+
   Firestore* db = TestFirestore();
   auto bundle = CreateBundle("other-project");
   std::vector<LoadBundleTaskProgress> progresses;
@@ -363,6 +387,9 @@ TEST_F(BundleTest, LoadDocumentsFromOtherProjectsShouldFail) {
 }
 
 TEST_F(BundleTest, GetInvalidNamedQuery) {
+  GTEST_SKIP() << "Bundle tests fails often on Android and iOS in GitHub "
+                  "Actions and needs to be investigated (b/233751585)";
+
   Firestore* db = TestFirestore();
   {
     auto future = db->NamedQuery("DOES_NOT_EXIST");