Update TabularDataByMQL to accept list of BSON docs instead of String…

…. Resolves #RSDK-5873. (#154)
viamrobotics · Nov 30, 2023 · b52eaae · b52eaae
1 parent fbc03ba
commit b52eaae
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/lib/src/app/data.dart b/lib/src/app/data.dart
@@ -1,5 +1,6 @@
 import 'dart:io';
 import 'dart:math';
+import 'dart:typed_data';
 
 import 'package:async/async.dart';
 import 'package:collection/collection.dart';
@@ -124,10 +125,10 @@ class DataClient {
   }
 
   /// Obtain unified tabular data and metadata, queried with MQL.
-  Future<List<Map<String, dynamic>>> tabularDataByMql(String organizationId, String query) async {
+  Future<List<Map<String, dynamic>>> tabularDataByMql(String organizationId, List<Uint8List> query) async {
     final request = TabularDataByMQLRequest()
       ..organizationId = organizationId
-      ..mqlQuery = query;
+      ..mqlBinary.addAll(query);
     final response = await _dataClient.tabularDataByMQL(request);
     return response.data.map((e) => e.toMap()).toList();
   }

diff --git a/test/unit_test/app/data_client_test.dart b/test/unit_test/app/data_client_test.dart
@@ -1,3 +1,5 @@
+import 'dart:typed_data';
+
 import 'package:fixnum/fixnum.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:grpc/src/client/call.dart';
@@ -145,7 +147,7 @@ void main() {
         when(serviceClient.tabularDataByMQL(any))
             .thenAnswer((_) => MockResponseFuture.value(TabularDataByMQLResponse()..data.addAll(data.map((e) => e.toStruct()))));
 
-        final response = await dataClient.tabularDataByMql('some_org_id', 'some_query');
+        final response = await dataClient.tabularDataByMql('some_org_id', [Uint8List.fromList('some_query'.codeUnits)]);
         expect(response, equals(data));
       });
     });