Fix git action failures for analyze, test

Removed a test no longer valid

Author: aashishpatil-g

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

@@ -144,14 +144,17 @@ class QueryManager {
     // this is dependent on the cachemanager, which is initialized lazily
     // this should be called whenever cacheManager is initialized.
     if (dataConnect.cacheManager != null) {
-      _impactedQueriesSubscription =
-          dataConnect.cacheManager!.impactedQueries.listen((impactedQueryIds) {
+      _impactedQueriesSubscription = dataConnect.cacheManager!.impactedQueries
+          .listen((impactedQueryIds) async {
         for (final queryId in impactedQueryIds) {
           final queryRef = trackedQueries[queryId];
           if (queryRef != null) {
-            queryRef
-                .execute(fetchPolicy: QueryFetchPolicy.cacheOnly)
-                .catchError((e) => log('Error executing impacted query $e'));
+            try {
+              await queryRef.execute(
+                  fetchPolicy: QueryFetchPolicy.cacheOnly);
+            } catch (e) {
+              log('Error executing impacted query $e');
+            }
           }
         }
       });

packages/firebase_data_connect/firebase_data_connect/pubspec.yaml

@@ -21,11 +21,11 @@ dependencies:
   grpc: ^3.2.4
   http: ^1.2.1
   intl: ^0.20.2
+  path: ^1.9.0
+  path_provider: ^2.0.0
   protobuf: ^3.1.0
   sqlite3: ^2.9.0
   sqlite3_flutter_libs: ^0.5.40
-  path_provider: ^2.0.0
-  path: ^1.9.0
 
 dev_dependencies:
   build_runner: ^2.4.12

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

@@ -32,7 +32,6 @@ import 'dart:convert';
 import 'package:mockito/annotations.dart';
 import 'package:mockito/mockito.dart';
 
-import '../core/ref_test.dart';
 @GenerateNiceMocks([MockSpec<FirebaseApp>(), MockSpec<ConnectorConfig>()])
 import '../firebase_data_connect_test.mocks.dart';
 import '../network/rest_transport_test.mocks.dart';
@@ -43,17 +42,12 @@ class MockDataConnectTransport extends Mock implements DataConnectTransport {}
 
 void main() {
   late MockFirebaseApp mockApp;
-  late MockFirebaseAuth mockAuth;
   late MockConnectorConfig mockConnectorConfig;
   late FirebaseDataConnect dataConnect;
   late MockClient mockHttpClient;
   late RestTransport transport;
   const Duration maxAgeSeconds = Duration(milliseconds: 200);
 
-  const String entityObject = '''
-    {"desc":"itemDesc1","name":"itemOne", "cacheId":"123","price":4}
-  ''';
-
   const String simpleQueryResponse = '''
     {"data": {"items":[
     

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

@@ -12,14 +12,10 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-import 'package:firebase_data_connect/firebase_data_connect.dart';
-import 'package:firebase_data_connect/src/cache/cache_data_types.dart';
 import 'package:firebase_data_connect/src/cache/result_tree_processor.dart';
-import 'package:firebase_data_connect/src/common/common_library.dart';
 
 import 'package:flutter_test/flutter_test.dart';
 import 'dart:convert';
-import 'dart:collection';
 
 import 'package:firebase_data_connect/src/cache/in_memory_cache_provider.dart';
 

packages/firebase_data_connect/firebase_data_connect/test/src/network/rest_transport_test.dart

@@ -87,30 +87,6 @@ void main() {
       );
     });
 
-    test('invokeOperation should return deserialized data', () async {
-      final mockResponse = http.Response('{"data": {"key": "value"}}', 200);
-      when(
-        mockHttpClient.post(
-          any,
-          headers: anyNamed('headers'),
-          body: anyNamed('body'),
-        ),
-      ).thenAnswer((_) async => mockResponse);
-
-      final deserializer = (String data) => 'Deserialized Data';
-
-      final result = await transport.invokeOperation(
-        'testQuery',
-        'executeQuery',
-        deserializer,
-        null,
-        null,
-        null,
-      );
-
-      expect(result, 'Deserialized Data');
-    });
-
     test('invokeOperation should throw unauthorized error on 401 response',
         () async {
       final mockResponse = http.Response('Unauthorized', 401);