Cleanup and some fixes

Author: aashishpatil-g

packages/firebase_data_connect/firebase_data_connect/lib/firebase_data_connect.dart

@@ -39,4 +39,5 @@ export 'src/optional.dart'
         listDeserializer,
         listSerializer;
 export 'src/timestamp.dart' show Timestamp;
-export 'src/cache/cache_data_types.dart' show CacheSettings, QueryFetchPolicy, CacheStorage;
+export 'src/cache/cache_data_types.dart'
+    show CacheSettings, QueryFetchPolicy, CacheStorage;

packages/firebase_data_connect/firebase_data_connect/lib/src/cache/cache_manager.dart

@@ -18,7 +18,8 @@ import 'dart:developer' as developer;
 import 'package:firebase_auth/firebase_auth.dart';
 import 'package:firebase_data_connect/firebase_data_connect.dart';
 import 'cache_provider.dart';
-import 'in_memory_cache_provider.dart' if (dart.library.io) 'sqlite_cache_provider.dart';
+import 'in_memory_cache_provider.dart'
+    if (dart.library.io) 'sqlite_cache_provider.dart';
 
 import '../common/common_library.dart';
 
@@ -110,8 +111,6 @@ class Cache {
 
     Set<String> impactedQueryIds = dehydrationResult.impactedQueryIds;
     impactedQueryIds.remove(queryId); // remove query being cached
-    print(
-        'adding to impactedQueryController ${dehydrationResult.impactedQueryIds}');
     _impactedQueryController.add(impactedQueryIds);
   }
 

packages/firebase_data_connect/firebase_data_connect/lib/src/cache/in_memory_cache_provider.dart

@@ -31,8 +31,7 @@ class InMemoryCacheProvider implements CacheProvider {
 
   @override
   Future<bool> initialize() async {
-    // nothing to be intialized
-    print('Initialize inmemory provider called');
+    // nothing to be intialized.
     return true;
   }
 
@@ -55,7 +54,6 @@ class InMemoryCacheProvider implements CacheProvider {
   EntityDataObject getEntityDataObject(String guid) {
     EntityDataObject? edo = _edos[guid];
     if (edo != null) {
-      print('Returning existing edo for $guid');
       return edo;
     } else {
       edo = EntityDataObject(guid: guid);
@@ -76,4 +74,5 @@ class InMemoryCacheProvider implements CacheProvider {
   }
 }
 
-CacheProvider cacheImplementation(String identifier, bool memory) => InMemoryCacheProvider(identifier);
+CacheProvider cacheImplementation(String identifier, bool memory) =>
+    InMemoryCacheProvider(identifier);

packages/firebase_data_connect/firebase_data_connect/lib/src/cache/sqlite_cache_provider.dart

@@ -126,4 +126,5 @@ class SQLite3CacheProvider implements CacheProvider {
   }
 }
 
-CacheProvider cacheImplementation(String identifier, bool memory) => SQLite3CacheProvider(identifier, memory: memory);
+CacheProvider cacheImplementation(String identifier, bool memory) =>
+    SQLite3CacheProvider(identifier, memory: memory);

packages/firebase_data_connect/firebase_data_connect/lib/src/core/ref.dart

@@ -146,7 +146,6 @@ class QueryManager {
     if (dataConnect.cacheManager != null) {
       _impactedQueriesSubscription =
           dataConnect.cacheManager!.impactedQueries.listen((impactedQueryIds) {
-        print('impactedQueries received updated $impactedQueryIds');
         for (final queryId in impactedQueryIds) {
           final queryParts = queryId.split('-');
           final queryName = queryParts[0];

packages/firebase_data_connect/firebase_data_connect/lib/src/firebase_data_connect.dart

@@ -186,7 +186,7 @@ class FirebaseDataConnect extends FirebasePluginPlatform {
 
     if (cachedInstances[app.name] != null &&
         cachedInstances[app.name]![connectorConfig.toJson()] != null) {
-          print('Returning cached FirebaseDataConnect instance');
+      print('Returning cached FirebaseDataConnect instance');
       return cachedInstances[app.name]![connectorConfig.toJson()]!;
     }
 

packages/firebase_data_connect/firebase_data_connect/test/src/cache/cache_manager_test.dart

@@ -93,7 +93,7 @@ void main() {
       dataConnect = FirebaseDataConnect(
           app: mockApp,
           connectorConfig: mockConnectorConfig,
-          cacheSettings: const CacheSettings(storage: CacheStorage.memory));
+          cacheSettings: CacheSettings(storage: CacheStorage.memory));
       dataConnect.checkTransport();
       dataConnect.checkAndInitializeCache();
     });