PR changes

Author: simonsabin

src/Config/DatabasePrimitives/DatabaseObject.cs

@@ -235,7 +235,6 @@ public bool IsAnyColumnNullable(List<string> columnsToCheck)
         if (Columns.TryGetValue(paramName, out ColumnDefinition? columnDefinition))
         {
             return columnDefinition.Length;
-            ;
         }
 
         return null;

src/Service.Tests/SqlTests/GraphQLQueryTests/MsSqlGraphQLQueryTests.cs

@@ -202,6 +202,7 @@ public virtual async Task TestFilterParamForStringFilter()
         /// the string being passed as a parameter maybe longer than the length of the column. The parameter data type
         /// can't be fixed to the length of the underlying column, otherwise the parameter value would be truncated and
         /// we'd get incorret results
+        /// Thus checking the parameter length is overridden to cater for teh extra length i.e. lengthOverride = true codepath?
         /// </remarks>
         [DataTestMethod]
         [DataRow("contains")]
@@ -241,7 +242,8 @@ public virtual async Task TestFilterParamForStringFilterWorkWithComplexOp(string
         /// When using complex operators i.e. NotContains due to wildcards being added or special characters being escaped
         /// the string being passed as a parameter maybe longer than the length of the column. The parameter data type
         /// can't be fixed to the length of the underlying column, otherwise the parameter value would be truncated and
-        /// we'd get incorret results
+        /// we'd get incorret results.
+        /// Thus checking the parameter length is overridden to cater for teh extra length i.e. lengthOverride = true codepath?
         /// </remarks>
         [TestMethod]
         public virtual async Task TestFilterParamForStringFilterWorkWithNotContains(string op)