Merge branch 'firestore-pipelines-dart-api-v2' into firestore-pipelines-android

Author: SelaseKay

packages/cloud_firestore/cloud_firestore/lib/src/pipeline.dart

@@ -211,8 +211,8 @@ class Pipeline {
   ///
   /// Example:
   /// ```dart
-  /// pipeline.aggregateStage(
-  ///   AggregateStage(
+  /// pipeline.aggregateWithOptions(
+  ///   AggregateStageOptions(
   ///     accumulators: [
   ///       Expression.field('likes').sum().as('total_likes'),
   ///       Expression.field('likes').average().as('avg_likes'),
@@ -224,8 +224,8 @@ class Pipeline {
   ///
   /// With options:
   /// ```dart
-  /// pipeline.aggregateStage(
-  ///   AggregateStage(
+  /// pipeline.aggregateWithOptions(
+  ///   AggregateStageOptions(
   ///     accumulators: [
   ///       Expression.field('likes').sum().as('total_likes'),
   ///     ],

packages/cloud_firestore/cloud_firestore/lib/src/pipeline_expression.dart

@@ -379,7 +379,10 @@ abstract class Expression implements PipelineSerializable {
   /// Replaces all occurrences of a string literal
   Expression stringReplaceAllLiteral(String find, String replacement) {
     return _StringReplaceAllExpression(
-        this, Constant(find), Constant(replacement));
+      this,
+      Constant(find),
+      Constant(replacement),
+    );
   }
 
   /// Splits this string expression by a delimiter

packages/cloud_firestore/cloud_firestore/test/pipeline_expression_test.dart

@@ -750,24 +750,24 @@ void main() {
 
     test('timestampAddLiteral serializes correctly', () {
       final ts = Field('created');
-      final expr = Expression.timestampAddLiteral(ts, 'days', 1);
+      final expr = Expression.timestampAddLiteral(ts, 'day', 1);
       expect(expr.toMap(), {
         'name': 'timestamp_add',
         'args': {
           'timestamp': ts.toMap(),
-          'unit': 'days',
+          'unit': 'day',
           'amount': Constant(1).toMap(),
         },
       });
     });
 
     test('timestampTruncate serializes correctly', () {
-      final expr = Expression.timestampTruncate(Field('ts'), 'days');
+      final expr = Expression.timestampTruncate(Field('ts'), 'day');
       expect(expr.toMap(), {
         'name': 'timestamp_truncate',
         'args': {
           'timestamp': Field('ts').toMap(),
-          'unit': 'days',
+          'unit': 'day',
         },
       });
     });