fix(firestore, android): remove usage of Guava library WIP

there was only one usage of Guava in firestore, and it was easily
replaced with a for loop which eliminates the dependency

this is important in the context of firebase-android-sdk where guava
is no longer transitively available - best to remove it here as well
vs bring it in directly as a dependency we specifically we need

WIP added logging to guava removed / new-algo in firestore serializer

Author: mikehardy

packages/firestore/android/src/reactnative/java/io/invertase/firebase/firestore/ReactNativeFirebaseFirestoreSerialize.java

@@ -28,7 +28,6 @@
 import com.facebook.react.bridge.ReadableMapKeySetIterator;
 import com.facebook.react.bridge.WritableArray;
 import com.facebook.react.bridge.WritableMap;
-import com.google.common.collect.Iterables;
 import com.google.firebase.Timestamp;
 import com.google.firebase.firestore.Blob;
 import com.google.firebase.firestore.DocumentChange;
@@ -193,23 +192,31 @@ private static WritableArray documentChangesToWritableArray(
       @Nullable List<DocumentChange> comparableDocumentChanges) {
     WritableArray documentChangesWritable = Arguments.createArray();
 
+    Log.w(TAG, "SDK33 in documentChangesToWritableArray");
+
     boolean checkIfMetadataChange = comparableDocumentChanges != null;
+    Log.w(TAG, "SDK33 checkIfMetadataChange is " + checkIfMetadataChange);
+    Log.w(TAG, "SDK33 documentChanges size: " + documentChanges.size());
 
     for (DocumentChange documentChange : documentChanges) {
-      boolean isMetadataChange = false;
+      boolean isMetadataChange = true;
       if (checkIfMetadataChange) {
         int hashCode = documentChange.hashCode();
-        DocumentChange exists =
-            Iterables.tryFind(
-                    comparableDocumentChanges, docChange -> docChange.hashCode() == hashCode)
-                .orNull();
-
-        // Exists in docChanges with meta, but doesnt exist in docChanges without meta
-        if (exists == null) {
-          isMetadataChange = true;
+        Log.w(TAG, "SDK33 examining change with hashCode " + hashCode);
+        Log.w(TAG, "SDK33 comp changes size " + comparableDocumentChanges.size());
+        // is a metadata-only change if exists in docChanges with meta,
+        // but doesnt exist in docChanges without meta
+        for (DocumentChange comparableDocumentChange : comparableDocumentChanges) {
+          Log.w(TAG, "SDK33 comp change with hashCode " + hashCode);
+          if (comparableDocumentChange.hashCode() == hashCode) {
+            isMetadataChange = false;
+            Log.w(TAG, "SDK33 we have a match! it is not just a metadata change");
+            break;
+          }
         }
       }
 
+      Log.w(TAG, "SDK33 push doc " + documentChange.hashCode() + " isMetadataChange: " + isMetadataChange);
       documentChangesWritable.pushMap(
           documentChangeToWritableMap(appName, documentChange, isMetadataChange));
     }

tests/e2e/.mocharc.js

@@ -5,27 +5,27 @@ module.exports = {
   timeout: 1000000,
   reporter: 'spec',
   slow: 2000,
-  retries: 4,
+  retries: 0,
   bail: true,
   exit: true,
   require: 'node_modules/jet/platform/node',
   spec: [
-    '../packages/app/e2e/**/*.e2e.js',
-    '../packages/app-check/e2e/**/*.e2e.js',
-    '../packages/app-distribution/e2e/**/*.e2e.js',
-    '../packages/analytics/e2e/**/*.e2e.js',
-    '../packages/auth/e2e/**/*.e2e.js',
-    '../packages/crashlytics/e2e/**/*.e2e.js',
-    '../packages/database/e2e/**/*.e2e.js',
-    '../packages/dynamic-links/e2e/**/*.e2e.js',
+    // '../packages/app/e2e/**/*.e2e.js',
+    // '../packages/app-check/e2e/**/*.e2e.js',
+    // '../packages/app-distribution/e2e/**/*.e2e.js',
+    // '../packages/analytics/e2e/**/*.e2e.js',
+    // '../packages/auth/e2e/**/*.e2e.js',
+    // '../packages/crashlytics/e2e/**/*.e2e.js',
+    // '../packages/database/e2e/**/*.e2e.js',
+    // '../packages/dynamic-links/e2e/**/*.e2e.js',
     '../packages/firestore/e2e/**/*.e2e.js',
-    '../packages/functions/e2e/**/*.e2e.js',
-    '../packages/perf/e2e/**/*.e2e.js',
-    '../packages/messaging/e2e/**/*.e2e.js',
-    '../packages/ml/e2e/**/*.e2e.js',
-    '../packages/in-app-messaging/e2e/**/*.e2e.js',
-    '../packages/installations/e2e/**/*.e2e.js',
-    '../packages/remote-config/e2e/**/*.e2e.js',
-    '../packages/storage/e2e/**/*.e2e.js',
+    // '../packages/functions/e2e/**/*.e2e.js',
+    // '../packages/perf/e2e/**/*.e2e.js',
+    // '../packages/messaging/e2e/**/*.e2e.js',
+    // '../packages/ml/e2e/**/*.e2e.js',
+    // '../packages/in-app-messaging/e2e/**/*.e2e.js',
+    // '../packages/installations/e2e/**/*.e2e.js',
+    // '../packages/remote-config/e2e/**/*.e2e.js',
+    // '../packages/storage/e2e/**/*.e2e.js',
   ],
 };