Feedback.

Author: cincuranet

test/EFCore.Sqlite.FunctionalTests/Query/PrimitiveCollectionsQuerySqliteTest.cs

@@ -15895,39 +15895,39 @@ FROM json_each("p"."Ints") AS "i"
 """);
     }
 
-    // TODO: The base implementations no longer compile since https://github.com/dotnet/runtime/pull/110197 (Contains overload added with
-    // optional parameter, not supported in expression trees). #35547 is tracking on the EF side.
-    //
-    //
-    //     public override async Task Column_collection_of_nullable_ints_Contains(bool async)
-    //     {
-    //         await base.Column_collection_of_nullable_ints_Contains(async);
-    //
-    //         AssertSql(
-    //             """
-    // SELECT "p"."Id", "p"."Bool", "p"."Bools", "p"."DateTime", "p"."DateTimes", "p"."Enum", "p"."Enums", "p"."Int", "p"."Ints", "p"."NullableInt", "p"."NullableInts", "p"."NullableString", "p"."NullableStrings", "p"."NullableWrappedId", "p"."NullableWrappedIdWithNullableComparer", "p"."String", "p"."Strings", "p"."WrappedId"
-    // FROM "PrimitiveCollectionsEntity" AS "p"
-    // WHERE 10 IN (
-    //     SELECT "n"."value"
-    //     FROM json_each("p"."NullableInts") AS "n"
-    // )
-    // """);
-    //     }
-    //
-    //     public override async Task Column_collection_of_nullable_ints_Contains_null(bool async)
-    //     {
-    //         await base.Column_collection_of_nullable_ints_Contains_null(async);
-    //
-    //         AssertSql(
-    //             """
-    // SELECT "p"."Id", "p"."Bool", "p"."Bools", "p"."DateTime", "p"."DateTimes", "p"."Enum", "p"."Enums", "p"."Int", "p"."Ints", "p"."NullableInt", "p"."NullableInts", "p"."NullableString", "p"."NullableStrings", "p"."NullableWrappedId", "p"."NullableWrappedIdWithNullableComparer", "p"."String", "p"."Strings", "p"."WrappedId"
-    // FROM "PrimitiveCollectionsEntity" AS "p"
-    // WHERE EXISTS (
-    //     SELECT 1
-    //     FROM json_each("p"."NullableInts") AS "n"
-    //     WHERE "n"."value" IS NULL)
-    // """);
-    //     }
+// TODO: The base implementations no longer compile since https://github.com/dotnet/runtime/pull/110197 (Contains overload added with
+// optional parameter, not supported in expression trees). #35547 is tracking on the EF side.
+//
+//
+//     public override async Task Column_collection_of_nullable_ints_Contains(bool async)
+//     {
+//         await base.Column_collection_of_nullable_ints_Contains(async);
+//
+//         AssertSql(
+//             """
+// SELECT "p"."Id", "p"."Bool", "p"."Bools", "p"."DateTime", "p"."DateTimes", "p"."Enum", "p"."Enums", "p"."Int", "p"."Ints", "p"."NullableInt", "p"."NullableInts", "p"."NullableString", "p"."NullableStrings", "p"."NullableWrappedId", "p"."NullableWrappedIdWithNullableComparer", "p"."String", "p"."Strings", "p"."WrappedId"
+// FROM "PrimitiveCollectionsEntity" AS "p"
+// WHERE 10 IN (
+//     SELECT "n"."value"
+//     FROM json_each("p"."NullableInts") AS "n"
+// )
+// """);
+//     }
+//
+//     public override async Task Column_collection_of_nullable_ints_Contains_null(bool async)
+//     {
+//         await base.Column_collection_of_nullable_ints_Contains_null(async);
+//
+//         AssertSql(
+//             """
+// SELECT "p"."Id", "p"."Bool", "p"."Bools", "p"."DateTime", "p"."DateTimes", "p"."Enum", "p"."Enums", "p"."Int", "p"."Ints", "p"."NullableInt", "p"."NullableInts", "p"."NullableString", "p"."NullableStrings", "p"."NullableWrappedId", "p"."NullableWrappedIdWithNullableComparer", "p"."String", "p"."Strings", "p"."WrappedId"
+// FROM "PrimitiveCollectionsEntity" AS "p"
+// WHERE EXISTS (
+//     SELECT 1
+//     FROM json_each("p"."NullableInts") AS "n"
+//     WHERE "n"."value" IS NULL)
+// """);
+//     }
 
     public override async Task Column_collection_of_strings_contains_null(bool async)
     {