From cd60f5826bfd70ab286a63088bf4153f7efd75e4 Mon Sep 17 00:00:00 2001
From: James Kovacs <jkovacs@post.harvard.edu>
Date: Tue, 8 Mar 2022 18:06:36 -0700
Subject: [PATCH] CSHARP-4089: Move TranslationContextData to be internal.
 (#748)

---
 .../AggregateExpressionDefinition.cs          | 10 +++++--
 .../Translators/TranslationContextData.cs     | 28 +------------------
 2 files changed, 8 insertions(+), 30 deletions(-)

diff --git a/src/MongoDB.Driver/AggregateExpressionDefinition.cs b/src/MongoDB.Driver/AggregateExpressionDefinition.cs
index be0593e9407..7bc36877946 100644
--- a/src/MongoDB.Driver/AggregateExpressionDefinition.cs
+++ b/src/MongoDB.Driver/AggregateExpressionDefinition.cs
@@ -124,11 +124,15 @@ public sealed class ExpressionAggregateExpressionDefinition<TSource, TResult> :
         /// </summary>
         /// <param name="expression">The expression.</param>
         /// <param name="translationOptions">The translation options.</param>
-        /// <param name="contextData">Any optional data for the TranslationContext.</param>
-        public ExpressionAggregateExpressionDefinition(
+        public ExpressionAggregateExpressionDefinition(Expression<Func<TSource, TResult>> expression, ExpressionTranslationOptions translationOptions)
+            : this(expression, translationOptions, null)
+        {
+        }
+
+        internal ExpressionAggregateExpressionDefinition(
             Expression<Func<TSource, TResult>> expression,
             ExpressionTranslationOptions translationOptions,
-            TranslationContextData contextData = null)
+            TranslationContextData contextData)
         {
             _expression = Ensure.IsNotNull(expression, nameof(expression));
             _translationOptions = translationOptions; // can be null
diff --git a/src/MongoDB.Driver/Linq/Linq3Implementation/Translators/TranslationContextData.cs b/src/MongoDB.Driver/Linq/Linq3Implementation/Translators/TranslationContextData.cs
index e5acf3b5c3d..d2b3faef80a 100644
--- a/src/MongoDB.Driver/Linq/Linq3Implementation/Translators/TranslationContextData.cs
+++ b/src/MongoDB.Driver/Linq/Linq3Implementation/Translators/TranslationContextData.cs
@@ -18,16 +18,10 @@
 
 namespace MongoDB.Driver.Linq.Linq3Implementation.Translators
 {
-    /// <summary>
-    /// Represents arbitrary data for a LINQ3 translation context.
-    /// </summary>
-    public sealed class TranslationContextData
+    internal sealed class TranslationContextData
     {
         private readonly Dictionary<string, object> _data;
 
-        /// <summary>
-        /// Initializes a new instance of a TranslationContextData.
-        /// </summary>
         public TranslationContextData()
             : this(new Dictionary<string, object>())
         {
@@ -38,36 +32,16 @@ private TranslationContextData(Dictionary<string, object> data)
             _data = Ensure.IsNotNull(data, nameof(data));
         }
 
-        /// <summary>
-        /// Gets a value.
-        /// </summary>
-        /// <typeparam name="TValue">The type of the value.</typeparam>
-        /// <param name="key">The key.</param>
-        /// <returns>The value.</returns>
         public TValue GetValue<TValue>(string key)
         {
             return (TValue)_data[key];
         }
 
-        /// <summary>
-        /// Gets a value or a default value if they key is not present.
-        /// </summary>
-        /// <typeparam name="TValue">The type of the value.</typeparam>
-        /// <param name="key">The key.</param>
-        /// <param name="defaultValue">The default value.</param>
-        /// <returns>The value.</returns>
         public TValue GetValueOrDefault<TValue>(string key, TValue defaultValue)
         {
             return _data.TryGetValue(key, out var value) ? (TValue)value : defaultValue;
         }
 
-        /// <summary>
-        /// Returns a new TranslationContextData with an additional value.
-        /// </summary>
-        /// <typeparam name="TValue">The type of the value.</typeparam>
-        /// <param name="key">The key.</param>
-        /// <param name="value">The value.</param>
-        /// <returns>A new TranslationContextData with an additional value</returns>
         public TranslationContextData With<TValue>(string key, TValue value)
         {
             var clonedData = new Dictionary<string, object>(_data);