update some xmldoc 😁

Author: ceastwood

QueryBuilder.Tests/Infrastructure/TestCompilersContainer.cs

@@ -7,6 +7,12 @@ namespace SqlKata.Tests.Infrastructure
 {
     public class TestCompilersContainer
     {
+        private static class Messages
+        {
+            public const string ERR_INVALID_ENGINECODE = "Engine code '{0}' is not valid";
+            public const string ERR_INVALID_ENGINECODES = "Invalid engine codes supplied '{0}'";
+        }
+
         protected readonly IDictionary<string, Compiler> Compilers = new Dictionary<string, Compiler>
         {
             [EngineCodes.Firebird] = new FirebirdCompiler(),
@@ -23,15 +29,15 @@ public IEnumerable<string> KnownEngineCodes
         }
 
         /// <summary>
-        /// Returns the compiler instance for the given <param name="engineCode"></param>
+        /// Returns a <see cref="Compiler"/> instance for the given engine code
         /// </summary>
         /// <param name="engineCode"></param>
         /// <returns></returns>
         public Compiler Get(string engineCode)
         {
             if (!Compilers.ContainsKey(engineCode))
             {
-                throw new InvalidOperationException($"Engine code '{engineCode}' is not valid");
+                throw new InvalidOperationException(string.Format(Messages.ERR_INVALID_ENGINECODE, engineCode));
             }
 
             return Compilers[engineCode];
@@ -50,7 +56,7 @@ public TCompiler Get<TCompiler>(string engineCode) where TCompiler : Compiler
         }
 
         /// <summary>
-        /// Compiles the query for the given <param name="engineCode"></param>
+        /// Compiles the <see cref="Query"/> against the given engine code
         /// </summary>
         /// <param name="engineCode"></param>
         /// <param name="query"></param>
@@ -61,6 +67,12 @@ public SqlResult CompileFor(string engineCode, Query query)
             return compiler.Compile(query);
         }
 
+        /// <summary>
+        /// Compiles the <see cref="Query"/> against the given engine codes
+        /// </summary>
+        /// <param name="engineCodes"></param>
+        /// <param name="query"></param>
+        /// <returns></returns>
         public TestSqlResultContainer Compile(IEnumerable<string> engineCodes, Query query)
         {
             var codes = engineCodes.ToList();
@@ -72,14 +84,15 @@ public TestSqlResultContainer Compile(IEnumerable<string> engineCodes, Query que
             if (results.Count != codes.Count)
             {
                 var missingCodes = codes.Where(w => Compilers.All(a => a.Key != w));
-                throw new InvalidOperationException($"Invalid engine codes supplied '{string.Join(", ", missingCodes)}'");
+                var templateArg = string.Join(", ", missingCodes);
+                throw new InvalidOperationException(string.Format(Messages.ERR_INVALID_ENGINECODES, templateArg));
             }
 
             return new TestSqlResultContainer(results);
         }
 
         /// <summary>
-        /// Compiles the query for all known compilers
+        /// Compiles the <see cref="Query"/> against all <see cref="Compiler"/>s
         /// </summary>
         /// <param name="query"></param>
         /// <returns></returns>