fix(data_connect): avoid calling toJson on raw JSON map or null object (#17356)

* fix(data_connect): avoid calling toJson on raw JSON map or null object * chore: add null values to test case

Author: SelaseKay

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

@@ -27,7 +27,7 @@ class AnyValue {
  return value;
  } else {
  if (value is List) {
- return (value as List).map((e) => e.toJson()).toList();
+ return (value as List).map((e) => AnyValue(e).toJson()).toList();
  } else if (value is Map) {
  // TODO(mtewani): Throw an error if this is the wrong type.
  return convertMap(value as Map<String, dynamic>);

packages/firebase_data_connect/firebase_data_connect/test/src/any_test.dart

@@ -59,5 +59,28 @@ void main() {
  final any = AnyValue(map);
  expect(any.toJson(), equals(map));
  });
+ test('constructor serializes List of map', () {
+ final listOfMap = [
+ {'a': 1, 'b': 2.0},
+ {'c': 3, 'd': 4.0},
+ {'e': 5, 'f': null},
+ ];
+ final any = AnyValue(listOfMap);
+ expect(any.toJson(), equals(listOfMap));
+ });
+ test('constructor serializes List of primitive type', () {
+ final cases = [
+ [1, 2, 3, 4, 5],
+ [1.0, 2.0, 3.0, 4.0, 5.0],
+ ['a', 'b', 'c', 'd', 'e'],
+ [true, false, true, false],
+ [1, 2.0, null, 4],
+ ];
+
+ for (final list in cases) {
+ final any = AnyValue(list);
+ expect(any.toJson(), equals(list));
+ }
+ });
  });
 }